Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Mar 20, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

Preloading shutdown calls request shutdown which will deactivate the
virtual cwd state. However, further startup code still assumes the state
that was set by virtual_cwd_startup(). So we need to reactivate it
manually.

Creating a test was a bit difficult because the INI setting I wanted to
test this with is overridden by the test runner apparently.
To reproduce the issue, create an empty file test.php and execute this
in a ZTS build:
`php -d opcache.preload=./ext/opcache/tests/preload_class_alias_2.inc -d "error_log=" -d "allow_url_include=1" test.php`

Closes GH-18117.
* PHP-8.3:
  Fix GH-18112: NULL access with preloading and INI option
* PHP-8.4:
  Fix GH-18112: NULL access with preloading and INI option
* PHP-8.3:
  [ci skip] Fix NEWS order
* PHP-8.4:
  [ci skip] Fix NEWS order
This should call zend_object_std_dtor() to clean the property table etc.
This also has a semantic influence because previously weak refs were not
notified for example.

This fixes the final issue in GH-18114 (the crash was master-only and
fixed already).

Closes GH-18114.
Closes GH-18123.
* PHP-8.3:
  Fix memory leak when destroying PDORow
* PHP-8.4:
  Fix memory leak when destroying PDORow
@pull pull bot added the ⤵️ pull label Mar 20, 2025
@pull pull bot merged commit c4a7961 into wudi:master Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant