Commit: 8aff7f05287db900d51ab49457ef04f39cdcf527 Author: Nikita Popov <ni...@php.net> Fri, 29 Mar 2013 12:36:57 +0100 Parents: 8345abcabb487c128ac5ee29884e0ae7d705bd47 Branches: PHP-5.5 master
Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=8aff7f05287db900d51ab49457ef04f39cdcf527 Log: Fix "passing NULL to non-pointer argument" warnings in intl The second argument to spprintf is a size_t (maximum length). Changed paths: M ext/intl/breakiterator/breakiterator_methods.cpp Diff: diff --git a/ext/intl/breakiterator/breakiterator_methods.cpp b/ext/intl/breakiterator/breakiterator_methods.cpp index 6a61f8c..1a1b4fd 100644 --- a/ext/intl/breakiterator/breakiterator_methods.cpp +++ b/ext/intl/breakiterator/breakiterator_methods.cpp @@ -53,7 +53,7 @@ static void _breakiter_factory(const char *func_name, if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s!", &locale_str, &dummy) == FAILURE) { - spprintf(&msg, NULL, "%s: bad arguments", func_name); + spprintf(&msg, 0, "%s: bad arguments", func_name); intl_error_set(NULL, U_ILLEGAL_ARGUMENT_ERROR, msg, 1 TSRMLS_CC); efree(msg); RETURN_NULL(); @@ -66,7 +66,7 @@ static void _breakiter_factory(const char *func_name, biter = func(Locale::createFromName(locale_str), status); intl_error_set_code(NULL, status TSRMLS_CC); if (U_FAILURE(status)) { - spprintf(&msg, NULL, "%s: error creating BreakIterator", + spprintf(&msg, 0, "%s: error creating BreakIterator", func_name); intl_error_set_custom_msg(NULL, msg, 1 TSRMLS_CC); efree(msg); @@ -201,7 +201,7 @@ static void _breakiter_no_args_ret_int32( object = getThis(); if (zend_parse_parameters_none() == FAILURE) { - spprintf(&msg, NULL, "%s: bad arguments", func_name); + spprintf(&msg, 0, "%s: bad arguments", func_name); intl_error_set(NULL, U_ILLEGAL_ARGUMENT_ERROR, msg, 1 TSRMLS_CC); efree(msg); RETURN_FALSE; @@ -225,7 +225,7 @@ static void _breakiter_int32_ret_int32( object = getThis(); if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &arg) == FAILURE) { - spprintf(&msg, NULL, "%s: bad arguments", func_name); + spprintf(&msg, 0, "%s: bad arguments", func_name); intl_error_set(NULL, U_ILLEGAL_ARGUMENT_ERROR, msg, 1 TSRMLS_CC); efree(msg); RETURN_FALSE; @@ -234,7 +234,7 @@ static void _breakiter_int32_ret_int32( BREAKITER_METHOD_FETCH_OBJECT; if (arg < INT32_MIN || arg > INT32_MAX) { - spprintf(&msg, NULL, "%s: offset argument is outside bounds of " + spprintf(&msg, 0, "%s: offset argument is outside bounds of " "a 32-bit wide integer", func_name); intl_error_set(NULL, U_ILLEGAL_ARGUMENT_ERROR, msg, 1 TSRMLS_CC); efree(msg); -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php