Skip to content

Conversation

@eriklax
Copy link
Contributor

@eriklax eriklax commented May 1, 2018

Not all BIO_new_file calls checks the basedir, php_openssl_x509_from_zval above triggers a warning for a call deeper down.

<?php
ini_set('open_basedir', '/tmp/');
openssl_pkey_get_public("file:///any/path/cert.pem");

eriklax added 2 commits May 1, 2018 18:16
Not all BIO_new_file calls checks the basedir, php_openssl_x509_from_zval above triggers a warning for a call deeper down.

```
<?php
ini_set('open_basedir', '/tmp/');
openssl_pkey_get_public("file:///etc/passwd");
```
@eriklax eriklax changed the base branch from master to PHP-7.1 May 2, 2018 20:36
@eriklax eriklax changed the base branch from PHP-7.1 to master May 2, 2018 20:36
@eriklax
Copy link
Contributor Author

eriklax commented May 2, 2018

@krakjoe krakjoe added the Bug label May 10, 2018
@php-pulls
Copy link

Comment on behalf of bukka at php.net:

Merged in d6bc266 and test added in 798e707 . Thanks!

@php-pulls php-pulls closed this May 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants