fix gopher don't send NUL byte bug #466

Closed
wants to merge 3 commits into
from

Projects

None yet

2 participants

@orangetw
orangetw commented Oct 1, 2015

I find that gopher scheme (gopher://) do not support NUL byte.
I read the source code and find that gopher.c use STRLEN(SEL) to process the URL-DECODED data.
But "curl_easy_unescape" already has specified OUTLENGTH. so I change the STRLEN(SEL) to OUTLENGTH.

Orange Tsai added some commits Oct 1, 2015
Orange Tsai fix gopher don't send NUL byte bug
968c3a7
Orange Tsai forget to add curly brackets
e6bab0b
Orange Tsai change if-else syntax for checksrc
f6be3fe
@bagder bagder added a commit that closed this pull request Oct 1, 2015
@bagder Orange Tsai + bagder gopher: don't send NUL byte
Closes #466
5bf36ea
@bagder bagder closed this in 5bf36ea Oct 1, 2015
@bagder
Member
bagder commented Oct 1, 2015

Thanks a lot!

@jgsogo jgsogo added a commit to jgsogo/curl that referenced this pull request Oct 19, 2015
@jgsogo Orange Tsai + jgsogo gopher: don't send NUL byte
Closes #466
58f9912
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment