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

Reply via email to