ID: 33643
Updated by: [EMAIL PROTECTED]
Reported By: Jason at hybd dot net
-Status: Assigned
+Status: Wont fix
Bug Type: Scripting Engine problem
Operating System: *
PHP Version: 4CVS-2005-07-18
Assigned To: derick
New Comment:
This is not fixable in PHP 4.4 without breaking BC, so we can't fix it
now. Luckily it does not create any memory corruption.
Previous Comments:
------------------------------------------------------------------------
[2005-07-14 16:37:02] [EMAIL PROTECTED]
This sort of thing needs to throw an error in any version of PHP I
think:
function foo() { return "blah"; }
function bar(&$arg) { $arg = 1; }
bar(foo());
While this should work:
function & foo() { static $a; return $a; }
function bar(&$arg) { $arg = 1; }
bar(foo());
And it does in PHP5.1 at least. The first case indicates a clear bug
in the code that we really should be catching. I suppose we could
compromise and throw a warning in PHP4 and then fall back to passing
the argument by value, but that is changing the actual workings of the
script and is likely going to be wrong. I think this needs to be part
of the reference fixes in PHP4.4. Sucks that it wasn't caught before
the 4.4.0 release, but that's the way it goes.
------------------------------------------------------------------------
[2005-07-14 13:06:00] [EMAIL PROTECTED]
The "Fatal error:" in HEAD and PHP_5_0 occurs because of fix for bug
#33257. The patch was not applied into PHP 4.4 because it wasn't
affected by this bug (PHP 4 doesn't allow static properties).
Now I found reproducable case for PHP_4_4. :(
<?php
$x = array('foo');
function foo() {
return $GLOBALS['x'];
}
function bar() {
static $x = array('bar');
return $x;
}
echo array_pop(foo())."\n";
echo array_pop(foo())."\n"; /* BUG */
echo array_pop(bar())."\n";
echo array_pop(bar())."\n";
?>
Expected result:
----------------
Fatal error: Only variables can be passed by reference
Or:
---
foo
foo
bar
bar
Actual result:
--------------
foo
bar
bar
The current behavoir of HEAD is proper from my point of view. PHP_4_4
may be fixed but it will break BC again.
Derick, Andi?
------------------------------------------------------------------------
[2005-07-14 11:16:56] [EMAIL PROTECTED]
I think we need to discuss this a bit more, PHP 4.4 indeed gives no
warning or notice on this. Andi, Dmitry?
------------------------------------------------------------------------
[2005-07-14 11:01:13] [EMAIL PROTECTED]
Actual result (4.x after removing "public static"):
4.3.10: No error
4.4.0: No error
5.0.4: No error
5.1.0: Fatal error
Is this really the expected result? I thought 4.4.0 should produce
something too. In which cases 4.4.0 produces errors?
------------------------------------------------------------------------
[2005-07-11 13:06:46] Jason at hybd dot net
Description:
------------
This bug is a rehash of 33516, but I can't reopen as I'm not the
original author.
It seems like you can not pass tempories to function / method
parameters.
<?php
class Foo {
public static function bar() {
return array('a' => '1234567890');
}
}
$tmp = Foo::bar();
$var = array_pop($tmp);
?>
Seems to work, but it shouldn't be down to the PHP user (imho) to
implement this as this change in PHP 5.1 breaks a lot of existing code.
Reproduce code:
---------------
<?php
class Foo {
public static function bar() {
return array('a' => '1234567890');
}
}
$var = array_pop(Foo::bar());
?>
Expected result:
----------------
$var = '1234567890';
Foo::bar() gets placed in temporary variable
Actual result:
--------------
Fatal error: Only variables can be passed by reference
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=33643&edit=1
