From 0024a98640d07fa184624aea1e66a5390268ab8b Mon Sep 17 00:00:00 2001 From: Olivier Houchard Date: Wed, 28 Nov 2018 13:26:11 +0100 Subject: [PATCH] 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. --- src/mux_h2.c | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/mux_h2.c b/src/mux_h2.c index b1bf62bf2..4914927ec 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;