From f1e6fa35deb08de6025e0126495d3424ca564282 Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Tue, 29 Jan 2019 18:51:41 +0100 Subject: [PATCH] CLEANUP: mux-h2: remove two useless but misleading assignments h2c->st0 was assigned to H2_CS_ERROR right after returning from h2c_error(), which had already done it. It's useless and confusing, let's remove this. --- src/mux_h2.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/mux_h2.c b/src/mux_h2.c index f74ae3288..afb3ad53b 100644 --- a/src/mux_h2.c +++ b/src/mux_h2.c @@ -2237,7 +2237,6 @@ static void h2_process_demux(struct h2c *h2c) if ((int)hdr.len < 0 || (int)hdr.len > global.tune.bufsize) { h2c_error(h2c, H2_ERR_FRAME_SIZE_ERROR); - h2c->st0 = H2_CS_ERROR; if (!h2c->nb_streams) { /* only log if no other stream can report the error */ sess_log(h2c->conn->owner); @@ -2310,7 +2309,6 @@ static void h2_process_demux(struct h2c *h2c) * this state MUST be treated as a connection error */ h2c_error(h2c, H2_ERR_PROTOCOL_ERROR); - h2c->st0 = H2_CS_ERROR; if (!h2c->nb_streams) { /* only log if no other stream can report the error */ sess_log(h2c->conn->owner);