From 3c0feb51bc16626aabae983444934dc4b6328b2f Mon Sep 17 00:00:00 2001 From: Dmitry Stogov Date: Wed, 19 Mar 2014 23:51:28 +0400 Subject: [PATCH] Fixed assignment of "new" by reference --- Zend/zend_vm_def.h | 4 +++- Zend/zend_vm_execute.h | 16 ++++++++++++---- 2 files changed, 15 insertions(+), 5 deletions(-) diff --git a/Zend/zend_vm_def.h b/Zend/zend_vm_def.h index 8076ba6caa..d98087ac5e 100644 --- a/Zend/zend_vm_def.h +++ b/Zend/zend_vm_def.h @@ -1841,7 +1841,9 @@ ZEND_VM_HANDLER(39, ZEND_ASSIGN_REF, VAR|CV, VAR|CV) zend_assign_to_variable_reference(variable_ptr, value_ptr TSRMLS_CC); if (OP2_TYPE == IS_VAR && opline->extended_value == ZEND_RETURNS_NEW) { -//??? Z_DELREF_P(variable_ptr); + if (!OP2_FREE) { + Z_DELREF_P(variable_ptr); + } } if (RETURN_VALUE_USED(opline)) { diff --git a/Zend/zend_vm_execute.h b/Zend/zend_vm_execute.h index c635d8a06d..f8429bccfe 100644 --- a/Zend/zend_vm_execute.h +++ b/Zend/zend_vm_execute.h @@ -19821,7 +19821,9 @@ static int ZEND_FASTCALL ZEND_ASSIGN_REF_SPEC_VAR_VAR_HANDLER(ZEND_OPCODE_HANDL zend_assign_to_variable_reference(variable_ptr, value_ptr TSRMLS_CC); if (IS_VAR == IS_VAR && opline->extended_value == ZEND_RETURNS_NEW) { -//??? Z_DELREF_P(variable_ptr); + if (!(free_op2.var != NULL)) { + Z_DELREF_P(variable_ptr); + } } if (RETURN_VALUE_USED(opline)) { @@ -23265,7 +23267,9 @@ static int ZEND_FASTCALL ZEND_ASSIGN_REF_SPEC_VAR_CV_HANDLER(ZEND_OPCODE_HANDLE zend_assign_to_variable_reference(variable_ptr, value_ptr TSRMLS_CC); if (IS_CV == IS_VAR && opline->extended_value == ZEND_RETURNS_NEW) { -//??? Z_DELREF_P(variable_ptr); + if (!0) { + Z_DELREF_P(variable_ptr); + } } if (RETURN_VALUE_USED(opline)) { @@ -36856,7 +36860,9 @@ static int ZEND_FASTCALL ZEND_ASSIGN_REF_SPEC_CV_VAR_HANDLER(ZEND_OPCODE_HANDLE zend_assign_to_variable_reference(variable_ptr, value_ptr TSRMLS_CC); if (IS_VAR == IS_VAR && opline->extended_value == ZEND_RETURNS_NEW) { -//??? Z_DELREF_P(variable_ptr); + if (!(free_op2.var != NULL)) { + Z_DELREF_P(variable_ptr); + } } if (RETURN_VALUE_USED(opline)) { @@ -40040,7 +40046,9 @@ static int ZEND_FASTCALL ZEND_ASSIGN_REF_SPEC_CV_CV_HANDLER(ZEND_OPCODE_HANDLER zend_assign_to_variable_reference(variable_ptr, value_ptr TSRMLS_CC); if (IS_CV == IS_VAR && opline->extended_value == ZEND_RETURNS_NEW) { -//??? Z_DELREF_P(variable_ptr); + if (!0) { + Z_DELREF_P(variable_ptr); + } } if (RETURN_VALUE_USED(opline)) { -- 2.40.0