php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #76206 Arrays in objects using splat aren't passed by reference in a dynamic call
Submitted: 2018-04-10 18:10 UTC Modified: 2019-08-21 13:22 UTC
From: d28b312d at opayq dot com Assigned:
Status: Verified Package: Scripting Engine problem
PHP Version: Irrelevant OS: *
Private report: No CVE-ID: None
View Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
If you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: d28b312d at opayq dot com
New email:
PHP Version: OS:

 

 [2018-04-10 18:10 UTC] d28b312d at opayq dot com
Description:
------------
If you have an object with an array parameter, and you try to pass that to a closure which accepts parameters by reference, the parameters aren't taken in by reference and instead by value.

Modifying the below script to use `$foo` instead of `$this->foo` in both cases will work, but using the instance assigned to an object parameter seems to pass the values by value without an error, instead of by reference like the function implies.

Note:
If you try to use `call_user_func_array` it throws a message saying it's passing by value to a reference expecting function which is at least better than showing nothing and just treating as if it's a value (but both are wrong!).
`PHP Warning:  Parameter 1 to {closure}() expected to be a reference, value given`

Test script:
---------------
<?php

class Foo
{
    private $foo;

    public function call(Closure $cl, $foo)
    {
        $this->foo = $foo;
        $cl(...$this->foo);
        var_dump($this->foo);
        return $cl(...$this->foo);
    }
}

$f = new Foo;
echo $f->call(function(&$i){return ++$i;}, [0]);

Expected result:
----------------
/tmp/classtest.php:11:
array(1) {
  [0] =>
  int(1)
}
2

Actual result:
--------------
/tmp/classtest.php:11:
array(1) {
  [0] =>
  int(0)
}
1

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2019-08-21 13:22 UTC] nikic@php.net
I tried to fix this in https://github.com/php/php-src/commit/6913ec3282149914e999d91b056fe1cc68d15ed7, but had to revert due to bug #78356.
 [2019-08-21 13:22 UTC] nikic@php.net
-Status: Open +Status: Verified -Package: Unknown/Other Function +Package: Scripting Engine problem
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Sat Dec 21 12:01:31 2024 UTC