From 8df40bdb313d84bf606ce0052954b8031bc5a91f Mon Sep 17 00:00:00 2001 From: Dmitry Stogov Date: Wed, 31 May 2006 13:01:57 +0000 Subject: [PATCH] Keeping consistent arg_stack during arguments freeing (Exception from destructor may use inconsistent arg_stack for backtrace). --- Zend/zend_builtin_functions.c | 10 +++++++--- 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 3d24311310..87ead12e3a 100644 --- a/Zend/zend_builtin_functions.c +++ b/Zend/zend_builtin_functions.c @@ -1628,9 +1628,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 b3b084bd66..a76c688513 100644 --- a/Zend/zend_execute.h +++ b/Zend/zend_execute.h @@ -136,7 +136,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; } -- 2.40.0