From dcd86d154e5b5eb5e8dfe440dbb85bb90dfb8e7e Mon Sep 17 00:00:00 2001 From: Yasuo Ohgaki Date: Thu, 14 Mar 2002 09:36:13 +0000 Subject: [PATCH] Fixed var_dump() crash when there is recursion. # Since I said it's easy to fix :) --- ext/standard/var.c | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/ext/standard/var.c b/ext/standard/var.c index 65fa4e9431..3604d74b0c 100644 --- a/ext/standard/var.c +++ b/ext/standard/var.c @@ -60,6 +60,7 @@ static int php_array_element_dump(zval **zv, int num_args, va_list args, zend_ha void php_var_dump(zval **struc, int level TSRMLS_DC) { HashTable *myht; + zend_object *object = NULL; if (level > 1) { php_printf("%*c", level - 1, ' '); @@ -85,13 +86,31 @@ void php_var_dump(zval **struc, int level TSRMLS_DC) break; case IS_ARRAY: myht = Z_ARRVAL_PP(struc); + if (++((*struc)->value.ht->nApplyCount) > 1) { + PUTS("*RECURSION*\n"); + (*struc)->value.ht->nApplyCount = 0; + break; + } php_printf("%sarray(%d) {\n", COMMON, zend_hash_num_elements(myht)); goto head_done; case IS_OBJECT: + object = Z_OBJ_PP(struc); + if (++object->properties->nApplyCount > 1) { + PUTS("*RECURSION*\n"); + object->properties->nApplyCount = 0; + return; + } myht = Z_OBJPROP_PP(struc); php_printf("%sobject(%s)(%d) {\n", COMMON, Z_OBJCE_PP(struc)->name, zend_hash_num_elements(myht)); head_done: + myht = Z_ARRVAL_PP(struc); zend_hash_apply_with_arguments(myht, (apply_func_args_t) php_array_element_dump, 1, level); + if (Z_TYPE_PP(struc) == IS_ARRAY) { + (*struc)->value.ht->nApplyCount--; + } + else { + object->properties->nApplyCount--; + } if (level > 1) { php_printf("%*c", level-1, ' '); } -- 2.50.1