Skip to content

ext/curl: apply open_basedir restriction to CURLOPT_SSLENGINE - #23213

Open
Sjord wants to merge 1 commit into
php:masterfrom
Sjord:curlopt-sslengine-open-basedir
Open

ext/curl: apply open_basedir restriction to CURLOPT_SSLENGINE#23213
Sjord wants to merge 1 commit into
php:masterfrom
Sjord:curlopt-sslengine-open-basedir

Conversation

@Sjord

@Sjord Sjord commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

CURLOPT_SSLENGINE normally takes an engine name, but in some cases the passed string is loaded as a shared library. Validate it against open_basedir to restrict the locations the shared library can be loaded from.

Related to #22035, but does not solve it.

CURLOPT_SSLENGINE normally takes an engine name, but in some cases the
passed string is loaded as a shared library. Validate it against
open_basedir to restrict the locations the shared library can be loaded
from.

Related to php#22035, but does not
solve it.
@LamentXU123

Copy link
Copy Markdown
Member

I think this is reasonable, and I'd like to link my former issue here as a reference. #21971
From the issue we can know that there are a bunch of cases that php's core extension doesn't care about open_basedir.
Also linking #23214 as a similar issue here in the intl extension.

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