Skip to content

Fix GH-19520: php_admin_value[extension] in FPM unsafely runs RINIT - #20636

Open
bukka wants to merge 1 commit into
php:PHP-8.5from
bukka:fpm_gh19520_ext_dl_init
Open

bukka wants to merge 1 commit into
php:PHP-8.5from
bukka:fpm_gh19520_ext_dl_init

Conversation

@bukka

@bukka bukka commented Dec 3, 2025

Copy link
Copy Markdown
Member

This redifines start_now param to start_mode in php_dl / php_load_extension to allow selection of start hooks to run. And for php-fpm it uses just starting of module.

Comment thread ext/standard/dl.h Outdated
@bukka
bukka changed the base branch from master to PHP-8.4 October 27, 2026 19:09
This redifines start_now param to start_mode in php_dl /
php_load_extension to allow selection of start hooks to run. And
for php-fpm it uses just starting of module.
@bukka
bukka changed the base branch from PHP-8.4 to PHP-8.5 October 27, 2026 19:18
@bukka
bukka force-pushed the fpm_gh19520_ext_dl_init branch from 446ea62 to 7783a8d Compare October 27, 2026 19:24
@bukka
bukka marked this pull request as ready for review October 27, 2026 19:26
@bukka

bukka commented Oct 27, 2026

Copy link
Copy Markdown
Member Author

I have done more testing and also did some changes to make it a bit more backward compatible so it should be now safe for bug fix version. Targeting 8.5 as 8.4 is soon in security support and it should be even more risky if targeting just 8.5.

@bukka bukka removed the ABI break label Oct 27, 2026
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.

2 participants