Revert "BUG/MAJOR: mux-quic: fix BUG_ON on empty STREAM emission"
authorAmaury Denoyelle <adenoyelle@haproxy.com>
Thu, 2 Jan 2025 09:56:13 +0000 (10:56 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Mon, 6 Jan 2025 10:32:14 +0000 (11:32 +0100)
commit18c745d9a70f6f36d271abe8070874cef6cdcbd1
tree2d5ee94ec385d6051dad7d262394cbd78bf07a79
parent5909764827626c5eea30198eb9e55740a4a0e995
Revert "BUG/MAJOR: mux-quic: fix BUG_ON on empty STREAM emission"

This reverts commit 98064537423fafe05b9ddd97e81cedec8b6b278d.

Above patch tried to fix a BUG_ON() occurence when MUX only emitted
empty STREAM frames via qcc_build_frms(). Return value of qcs_send() was
changed from the payload STREAM frame to the whole frame length.
However, this is invalid as this return value is used to ensure
connection flow-control is not exceeded on sending retry. This causes
occurence of BUG_ON() crash in qcc_io_send() as send-list is not
properly purged after QCS emission.

Reverts this incorrect fix. The original issue will be properly dealt in
the next commit.

This commit must be backported to 3.1 if reverted commit was already
applied on it.

(cherry picked from commit 85e27f1e92d0d6fd4dd0588992d3da3039a065f9)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
src/mux_quic.c