BUG/MEDIUM: mux-h2: Propagate term flags to SE on error in h2s_wake_one_stream
authorChristopher Faulet <cfaulet@haproxy.com>
Thu, 1 Aug 2024 14:22:41 +0000 (16:22 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Tue, 3 Sep 2024 16:33:40 +0000 (18:33 +0200)
commitfef9d21bae7c655335746216e2a6cdafbc535369
treeb5d7dbd996afcd50e87997038a5f04bc7bf015aa
parentcac1e39e86fb997b1958265d77d8258a0a03fbd5
BUG/MEDIUM: mux-h2: Propagate term flags to SE on error in h2s_wake_one_stream

When a stream is explicitly woken up by the H2 conneciton, if an error
condition is detected, the corresponding error flag is set on the SE. So
SE_FL_ERROR or SE_FL_ERR_PENDING, depending if the end of stream was
reported or not.

However, there is no attempt to propagate other termination flags. We must
be sure to properly set SE_FL_EOI and SE_FL_EOS when appropriate to be able
to switch a pending error to a fatal error.

Because of this bug, the SE remains with a pending error and no end of
stream, preventing the applicative stream to trully abort it. It means on
some abort scenario, it is possible to block a stream infinitely.

This patch must be backported at least as far as 2.8. No bug was observed on
older versions while the same code is inuse.

(cherry picked from commit 184f16ded7a0274bffe99a4795d0a27f8be7c006)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
src/mux_h2.c