scottmac Mon Jun 18 15:53:15 2007 UTC Modified files: (Branch: PHP_4_4) /php-src NEWS /php-src/sapi/apache mod_php4.c Log: MFB: Fixed bug #41628 (PHP settings leak between Virtual Hosts in Apache 1.3). http://cvs.php.net/viewvc.cgi/php-src/NEWS?r1=1.1247.2.920.2.237&r2=1.1247.2.920.2.238&diff_format=u Index: php-src/NEWS diff -u php-src/NEWS:1.1247.2.920.2.237 php-src/NEWS:1.1247.2.920.2.238 --- php-src/NEWS:1.1247.2.920.2.237 Tue Jun 12 12:56:17 2007 +++ php-src/NEWS Mon Jun 18 15:53:15 2007 @@ -10,6 +10,8 @@ input variables. Fix for MOPB-03-2007. (Stas) - Fixed bug #41630 (segfault when an invalid color index is present in the image data). (Reported by Elliot <[EMAIL PROTECTED] dot com>) (Pierre) +- Fixed bug #41628 (PHP settings leak between Virtual Hosts in + Apache 1.3). (Scott, manuel at mausz dot at) - Fixed bug #38798 (OpenSSL init corrected in php5 but not in php4). (Tony) 04 May 2007, Version 4.4.7 http://cvs.php.net/viewvc.cgi/php-src/sapi/apache/mod_php4.c?r1=1.146.2.15.2.4&r2=1.146.2.15.2.5&diff_format=u Index: php-src/sapi/apache/mod_php4.c diff -u php-src/sapi/apache/mod_php4.c:1.146.2.15.2.4 php-src/sapi/apache/mod_php4.c:1.146.2.15.2.5 --- php-src/sapi/apache/mod_php4.c:1.146.2.15.2.4 Mon Jan 1 09:46:51 2007 +++ php-src/sapi/apache/mod_php4.c Mon Jun 18 15:53:15 2007 @@ -17,7 +17,7 @@ | PHP 4.0 patches by Zeev Suraski <[EMAIL PROTECTED]> | +----------------------------------------------------------------------+ */ -/* $Id: mod_php4.c,v 1.146.2.15.2.4 2007/01/01 09:46:51 sebastian Exp $ */ +/* $Id: mod_php4.c,v 1.146.2.15.2.5 2007/06/18 15:53:15 scottmac Exp $ */ #include "php_apache_http.h" #include "http_conf_globals.h" @@ -728,9 +728,15 @@ */ static void *php_merge_dir(pool *p, void *basev, void *addv) { - /* This function *must* return addv, and not modify basev */ - zend_hash_merge_ex((HashTable *) addv, (HashTable *) basev, (copy_ctor_func_t) copy_per_dir_entry, sizeof(php_per_dir_entry), (zend_bool (*)(void *, void *)) should_overwrite_per_dir_entry); - return addv; + /* This function *must* not modify addv or basev */ + HashTable *new; + + /* need a copy of addv to merge */ + new = php_create_dir(p, "php_merge_dir"); + zend_hash_copy(new, (HashTable *) addv, (copy_ctor_func_t) copy_per_dir_entry, NULL, sizeof(php_per_dir_entry)); + + zend_hash_merge_ex(new, (HashTable *) basev, (copy_ctor_func_t) copy_per_dir_entry, sizeof(php_per_dir_entry), (zend_bool (*)(void *, void *)) should_overwrite_per_dir_entry); + return new; } /* }}} */
-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php