]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
CLEANUP: mux-h2: clean the stream error path on HEADERS frame processing
authorWilly Tarreau <w@1wt.eu>
Thu, 3 Jan 2019 10:48:45 +0000 (11:48 +0100)
committerWilly Tarreau <w@1wt.eu>
Thu, 3 Jan 2019 17:45:38 +0000 (18:45 +0100)
In h2c_frt_handle_headers() and h2c_bck_handle_headers() we have an unused
error path made of the strm_err label, while send_rst is used to emit an
RST upon stream error after forcing the stream to h2_refused_stream. Let's
remove this unused strm_err block now.

src/mux_h2.c

index d74d51481806f760b6c9079556ef3e9289f35014..59d76b7c3a8aac09a1c60f2ddaf1ca6d6368f305 100644 (file)
@@ -1927,13 +1927,6 @@ static struct h2s *h2c_frt_handle_headers(struct h2c *h2c, struct h2s *h2s)
        h2c_error(h2c, error);
        goto out;
 
- strm_err:
-       if (h2s) {
-               h2s_error(h2s, error);
-               h2c->st0 = H2_CS_FRAME_E;
-       }
-       else
-               h2c_error(h2c, error);
  out:
        h2_release_buf(h2c, &rxbuf);
        return NULL;
@@ -2003,15 +1996,6 @@ static struct h2s *h2c_bck_handle_headers(struct h2c *h2c, struct h2s *h2s)
  conn_err:
        h2c_error(h2c, error);
        return NULL;
-
- strm_err:
-       if (h2s) {
-               h2s_error(h2s, error);
-               h2c->st0 = H2_CS_FRAME_E;
-       }
-       else
-               h2c_error(h2c, error);
-       return NULL;
 }
 
 /* processes a DATA frame. Returns > 0 on success or zero on missing data.