BUG/MINOR: mux-h1: Only report a SE error on demux error

When a demux error is reported by the H1S, an error must be reported on the
SE and not an end-of-input or an end-of-stream. So SE_FL_ERROR flag must be
set and not SE_FL_EOI/SE_FL_EOS.

It seems this bug has no impact. So there is no reason to backport it.
This commit is contained in:
Christopher Faulet 2024-12-23 11:42:08 +01:00
parent e56e718c82
commit 4ccca7efcf

View File

@ -2324,11 +2324,7 @@ static size_t h1_process_demux(struct h1c *h1c, struct buffer *buf, size_t count
err:
htx_to_buf(htx, buf);
se_fl_set(h1s->sd, SE_FL_EOI);
if (h1c->state < H1_CS_RUNNING) {
h1c->flags |= H1C_F_EOS;
se_fl_set(h1s->sd, SE_FL_EOS);
}
se_fl_set(h1s->sd, SE_FL_ERROR);
TRACE_DEVEL("leaving on error", H1_EV_RX_DATA|H1_EV_STRM_ERR, h1c->conn, h1s);
return 0;
}