Re: cvs: ZendEngine2 / zend_execute_API.c

Marcus Boerger <[email protected]>
Newsgroups gmane.comp.php.cvs.zend
Message-ID <[email protected]>
Hello Stanislav,

  sure, something might rely on function table or classe tables being
intact when freeing object storage. This is for instance the case with
reflection.

marcus

Monday, August 11, 2008, 7:36:37 PM, you wrote:

> Hi!

> Could you explain the reason for this change?

>>  #include <stdio.h>
>>  #include <signal.h>
>> @@ -311,6 +311,8 @@
>>  
>>               zend_vm_stack_destroy(TSRMLS_C);
>>  
>> +             zend_objects_store_free_object_storage(&EG(objects_store) TSRMLS_CC);
>> +
>>               /* Destroy all op arrays */
>>               if (EG(full_tables_cleanup)) {
>>                       zend_hash_apply(EG(function_table), (apply_func_t) clean_non_persistent_function_full TSRMLS_CC);
>> @@ -325,7 +327,6 @@
>>                       FREE_HASHTABLE(*EG(symtable_cache_ptr));
>>                       EG(symtable_cache_ptr)--;
>>               }
>> -             zend_objects_store_free_object_storage(&EG(objects_store) TSRMLS_CC);
>>       } zend_end_try();
>>  
>>       zend_try {
>> 
>> 
>> 

> -- 
> Stanislav Malyshev, Zend Software Architect
> [email protected]   http://www.zend.com/
> (408)253-8829   MSN: [email protected]




Best regards,
 Marcus


-- 
Zend Engine CVS Mailing List (http://cvs.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.