tree 4a5bbfcbbf6803b3504ec0cbac417f793c7ea5c8
parent 6557acd2db57fdd97e3fcc371ffa40293ddb5843
author Christopher Faulet <cfaulet@haproxy.com> 1722522161 +0200
committer Willy Tarreau <w@1wt.eu> 1725550076 +0200
encoding latin1

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>
(cherry picked from commit fef9d21bae7c655335746216e2a6cdafbc535369)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit c0a0daad409a3b6357cd94941306a6d931fc67f9)
Signed-off-by: Willy Tarreau <w@1wt.eu>