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

curl_easy_escape: limit *output* string length to 3 * max input #6192

Closed
wants to merge 1 commit into from
Closed

curl_easy_escape: limit *output* string length to 3 * max input #6192

wants to merge 1 commit into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Nov 9, 2020 •
edited

... instead of the limiting it to just the max input value. As every input byte can be expanded to 3 output bytes, this could limit the input string to 2.66 MB instead of 8.

Reported-by: Marc Schlatter

@bagder
curl_easy_escape: limit *output* string length to 3 * max input
3ddae8a 
... not the output string. As every input byte can be expanded to 3
output bytes, this could limit the input string to 2.66 MB instead of 8.

Reported-by: Marc Schlatter
@bagder bagder mentioned this pull request Nov 9, 2020
Closed
@bagder bagder closed this in 7ae5983 Nov 9, 2020
@bagder bagder deleted the bagder/escape-full-inputlen branch November 9, 2020 22:01
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
libcurl API
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@bagder