From: Felipe Pena <felipe@php.net>
Date: Mon, 23 May 2011 01:47:06 +0000 (+0000)
Subject: - Fixed bug #54529 (SAPI crashes on apache_config.c:197)
X-Git-Tag: php-5.4.0alpha1~147
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a95a02150927401ed3ab7cb9a775eccf6e8025c7;p=php

- Fixed bug #54529 (SAPI crashes on apache_config.c:197)
  patch by: hebergement at riastudio dot fr
---

diff --git a/sapi/apache2handler/apache_config.c b/sapi/apache2handler/apache_config.c
index e48e23b135..37a2195ab9 100644
--- a/sapi/apache2handler/apache_config.c
+++ b/sapi/apache2handler/apache_config.c
@@ -192,11 +192,12 @@ void apply_config(void *dummy)
 			zend_hash_get_current_key_ex(&d->config, &str, &str_len, NULL, 0, 
 				NULL) == HASH_KEY_IS_STRING;
 			zend_hash_move_forward(&d->config)) {
-		zend_hash_get_current_data(&d->config, (void **) &data);
-		phpapdebug((stderr, "APPLYING (%s)(%s)\n", str, data->value));
-		if (zend_alter_ini_entry(str, str_len, data->value, data->value_len, data->status, data->htaccess?PHP_INI_STAGE_HTACCESS:PHP_INI_STAGE_ACTIVATE) == FAILURE) {
-			phpapdebug((stderr, "..FAILED\n"));
-		}	
+		if (zend_hash_get_current_data(&d->config, (void **) &data) == SUCCESS) {
+			phpapdebug((stderr, "APPLYING (%s)(%s)\n", str, data->value));
+			if (zend_alter_ini_entry(str, str_len, data->value, data->value_len, data->status, data->htaccess?PHP_INI_STAGE_HTACCESS:PHP_INI_STAGE_ACTIVATE) == FAILURE) {
+				phpapdebug((stderr, "..FAILED\n"));
+			}
+		}
 	}
 }