From e6f04d4c8e9adf24ecb9c8d5d2aaf33d311d64be Mon Sep 17 00:00:00 2001 From: Dmitry Stogov Date: Wed, 13 Jul 2016 20:35:53 +0300 Subject: [PATCH] Unified behavior of modification of returned by reference value in finnally code. --- Zend/tests/bug72215_3.phpt | 5 ++--- Zend/zend_vm_def.h | 5 ++++- Zend/zend_vm_execute.h | 10 ++++++++-- 3 files changed, 14 insertions(+), 6 deletions(-) diff --git a/Zend/tests/bug72215_3.phpt b/Zend/tests/bug72215_3.phpt index 38c017b61c..6f10fd7dee 100644 --- a/Zend/tests/bug72215_3.phpt +++ b/Zend/tests/bug72215_3.phpt @@ -4,7 +4,7 @@ Bug #72215.3 (Wrong return value if var modified in finally) --EXPECT-- -NULL - +int(2) diff --git a/Zend/zend_vm_def.h b/Zend/zend_vm_def.h index d8e033fec0..ce3fbeb98c 100644 --- a/Zend/zend_vm_def.h +++ b/Zend/zend_vm_def.h @@ -8103,7 +8103,10 @@ ZEND_VM_HANDLER(51, ZEND_MAKE_REF, VAR|CV, UNUSED) if (OP1_TYPE == IS_CV) { if (UNEXPECTED(Z_TYPE_P(op1) == IS_UNDEF)) { - ZVAL_NULL(EX_VAR(opline->result.var)); + ZVAL_NEW_EMPTY_REF(op1); + Z_SET_REFCOUNT_P(op1, 2); + ZVAL_NULL(Z_REFVAL_P(op1)); + ZVAL_REF(EX_VAR(opline->result.var), Z_REF_P(op1)); } else { ZVAL_MAKE_REF(op1); ZVAL_COPY(EX_VAR(opline->result.var), op1); diff --git a/Zend/zend_vm_execute.h b/Zend/zend_vm_execute.h index 703049942b..ef73b195a0 100644 --- a/Zend/zend_vm_execute.h +++ b/Zend/zend_vm_execute.h @@ -21820,7 +21820,10 @@ static ZEND_OPCODE_HANDLER_RET ZEND_FASTCALL ZEND_MAKE_REF_SPEC_VAR_UNUSED_HANDL if (IS_VAR == IS_CV) { if (UNEXPECTED(Z_TYPE_P(op1) == IS_UNDEF)) { - ZVAL_NULL(EX_VAR(opline->result.var)); + ZVAL_NEW_EMPTY_REF(op1); + Z_SET_REFCOUNT_P(op1, 2); + ZVAL_NULL(Z_REFVAL_P(op1)); + ZVAL_REF(EX_VAR(opline->result.var), Z_REF_P(op1)); } else { ZVAL_MAKE_REF(op1); ZVAL_COPY(EX_VAR(opline->result.var), op1); @@ -42815,7 +42818,10 @@ static ZEND_OPCODE_HANDLER_RET ZEND_FASTCALL ZEND_MAKE_REF_SPEC_CV_UNUSED_HANDLE if (IS_CV == IS_CV) { if (UNEXPECTED(Z_TYPE_P(op1) == IS_UNDEF)) { - ZVAL_NULL(EX_VAR(opline->result.var)); + ZVAL_NEW_EMPTY_REF(op1); + Z_SET_REFCOUNT_P(op1, 2); + ZVAL_NULL(Z_REFVAL_P(op1)); + ZVAL_REF(EX_VAR(opline->result.var), Z_REF_P(op1)); } else { ZVAL_MAKE_REF(op1); ZVAL_COPY(EX_VAR(opline->result.var), op1); -- 2.40.0