php / php/php-src

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

Offen
#23,567 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bug Status: Needs Triage
Vorherrschende Sprache
C
Sterne
40.4k
Forks
8.1k
Ø Merge
2 T. 13 Std.
Gemergte PRs (30 T.)
96

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginnen Sie mit der bereitgestellten main file und dem Reproducer subclass.php und führen Sie dann die beiden Windows-CLI-Befehlssequenzen mit PHP 8.5 und aktiviertem Opcache aus. Bestätigen Sie, dass das Laden der base class gefolgt von der subclass nicht mehr mit STATUS_ACCESS_VIOLATION beendet wird, sondern stattdessen „sub ok“ mit dem Exit-Code 0 ausgibt; das Issue nennt keine Implementierungsdatei oder keinen Test.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
php
Bereich
backend, performance
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Aktiv
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.