From: Jérôme Loyet Date: Mon, 18 Jul 2011 21:03:44 +0000 (+0000) Subject: fix a segfault when passing an empty value to a ini parameter from the web server... X-Git-Tag: php-5.3.7RC4~39 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=627540cc0bb6a13c0789d54035777d7c2fc7b952;p=php fix a segfault when passing an empty value to a ini parameter from the web server (php_(admin_)?value) --- diff --git a/sapi/fpm/fpm/fpm_main.c b/sapi/fpm/fpm/fpm_main.c index c7ad49e40f..c2db60f111 100644 --- a/sapi/fpm/fpm/fpm_main.c +++ b/sapi/fpm/fpm/fpm_main.c @@ -1430,22 +1430,28 @@ static void init_request_info(TSRMLS_D) static void fastcgi_ini_parser(zval *arg1, zval *arg2, zval *arg3, int callback_type, void *arg TSRMLS_DC) /* {{{ */ { int *mode = (int *)arg; - char *key = Z_STRVAL_P(arg1); - char *value = Z_STRVAL_P(arg2); + char *key; + char *value = NULL; struct key_value_s kv; - if (!mode) return; + if (!mode || !arg1) return; if (callback_type != ZEND_INI_PARSER_ENTRY) { fprintf(stderr, "Passing INI directive through FastCGI: only classic entries are allowed\n"); return; } + key = Z_STRVAL_P(arg1); + if (!key || strlen(key) < 1) { fprintf(stderr, "Passing INI directive through FastCGI: empty key\n"); return; } + if (arg2) { + value = Z_STRVAL_P(arg2); + } + if (!value) { fprintf(stderr, "Passing INI directive through FastCGI: empty value for key '%s'\n", key); return;