From: Olivier Houchard <ohouchard@haproxy.com>
Date: Wed, 28 Nov 2018 12:26:11 +0000 (+0100)
Subject: BUG/MEDIUM: h2: Don't bogusly error if the previous stream was closed.
X-Git-Tag: v1.9-dev9~118
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=0024a98640d07fa184624aea1e66a5390268ab8b;p=thirdparty%2Fhaproxy.git

BUG/MEDIUM: h2: Don't bogusly error if the previous stream was closed.

In h2_process_demux(), if we're demuxing multiple frames, and the previous
frame led to a stream getting closed, don't bogusly consider that an error,
and destroy the next stream, as there are valid cases where the stream could
be closed.
---

diff --git a/src/mux_h2.c b/src/mux_h2.c
index b1bf62bf28..4914927eca 100644
--- a/src/mux_h2.c
+++ b/src/mux_h2.c
@@ -1920,13 +1920,6 @@ static void h2_process_demux(struct h2c *h2c)
 				tasklet_wakeup(h2s->recv_wait->task);
 				h2s->recv_wait = NULL;
 			}
-			if (h2c->st0 >= H2_CS_ERROR)
-				goto strm_err;
-
-			if (h2s->st >= H2_SS_ERROR) {
-				/* stream error : send RST_STREAM */
-				h2c->st0 = H2_CS_FRAME_E;
-			}
 		}
 		h2s = tmp_h2s;