From: Bob Weinand <bobwei9@hotmail.com>
Date: Mon, 13 Jul 2015 11:45:27 +0000 (+0200)
Subject: Preserve return value in zend_vm_call_opcode_handler with global opline/ex registers... 
X-Git-Tag: php-7.1.1RC1~35^2~10^2~3
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=82b1e22b5763350d22a1fcd638a648015ca85ca4;p=php

Preserve return value in zend_vm_call_opcode_handler with global opline/ex registers enabled
---

diff --git a/Zend/zend_vm_execute.h b/Zend/zend_vm_execute.h
index d2ea5a6afe..8419dccbc6 100644
--- a/Zend/zend_vm_execute.h
+++ b/Zend/zend_vm_execute.h
@@ -49553,7 +49553,10 @@ ZEND_API int zend_vm_call_opcode_handler(zend_execute_data* ex)
 	LOAD_OPLINE();
 #if defined(ZEND_VM_FP_GLOBAL_REG) && defined(ZEND_VM_IP_GLOBAL_REG)
 	((opcode_handler_t)OPLINE->handler)(ZEND_OPCODE_HANDLER_ARGS_PASSTHRU);
-	ret = (opline) ? 0 : -1;
+	ret = (opline) ? execute_data != ex : -1;
+	if (ret == 1 && execute_data->prev_execute_data != ex) {
+		ret = 2;;
+	}
 #else
 	ret = ((opcode_handler_t)OPLINE->handler)(ZEND_OPCODE_HANDLER_ARGS_PASSTHRU);
 #endif
diff --git a/Zend/zend_vm_gen.php b/Zend/zend_vm_gen.php
index 36c4a32f5c..ce4a32185f 100644
--- a/Zend/zend_vm_gen.php
+++ b/Zend/zend_vm_gen.php
@@ -1606,7 +1606,10 @@ function gen_vm($def, $skel) {
 		out($f, "\tLOAD_OPLINE();\n");
 		out($f,"#if defined(ZEND_VM_FP_GLOBAL_REG) && defined(ZEND_VM_IP_GLOBAL_REG)\n");
 		out($f, "\t((opcode_handler_t)OPLINE->handler)(ZEND_OPCODE_HANDLER_ARGS_PASSTHRU);\n");
-		out($f, "\tret = (opline) ? 0 : -1;\n");
+		out($f, "\tret = (opline) ? execute_data != ex : -1;\n");
+		out($f, "\tif (ret == 1 && execute_data->prev_execute_data != ex) {\n");
+		out($f, "\t\tret = 2;;\n");
+		out($f, "\t}\n");
 		out($f, "#else\n");
 		out($f, "\tret = ((opcode_handler_t)OPLINE->handler)(ZEND_OPCODE_HANDLER_ARGS_PASSTHRU);\n");
 		out($f, "#endif\n");