diff options
| author | Dmitry Stogov <dmitry@php.net> | 2006-05-31 13:02:15 +0000 |
|---|---|---|
| committer | Dmitry Stogov <dmitry@php.net> | 2006-05-31 13:02:15 +0000 |
| commit | 4e662470f77957e6edf0ca867a9e73888ec5fbb7 (patch) | |
| tree | fbe12149f815ee84abb144150484ce86aed1ac82 | |
| parent | 75fac72dc52ab24aaa14a2a1b520752961099044 (diff) | |
| download | php-git-4e662470f77957e6edf0ca867a9e73888ec5fbb7.tar.gz | |
Keeping consistent arg_stack during arguments freeing (Exception from destructor may use inconsistent arg_stack for backtrace).
| -rw-r--r-- | Zend/zend_builtin_functions.c | 10 | ||||
| -rw-r--r-- | Zend/zend_execute.h | 4 |
2 files changed, 10 insertions, 4 deletions
diff --git a/Zend/zend_builtin_functions.c b/Zend/zend_builtin_functions.c index bf75789423..c544badf04 100644 --- a/Zend/zend_builtin_functions.c +++ b/Zend/zend_builtin_functions.c @@ -1708,9 +1708,13 @@ static zval *debug_backtrace_get_args(void ***curpos TSRMLS_DC) while (--arg_count >= 0) { arg = (zval **) p++; - SEPARATE_ZVAL_TO_MAKE_IS_REF(arg); - (*arg)->refcount++; - add_next_index_zval(arg_array, *arg); + if (*arg) { + SEPARATE_ZVAL_TO_MAKE_IS_REF(arg); + (*arg)->refcount++; + add_next_index_zval(arg_array, *arg); + } else { + add_next_index_null(arg_array); + } } /* skip args from incomplete frames */ diff --git a/Zend/zend_execute.h b/Zend/zend_execute.h index 74773f2036..09432f77b4 100644 --- a/Zend/zend_execute.h +++ b/Zend/zend_execute.h @@ -152,7 +152,9 @@ static inline void zend_ptr_stack_clear_multiple(TSRMLS_D) EG(argument_stack).top -= (delete_count+2); while (--delete_count>=0) { - zval_ptr_dtor((zval **) --p); + zval *q = *(zval **)(--p); + *p = NULL; + zval_ptr_dtor(&q); } EG(argument_stack).top_element = p; } |
