Bug #77275 [Opn]: OPcache optimization problem for ArrayAccess->offsetGet(string)
| From: | nikic@php.net | Date: | Mon, 10 Dec 2018 11:06:38 +0000 |
| Subject: | Bug #77275 [Opn]: OPcache optimization problem for ArrayAccess->offsetGet(string) | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-218356@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=77275&edit=1
ID: 77275
Updated by: nikic@php.net
Reported by: kentaro at ranvis dot com
Summary: OPcache optimization problem for
ArrayAccess->offsetGet(string)
Status: Open
Type: Bug
Package: opcache
Operating System: Windows 10
PHP Version: 7.3.0
Block user comment: N
Private report: N
New Comment:
Can't repro on Ubuntu, though the issue is certainly plausible. At a guess the second literal
for the the offset lookup is not being preserved and being overwritten.
Previous Comments:
------------------------------------------------------------------------
[2018-12-10 05:46:36] kentaro at ranvis dot com
Description:
------------
In a certain condition, a call to the class method offsetGet() via
$instance['literalString'] is turned into $instance[anotherLiteral] when OPcache is
enabled.
Steps to reproduce:
1. On Windows x64, extract php-7.3.0-nts-Win32-VC15-x64.zip into the current directory
2. Save test script as test.php
3. Make sure no PHP built-in server is running (as shared memory matters?)
4. Launch PHP built-in server by running a command:
php -c php.ini-development -d extension_dir=ext -d zend_extension=php_opcache.dll -S
127.0.0.1:8000
5. Open browser and go to http://127.0.0.1:8000/test.php
The output is: string(1) "a"
6. Touch test.php to update the modification time, and wait 3 seconds (opcache.revalidate_freq)
7. Reload the browser and confirm the output
Test script:
---------------
<?php
namespace Foo;
class Bar { public function get() {} }
class Record implements \ArrayAccess {
public function offsetSet($offset, $value) { throw new \Exception; }
public function offsetGet($offset) { var_dump($offset); }
public function offsetExists($offset) { throw new \Exception; }
public function offsetUnset($offset) { throw new \Exception; }
}
class Baz {
public function run() {
$a = pow(1, 2);
$b = new Bar();
$c = new Bar();
$d = new Bar();
$id = $b->get('a', 'b', 'c');
$rec = new Record();
$id = $rec['a'];
}
}
(new Baz())->run();
Expected result:
----------------
The output is: string(1) "a"
Actual result:
--------------
The output is: string(1) "b"
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=77275&edit=1