New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes bug 65414 #3052

Closed
wants to merge 1 commit into
base: PHP-7.1
from

Conversation

2 participants
@bishopb
Contributor

bishopb commented Feb 2, 2018

While the common method phar_add_file was protected from being given files with multiple leading /, it was not prepared to deal with files containing a leading slash when verifying file addition to the state
directory .phar. Adds a check for leading slash, and test cases covering no, one, and several leading slashes.

https://bugs.php.net/bug.php?id=65414

Fixes bug 65414.
While the common method phar_add_file was protected from being given
files with multiple leading /, it was not prepared to deal with files
containing a leading slash when verifying file addition to the state
directory .phar. Adds a check for leading slash, and test cases covering
no, one, and several leading slashes.

@krakjoe krakjoe added the Bugfix label Feb 2, 2018

@krakjoe

This comment has been minimized.

Member

krakjoe commented Feb 8, 2018

Merged d806d03

Thanks.

@krakjoe krakjoe closed this Feb 8, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment