diff options
author | Dmitry Stogov <dmitry@zend.com> | 2015-04-16 21:45:40 +0300 |
---|---|---|
committer | Dmitry Stogov <dmitry@zend.com> | 2015-04-16 21:45:40 +0300 |
commit | 5b0d86edccb669b06bcf9f225a981f16c03da71a (patch) | |
tree | d4a6aa41b030e9ad61e45775e1d001e5c9e7affe /Zend/zend_execute.h | |
parent | e44ccde39a17cd316f3ab7fbf142d8d72546df68 (diff) | |
download | php-git-5b0d86edccb669b06bcf9f225a981f16c03da71a.tar.gz |
Safe destruction (variables and arguments may need to be cuptured by exception)
Diffstat (limited to 'Zend/zend_execute.h')
-rw-r--r-- | Zend/zend_execute.h | 18 |
1 files changed, 16 insertions, 2 deletions
diff --git a/Zend/zend_execute.h b/Zend/zend_execute.h index 4dfed1830d..67958cf32c 100644 --- a/Zend/zend_execute.h +++ b/Zend/zend_execute.h @@ -199,7 +199,15 @@ static zend_always_inline void zend_vm_stack_free_extra_args_ex(uint32_t call_in zval *p = end + (ZEND_CALL_NUM_ARGS(call) - call->func->op_array.num_args); do { p--; - i_zval_ptr_dtor(p ZEND_FILE_LINE_CC); + if (Z_REFCOUNTED_P(p)) { + if (!Z_DELREF_P(p)) { + zend_refcounted *r = Z_COUNTED_P(p); + ZVAL_NULL(p); + zval_dtor_func_for_ptr(r); + } else { + GC_ZVAL_CHECK_POSSIBLE_ROOT(p); + } + } } while (p != end); } } @@ -219,7 +227,13 @@ static zend_always_inline void zend_vm_stack_free_args(zend_execute_data *call) do { p--; - zval_ptr_dtor_nogc(p); + if (Z_REFCOUNTED_P(p)) { + if (!Z_DELREF_P(p)) { + zend_refcounted *r = Z_COUNTED_P(p); + ZVAL_NULL(p); + zval_dtor_func_for_ptr(r); + } + } } while (p != end); } } |