Edit report at http://bugs.php.net/bug.php?id=52939&edit=1

 ID:                 52939
 Updated by:         dmi...@php.net
 Reported by:        cataphr...@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] cataphr...@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] dmi...@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] dmi...@php.net

The bug is already fixed in SVN (see Zend/tests/bug52939.phpt)

------------------------------------------------------------------------
[2010-09-28 03:57:19] cataphr...@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] cataphr...@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

Reply via email to