]> granicus.if.org Git - php/commitdiff
Fixed bug #78154
authorNikita Popov <nikita.ppv@gmail.com>
Thu, 13 Jun 2019 08:59:01 +0000 (10:59 +0200)
committerNikita Popov <nikita.ppv@gmail.com>
Thu, 13 Jun 2019 08:59:01 +0000 (10:59 +0200)
NEWS
Zend/tests/bug78154.phpt [new file with mode: 0644]
Zend/zend_vm_def.h
Zend/zend_vm_execute.h

diff --git a/NEWS b/NEWS
index 7b1e20706a32ed07e7917ce9e16858dfa883f37e..c438915dd3091f9697d59e4d5e34ac4c196fc4aa 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -5,6 +5,7 @@ PHP                                                                        NEWS
 - Core:
   . Fixed bug #78151 (Segfault caused by indirect expressions in PHP 7.4a1).
     (Nikita)
+  . Fixed bug #78154 (SEND_VAR_NO_REF does not always send reference). (Nikita)
 
 13 Jun 2019, PHP 7.4.0alpha1
 
diff --git a/Zend/tests/bug78154.phpt b/Zend/tests/bug78154.phpt
new file mode 100644 (file)
index 0000000..2a69ebe
--- /dev/null
@@ -0,0 +1,23 @@
+--TEST--
+Bug #78154: SEND_VAR_NO_REF does not always send reference
+--FILE--
+<?php
+
+namespace {
+    var_dump(similar_text('a', 'a', $c=0x44444444));
+    var_dump($c);
+}
+namespace Foo {
+    var_dump(similar_text('a', 'a', $d=0x44444444));
+    var_dump($d);
+}
+
+?>
+--EXPECTF--
+Notice: Only variables should be passed by reference in %s on line %d
+int(1)
+int(1145324612)
+
+Notice: Only variables should be passed by reference in %s on line %d
+int(1)
+int(1145324612)
index 6c8bce1dd12cbeb792b874e62ed1c1ddac79c6c5..2cb9d038a44a0f8619e5c83985c549aa4a87b400 100644 (file)
@@ -4536,6 +4536,7 @@ ZEND_VM_HANDLER(106, ZEND_SEND_VAR_NO_REF, VAR, NUM)
        }
 
        SAVE_OPLINE();
+       ZVAL_NEW_REF(arg, arg);
        zend_error(E_NOTICE, "Only variables should be passed by reference");
        ZEND_VM_NEXT_OPCODE_CHECK_EXCEPTION();
 }
@@ -4576,6 +4577,7 @@ ZEND_VM_HOT_SEND_HANDLER(50, ZEND_SEND_VAR_NO_REF_EX, VAR, NUM, SPEC(QUICK_ARG))
        }
 
        SAVE_OPLINE();
+       ZVAL_NEW_REF(arg, arg);
        zend_error(E_NOTICE, "Only variables should be passed by reference");
        ZEND_VM_NEXT_OPCODE_CHECK_EXCEPTION();
 }
index 701087a83238005328a786c166cc403428caf73a..e6d8e884738550a17abd40ef54d580cbdea53173 100644 (file)
@@ -20304,6 +20304,7 @@ static ZEND_OPCODE_HANDLER_RET ZEND_FASTCALL ZEND_SEND_VAR_NO_REF_SPEC_VAR_HANDL
        }
 
        SAVE_OPLINE();
+       ZVAL_NEW_REF(arg, arg);
        zend_error(E_NOTICE, "Only variables should be passed by reference");
        ZEND_VM_NEXT_OPCODE_CHECK_EXCEPTION();
 }
@@ -20344,6 +20345,7 @@ static ZEND_OPCODE_HANDLER_RET ZEND_FASTCALL ZEND_SEND_VAR_NO_REF_EX_SPEC_VAR_HA
        }
 
        SAVE_OPLINE();
+       ZVAL_NEW_REF(arg, arg);
        zend_error(E_NOTICE, "Only variables should be passed by reference");
        ZEND_VM_NEXT_OPCODE_CHECK_EXCEPTION();
 }
@@ -20384,6 +20386,7 @@ static ZEND_VM_HOT ZEND_OPCODE_HANDLER_RET ZEND_FASTCALL ZEND_SEND_VAR_NO_REF_EX
        }
 
        SAVE_OPLINE();
+       ZVAL_NEW_REF(arg, arg);
        zend_error(E_NOTICE, "Only variables should be passed by reference");
        ZEND_VM_NEXT_OPCODE_CHECK_EXCEPTION();
 }