CLEANUP: mux-h2: clean the stream error path on HEADERS frame processing

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.
This commit is contained in:
Willy Tarreau 2019-01-03 11:48:45 +01:00
parent 3a429f04cb
commit b8c4dd3320

View File

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