ID: 9764
Updated by: sniper
Reported By: [EMAIL PROTECTED]
Old-Status: Open
Status: Closed
Bug Type: PHP options/info functions
Assigned To: 
Comments:

Fixed in CVS. (By Zeev, but not with this patch..)

--Jani


Previous Comments:
---------------------------------------------------------------------------

[2001-03-15 06:43:56] [EMAIL PROTECTED]
PHP4 under Apache tries to load extensions from php.ini

several times per process causing 'duplicate name' warnings

Also on the second (and next) loading extension name passed

to phpdl() in damaged form.



Investigation showed that problem is in the main/php_ini.c:

1. php_load_function_extension_cb() allocates extension name using estrdup but global 
llist with extensions declared as persistent. It keeps references to deallocated 
extensions' names causing damaged names passed to phpdl(). 

2. php_startup_loaded_extensions() doesn't free llist after

loading extensions, so next call to php_init_config() (per-process) will double this 
list.



Solution is to deallocate llist after loading extensions in 
php_startup_loaded_extensions. Then (1) wouldn't be a problem at all.



Proposed fix:



--- main/php_ini.c.orig Tue Mar  6 13:38:55 2001

+++ main/php_ini.c      Thu Mar 15 13:12:04 2001

@@ -165,6 +165,8 @@

 {

        if(php_load_extension_list) {

                zend_llist_apply(php_load_extension_list, 
php_startup_loaded_extension_cb);

+               zend_llist_destroy(php_load_extension_list);

+               php_load_extension_list = NULL;

        }

        return SUCCESS;

}

---------------------------------------------------------------------------



ATTENTION! Do NOT reply to this email!
To reply, use the web interface found at http://bugs.php.net/?id=9764&edit=2


-- 
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]

Reply via email to