diff options
author | Jay Satiro <raysatiro@yahoo.com> | 2021-01-10 19:01:08 -0500 |
---|---|---|
committer | Jay Satiro <raysatiro@yahoo.com> | 2021-01-11 03:28:01 -0500 |
commit | 70af17911868618e43ec86a1d7782829bbe27bec (patch) | |
tree | 643af2ab863daedd614eaf86888aa14a8eff2250 | |
parent | 3375584a023c9a0ee62885849e5fdd039162cf33 (diff) | |
download | curl-70af17911868618e43ec86a1d7782829bbe27bec.tar.gz |
http_proxy: Fix CONNECT chunked encoding race condition
- During the end-of-headers response phase do not mark the tunnel
complete unless the response body was completely parsed/ignored.
Prior to this change if the entirety of a CONNECT response with chunked
encoding was not received by the time the final header was parsed then
the connection would be marked done prematurely, before all the chunked
data could be read in and ignored (since this is what we do with any
CONNECT response body) and the connection could not be used.
Bug: https://curl.se/mail/lib-2021-01/0033.html
Reported-by: Fabian Keil
Closes https://github.com/curl/curl/pull/6432
-rw-r--r-- | lib/http_proxy.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/lib/http_proxy.c b/lib/http_proxy.c index 43bbe73d5..2e0c8d35f 100644 --- a/lib/http_proxy.c +++ b/lib/http_proxy.c @@ -489,9 +489,12 @@ static CURLcode CONNECT(struct connectdata *conn, } else s->keepon = KEEPON_DONE; - if(!s->cl) + + if(s->keepon == KEEPON_DONE && !s->cl) /* we did the full CONNECT treatment, go to COMPLETE */ s->tunnel_state = TUNNEL_COMPLETE; + + DEBUGASSERT(s->keepon == KEEPON_IGNORE || s->keepon == KEEPON_DONE); continue; } |