php / php/php-src

PHP8.5: Opcache can crash in `do_implement_interface` when trait causes implicit `Stringable` interface addition

Open
#23,567 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Bug Status: Needs Triage
Dominant language
C
Stars
40.4k
Forks
8.1k
Avg merge
2d 13h
Merged PRs (30d)
96

Description

Description

Just expanding on the description a little, in the following conditions:

  • PHP8.5 is used (8.4 at least is not affected)
  • opcache.enable & opcache.enable_cli are enabled (This may not require enable_cli with other SAPIs, but my issue was on the CLI / this is where I have narrowed down the issue)
  • A trait exists which has a __toString method
  • A class uses the above trait without declaring it implements Stringable (the base class)
  • A sub-class in a different file extends the base class (regardless of if implements Stringable or not)

With the above conditions, then if you have:

  1. A process that runs for at least the time for the second/third steps)
  2. A process runs that loads the base class into opcache's shared memory
  3. A process runs that loads the file the sub-class is in is ran

The process in step 3 will crash with an exit code of -1073741819 (0xC0000005 - STATUS_ACCESS_VIOLATION).

The above process will also cause the crash if the long running process loads the base class into opcache's shared memory - the important points are that when step 3 happens there is a process that was running when the base class was loaded - it isn't relevant if it is the process that did the loading or not.

Full code for replicating:
Main file

<?php
$mode = $argv[1] ?? "";
if($mode === "hold"){
	\sleep(30);
	echo "hold ok\n";
	exit(0);
}
trait TestTrait{
	public function __toString() : string{
		return "testString";
	}
}

class TestBaseClass{
	use TestTrait;
}
switch($mode){
	case "base":
		echo "base ok\n";
		exit(0);
	case "sub":
		require __DIR__ . "/subclass.php";
		echo "sub ok\n";
		exit(0);
}

subclass.php:

<?php
class TestSubclass extends TestBaseClass{}

Commands to run (terminal one):

php -n -d opcache.enable=1 -d opcache.enable_cli=1 test.php hold

Commands to run (terminal two):

php -n -d opcache.enable=1 -d opcache.enable_cli=1 test.php base
echo %errorlevel%
php -n -d opcache.enable=1 -d opcache.enable_cli=1 test.php sub
echo %errorlevel%

Resulted in this output (terminal one):

hold ok

Resulted in this output (terminal two):

base ok
0

-1073741819

But I expected this output instead (terminal one):

hold ok

But I expected this output instead (terminal two):

base ok
0
sub ok
0
PHP Version
PHP 8.4.6 (cli) (built: Apr  9 2025 09:45:15) (ZTS Visual C++ 2022 x64)
Copyright (c) The PHP Group
Zend Engine v4.4.6, Copyright (c) Zend Technologies
Operating System

Windows 11 x64

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the provided main file and subclass.php reproducer, then run the two Windows CLI command sequences with PHP 8.5 and Opcache enabled. Confirm that loading the base class followed by the subclass no longer exits with STATUS_ACCESS_VIOLATION and instead prints “sub ok” with exit code 0; the issue does not name an implementation file or test.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend, performance
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.