Bug #52939 [Asn->Fbk]: zend_call_function does not respect ZEND_SEND_PREFER_REF
| From: | dmitry@php.net | Date: | Tue, 12 Oct 2010 18:48:51 +0000 |
| Subject: | Bug #52939 [Asn->Fbk]: zend_call_function does not respect ZEND_SEND_PREFER_REF | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-155116@lists.php.net to get a copy of this message | ||
Edit report at http://bugs.php.net/bug.php?id=52939&edit=1
ID: 52939
Updated by: dmitry@php.net
Reported by: cataphract@php.net
Summary: zend_call_function does not respect
ZEND_SEND_PREFER_REF
-Status: Assigned
+Status: Feedback
Type: Bug
Package: Scripting Engine problem
Operating System: Windows 7 x64
PHP Version: trunk-SVN-2010-09-28 (SVN)
Assigned To: dmitry
Block user comment: N
New Comment:
Note that array($ar1) creates an array with a copy of $ar1 and its
modification doesn't affect the original $ar1 value. Probably the proper
test would be
<?php
$args = array(array("row1" => 2, "row2" => 1));
call_user_func_array("array_multisort", $args);
var_dump($args[0]);
?>
I'll check if it works tomorrow. But anyway it's not the thing you like.
Previous Comments:
------------------------------------------------------------------------
[2010-10-12 16:15:56] cataphract@php.net
The test you added doesn't test the issue.
The problem is not zend_call_function + ZEND_SEND_PREF_REF not accepting
reference; it's not accepting values. This is the correct test:
<?php
var_dump(array_multisort(array("row1" => 2, "row2" => 1)));
$ar1 = array("row1" => 2, "row2" => 1);
var_dump(call_user_func_array("array_multisort", array($ar1)));
var_dump($ar1);
The "normal" call works, because even though it's a value, it's accepted
because of ZEND_SEND_PREFER_REF. The call via call_user_func_array still
fails because of this bug.
------------------------------------------------------------------------
[2010-10-12 09:39:32] dmitry@php.net
This bug has been fixed in SVN.
Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
Thank you for the report, and for helping us make PHP better.
------------------------------------------------------------------------
[2010-10-12 09:39:21] dmitry@php.net
The bug is already fixed in SVN (see Zend/tests/bug52939.phpt)
------------------------------------------------------------------------
[2010-09-28 03:57:19] cataphract@php.net
The following patch has been added/updated:
Patch Name: zend_call_user_function_prefer_ref
Revision: 1285639039
URL:
http://bugs.php.net/patch-display.php?bug=52939&patch=zend_call_user_function_prefer_ref&revision=1285639039
------------------------------------------------------------------------
[2010-09-28 03:50:12] cataphract@php.net
The following patch has been added/updated:
Patch Name: zend_call_user_function_prefer_ref
Revision: 1285638612
URL:
http://bugs.php.net/patch-display.php?bug=52939&patch=zend_call_user_function_prefer_ref&revision=1285638612
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://bugs.php.net/bug.php?id=52939
--
Edit this bug report at http://bugs.php.net/bug.php?id=52939&edit=1