[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[gnurl] 183/222: http2_recv: a closed stream trumps pause state
From: |
gnunet |
Subject: |
[gnurl] 183/222: http2_recv: a closed stream trumps pause state |
Date: |
Thu, 07 Nov 2019 00:11:19 +0100 |
This is an automated email from the git hooks/post-receive script.
ng0 pushed a commit to branch master
in repository gnurl.
commit 95a4cfd88820452a5e5da5b64e46b22bceac963f
Author: Daniel Stenberg <address@hidden>
AuthorDate: Fri Oct 18 17:27:21 2019 +0200
http2_recv: a closed stream trumps pause state
... and thus should return 0, not EAGAIN.
Reported-by: Tom van der Woerdt
Fixes #4496
Closes #4505
---
lib/http2.c | 3 +++
1 file changed, 3 insertions(+)
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;
--
To stop receiving notification emails like this one, please contact
address@hidden.
- [gnurl] 159/222: RELEASE-NOTES: synced, (continued)
- [gnurl] 159/222: RELEASE-NOTES: synced, gnunet, 2019/11/06
- [gnurl] 157/222: cirrus: switch off blackhole status on the freebsd CI machines, gnunet, 2019/11/06
- [gnurl] 106/222: vtls: Fix comment typo about macosx-version-min compiler flag, gnunet, 2019/11/06
- [gnurl] 136/222: docs: add note on failed handles not being counted by curl_multi_perform, gnunet, 2019/11/06
- [gnurl] 139/222: http2: move state-init from creation to pre-transfer, gnunet, 2019/11/06
- [gnurl] 144/222: RELEASE-NOTES: synced, gnunet, 2019/11/06
- [gnurl] 165/222: appveyor: add a winbuild that uses VS2017, gnunet, 2019/11/06
- [gnurl] 176/222: CURLOPT_TIMEOUT.3: Clarify transfer timeout time includes queue time, gnunet, 2019/11/06
- [gnurl] 173/222: tests: line ending fixes for Windows, gnunet, 2019/11/06
- [gnurl] 179/222: configure: remove all cyassl references, gnunet, 2019/11/06
- [gnurl] 183/222: http2_recv: a closed stream trumps pause state,
gnunet <=
- [gnurl] 184/222: appveyor: Add MSVC ARM64 build, gnunet, 2019/11/06
- [gnurl] 194/222: appveyor: make winbuilds with DEBUG=no/yes and VS 2015/2017, gnunet, 2019/11/06
- [gnurl] 164/222: socketpair: fix include and define for older TCP header systems, gnunet, 2019/11/06
- [gnurl] 172/222: tests: use proxy feature, gnunet, 2019/11/06
- [gnurl] 204/222: appveyor: publish artifacts on appveyor, gnunet, 2019/11/06
- [gnurl] 156/222: tests: use port 2 instead of 60000 for a safer non-listening port, gnunet, 2019/11/06
- [gnurl] 158/222: connect: return CURLE_OPERATION_TIMEDOUT for errno == ETIMEDOUT, gnunet, 2019/11/06
- [gnurl] 155/222: KNOWN_BUGS: IDN tests failing on Windows, gnunet, 2019/11/06
- [gnurl] 160/222: socketpair: an implemention for Windows and more, gnunet, 2019/11/06
- [gnurl] 162/222: gskit: use the generic Curl_socketpair, gnunet, 2019/11/06