]> granicus.if.org Git - php/commit
Don't use zend_try around dtor_obj() and free_obj()
authorNikita Popov <nikita.ppv@gmail.com>
Tue, 2 May 2017 20:32:33 +0000 (22:32 +0200)
committerNikita Popov <nikita.ppv@gmail.com>
Thu, 4 May 2017 18:57:25 +0000 (20:57 +0200)
commit5af997ed81fc3e86544679d7a07b152ee13d1e04
tree4cc121cc092f272b2ed3c660f494f48b5da11682
parent3abef9c9609eef04f268e1fd002e6ca7edac11fa
Don't use zend_try around dtor_obj() and free_obj()

Without these zend_trys we may
a) leak the object (fine)
b) don't add it to the free list (also fine, we just lose an
   object store bucket)
c) don't remove it from the GC root buffer (also fine, because GC
   explicitly checks for both FREE_CALLED and invalid buckets)
Zend/zend_objects_API.c