]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MEDIUM: h2: Don't bogusly error if the previous stream was closed.
authorOlivier Houchard <ohouchard@haproxy.com>
Wed, 28 Nov 2018 12:26:11 +0000 (13:26 +0100)
committerWilly Tarreau <w@1wt.eu>
Wed, 28 Nov 2018 13:09:55 +0000 (14:09 +0100)
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

index b1bf62bf2829a52a425f8e8e36d727fed8d0c29d..4914927eca8463bc51a4221623120bbc52931f9b 100644 (file)
@@ -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;