summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2019-10-18 17:27:21 +0200
committerDaniel Stenberg <daniel@haxx.se>2019-10-18 17:27:21 +0200
commit4151573b8be3f0bcd3cff6f21d21d5a93c8a4bcb (patch)
tree29ff7b53e63d894fd9314f586a73942aea08dd17
parent6913136462cab523a9e8dc621c4ce5d4cf89f0c8 (diff)
downloadcurl-bagder/issue-4496.tar.gz
http2_recv: a closed stream trumps pause statebagder/issue-4496
... and thus should return 0, not EAGAIN. Reported-by: Tom van der Woerdt Fixes #4496
-rw-r--r--lib/http2.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/lib/http2.c b/lib/http2.c
index c0f29f83e..bae938811 100644
--- a/lib/http2.c
+++ b/lib/http2.c
@@ -1663,6 +1663,9 @@ static ssize_t http2_recv(struct connectdata *conn, int sockindex,
socket is not read. But it seems that usually streams are
notified with its drain property, and socket is read again
quickly. */
+ if(stream->closed)
+ /* closed overrides paused */
+ return 0;
H2BUGF(infof(data, "stream %x is paused, pause id: %x\n",
stream->stream_id, httpc->pause_stream_id));
*err = CURLE_AGAIN;