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

Jump to bottom

urlapi: fix parsing URL without slash with CURLU_URLENCODE #9763

Closed
wants to merge 3 commits into from
Closed

urlapi: fix parsing URL without slash with CURLU_URLENCODE #9763

wants to merge 3 commits into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Oct 18, 2022

When CURLU_URLENCODE is set, the parser would mistreat the path component if the URL was specified without a slash like in http://local.test:80?-123

Extended test 1560 to reproduce and verify the fix.

Reported-by: Trail of Bits

@bagder
urlapi: fix parsing URL without slash with CURLU_URLENCODE
7132f79 
When CURLU_URLENCODE is set, the parser would mistreat the path
component if the URL was specified without a slash like in
http://local.test:80?-123

Extended test 1560 to reproduce and verify the fix.

Reported-by: Trail of Bits
@bagder bagder added the URL label Oct 18, 2022
@bagder bagder closed this in 7d6cf06 Oct 20, 2022
@bagder bagder deleted the bagder/urlapi-path-urlencoded branch October 20, 2022 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
hacktoberfest-accepted URL
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@bagder