Date: 01/04/01
- Next message: tvkbhaskar2011 <email protected>: "[PHP-DEV] PHP 4.0 Bug #8545: Session variables not carried forward in Netscape Client"
- Previous message: Steven Roussey: "[PHP-DEV] Bug 5474"
- Next in thread: Andi Gutmans: "Re: [PHP-DEV] Re: Bug 5474 [FIX: change to zend engine included]"
- Reply: Andi Gutmans: "Re: [PHP-DEV] Re: Bug 5474 [FIX: change to zend engine included]"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hi!
I wanted to find a fix before I sent off the bug report, but I messed up. I
found a fix. Destructors for all persistent objects belonging to any
extension were never called.
This routine needs to be replaced in Zend/zend_list.c:
void plist_entry_destructor(void *ptr)
{
zend_rsrc_list_entry *le = (zend_rsrc_list_entry *) ptr;
zend_rsrc_list_dtors_entry *ld;
if (zend_hash_index_find(&list_destructors, le->type,(void **)
&ld)==SUCCESS) {
switch (ld->type) {
case ZEND_RESOURCE_LIST_TYPE_STD:
if (ld->plist_dtor) {
(ld->plist_dtor)(le->ptr);
}
break;
case ZEND_RESOURCE_LIST_TYPE_EX:
if (ld->plist_dtor_ex) {
ld->plist_dtor_ex(le);
}
break;
EMPTY_SWITCH_DEFAULT_CASE()
}
} else {
zend_error(E_WARNING,"Unknown persistent list entry type in
module shutdown (%d)",le->type);
}
}
That should do it. Should help MySQL, pgsql, mssql, and others.
Will run previously untested code, so additional debugging of other code is
advised.
Sincerely,
Steven Roussey
Network54.com
http://network54.com/?pp=e
-- PHP Development Mailing List <http://www.php.net/> To unsubscribe, e-mail: php-dev-unsubscribe <email protected> For additional commands, e-mail: php-dev-help <email protected> To contact the list administrators, e-mail: php-list-admin <email protected>
- Next message: tvkbhaskar2011 <email protected>: "[PHP-DEV] PHP 4.0 Bug #8545: Session variables not carried forward in Netscape Client"
- Previous message: Steven Roussey: "[PHP-DEV] Bug 5474"
- Next in thread: Andi Gutmans: "Re: [PHP-DEV] Re: Bug 5474 [FIX: change to zend engine included]"
- Reply: Andi Gutmans: "Re: [PHP-DEV] Re: Bug 5474 [FIX: change to zend engine included]"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

