diff options
author | Johannes Schlüter <johannes@php.net> | 2011-03-17 07:46:57 +0000 |
---|---|---|
committer | Johannes Schlüter <johannes@php.net> | 2011-03-17 07:46:57 +0000 |
commit | 6f15da050e9680025f6e3f4374efaa3e5dccb9c5 (patch) | |
tree | 90e653749ab77cc31f4529311dfa8aa0fd4599d7 | |
parent | 6e35b40b4c6dd3e3167b79a4d9a5ad7b3070e31e (diff) | |
download | php-git-6f15da050e9680025f6e3f4374efaa3e5dccb9c5.tar.gz |
- Revert r309308 temporarily to get 5.3.6 out
-rw-r--r-- | NEWS | 2 | ||||
-rw-r--r-- | Zend/tests/bug54265.phpt | 17 | ||||
-rw-r--r-- | Zend/zend_execute.c | 16 |
3 files changed, 2 insertions, 33 deletions
@@ -4,8 +4,6 @@ - Zend Engine: . Fixed bug numerous crashes due to setlocale (crash on error, pcre, mysql etc.) on Windows in thread safe mode. (Pierre) - . Fixed bug #54265 (crash when variable gets reassigned in error handler). - (Dmitry) . Fixed bug #54262 (Crash when assigning value to a dimension in a non-array). (Dmitry) diff --git a/Zend/tests/bug54265.phpt b/Zend/tests/bug54265.phpt deleted file mode 100644 index 43db028a2a..0000000000 --- a/Zend/tests/bug54265.phpt +++ /dev/null @@ -1,17 +0,0 @@ ---TEST-- -Bug #54265 (crash when variable gets reassigned in error handler) ---FILE-- -<?php -function my_errorhandler($errno,$errormsg) { - global $my_var; - $my_var = 0; - echo "EROOR: $errormsg\n"; -} -set_error_handler("my_errorhandler"); -$my_var = str_repeat("A",$my_var[0]->errormsg = "xyz"); -echo "ok\n"; -?> ---EXPECT-- -EROOR: Creating default object from empty value -ok - diff --git a/Zend/zend_execute.c b/Zend/zend_execute.c index f10fce38dc..e270816d8b 100644 --- a/Zend/zend_execute.c +++ b/Zend/zend_execute.c @@ -536,22 +536,10 @@ static inline void zend_assign_to_object(znode *result, zval **object_ptr, zval (Z_TYPE_P(object) == IS_BOOL && Z_LVAL_P(object) == 0) || (Z_TYPE_P(object) == IS_STRING && Z_STRLEN_P(object) == 0)) { SEPARATE_ZVAL_IF_NOT_REF(object_ptr); + zval_dtor(*object_ptr); + object_init(*object_ptr); object = *object_ptr; - Z_ADDREF_P(object); zend_error(E_STRICT, "Creating default object from empty value"); - if (Z_REFCOUNT_P(object) == 1) { - /* object was removed by error handler, nothing to assign to */ - zval_ptr_dtor(&object); - if (retval) { - *retval = &EG(uninitialized_zval); - PZVAL_LOCK(*retval); - } - FREE_OP(free_value); - return; - } - Z_DELREF_P(object); - zval_dtor(object); - object_init(object); } else { zend_error(E_WARNING, "Attempt to assign property of non-object"); if (!RETURN_VALUE_UNUSED(result)) { |