]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
CLEANUP: mux-h2: remove two useless but misleading assignments
authorWilly Tarreau <w@1wt.eu>
Tue, 29 Jan 2019 17:51:41 +0000 (18:51 +0100)
committerWilly Tarreau <w@1wt.eu>
Tue, 29 Jan 2019 17:51:41 +0000 (18:51 +0100)
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

index f74ae3288864b63a310fd3e9df291a9dba65cd9b..afb3ad53bdf9a62833333160105310d3bd0b6d64 100644 (file)
@@ -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);