Skip to content
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

[ticket/13531] Explicitly disallow trailing paths (e.g. PATH_INFO). #3316

Merged
merged 2 commits into from
Jan 21, 2015

Conversation

bantu
Copy link
Collaborator

@bantu bantu commented Jan 20, 2015

@bantu bantu added this to the 3.0.13 milestone Jan 20, 2015
// Check if trailing path is used
if (phpbb_has_trailing_path($phpEx))
{
exit('Trailing paths and path_info is not supported by phpBB 3.0');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it not be possible to perform a redirect? If not what about a proper status code?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately send_status_line() is not available here.

naderman added a commit that referenced this pull request Jan 21, 2015
[ticket/13531] Explicitly disallow trailing paths (e.g. PATH_INFO).
@naderman naderman merged commit eaeb881 into phpbb:develop-olympus Jan 21, 2015
@fgeek
Copy link

fgeek commented Jan 31, 2015

Please use CVE-2015-1431 for this issue, thanks.

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.

5 participants