summaryrefslogtreecommitdiff
path: root/Zend
diff options
context:
space:
mode:
authorDmitry Stogov <dmitry@php.net>2010-10-13 08:51:39 +0000
committerDmitry Stogov <dmitry@php.net>2010-10-13 08:51:39 +0000
commitb88f7753a1e2429701242c8a231ba76f76dd97bd (patch)
tree44e7ce50ac95d762bea1ee3c8ebded16f4a4ab34 /Zend
parent14a54b903e952a9f99bf5a4bf4639ab2e105b1c4 (diff)
downloadphp-git-b88f7753a1e2429701242c8a231ba76f76dd97bd.tar.gz
Fixed bug #52939 (zend_call_function does not respect ZEND_SEND_PREFER_REF)
Diffstat (limited to 'Zend')
-rw-r--r--Zend/tests/bug52939.phpt12
-rw-r--r--Zend/zend_execute_API.c3
2 files changed, 14 insertions, 1 deletions
diff --git a/Zend/tests/bug52939.phpt b/Zend/tests/bug52939.phpt
index 9810b8063f..91bd27bab5 100644
--- a/Zend/tests/bug52939.phpt
+++ b/Zend/tests/bug52939.phpt
@@ -10,6 +10,11 @@ $ar1 = array("row1" => 2, "row2" => 1);
$args = array(&$ar1);
var_dump(call_user_func_array("array_multisort", $args));
var_dump($ar1);
+
+$ar1 = array("row1" => 2, "row2" => 1);
+$args = array($ar1);
+var_dump(call_user_func_array("array_multisort", $args));
+var_dump($ar1);
?>
--EXPECT--
bool(true)
@@ -26,3 +31,10 @@ array(2) {
["row1"]=>
int(2)
}
+bool(true)
+array(2) {
+ ["row1"]=>
+ int(2)
+ ["row2"]=>
+ int(1)
+}
diff --git a/Zend/zend_execute_API.c b/Zend/zend_execute_API.c
index 56aee7bf6e..f6604df235 100644
--- a/Zend/zend_execute_API.c
+++ b/Zend/zend_execute_API.c
@@ -870,7 +870,8 @@ int zend_call_function(zend_fcall_info *fci, zend_fcall_info_cache *fci_cache TS
if (Z_REFCOUNT_PP(fci->params[i]) > 1) {
zval *new_zval;
- if (fci->no_separation) {
+ if (fci->no_separation &&
+ !ARG_MAY_BE_SENT_BY_REF(EX(function_state).function, i + 1)) {
if(i) {
/* hack to clean up the stack */
zend_vm_stack_push_nocheck((void *) (zend_uintptr_t)i TSRMLS_CC);