com php-src: -- missing length declaration, may have worked w/some gcc as length is not used in the macro but still need it, ifdef just make code
uncertain. will get opt out anyway if not used: main/php_ini .c
| From: | Pierre Joye | Date: | Wed, 07 May 2014 05:31:21 +0000 |
| Subject: | com php-src: -- missing length declaration, may have worked w/some gcc as length is not used in the macro but still need it, ifdef just make code uncertain. will get opt out anyway if not used: main/php_ini .c |
||
| Groups: | php.cvs | ||
| Request: | Send a blank email to php-cvs+get-78167@lists.php.net to get a copy of this message | ||
Commit: 38c886a449a54574119a8eb1c6c5e5376acba6d8
Author: Pierre Joye <pierre.php@gmail.com> Wed, 7 May 2014 07:31:21 +0200
Parents: 478a8b2ebbd6a5fd2e49c3e1069b284b1a38c018
Branches: phpng
Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=38c886a449a54574119a8eb1c6c5e5376acba6d8
Log:
-- missing length declaration, may have worked w/some gcc as length is not used in the macro but
still need it, ifdef just make code uncertain. will get opt out anyway if not used
Changed paths:
M main/php_ini.c
Diff:
diff --git a/main/php_ini..c b/main/php_ini.c
index 2b62e80..d742712 100644
--- a/main/php_ini.c
+++ b/main/php_ini.c
@@ -355,6 +355,7 @@ static void php_load_php_extension_cb(void *arg TSRMLS_DC)
static void php_load_zend_extension_cb(void *arg TSRMLS_DC)
{
char *filename = *((char **) arg);
+ const int length = strlen(filename);
if (IS_ABSOLUTE_PATH(filename, length)) {
zend_load_extension(filename TSRMLS_CC);