]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MINOR: mux-h2: fix expression when detecting excess of CONTINUATION frames
authorWilly Tarreau <w@1wt.eu>
Fri, 6 Dec 2024 17:53:19 +0000 (18:53 +0100)
committerWilly Tarreau <w@1wt.eu>
Fri, 6 Dec 2024 17:53:19 +0000 (18:53 +0100)
Latest commit f0eca8fe7 ("MINOR: mux-h2/glitches: add a description to
the H2 glitches") misplaced the optional glitch description field, with
it appearing at the end of the if () condition and always reporting
an excess of CONTINUATION frames from the first exceeding one.

This needs to be backported along with that commit once it gets backported.

src/mux_h2.c

index c0022226be2ccb53458935b12af645637af48c4f..a91f5613370837f1f046f1f7dd32525728733e1c 100644 (file)
@@ -5954,7 +5954,7 @@ next_frame:
         * its counter by 100.
         */
        if (unlikely(fragments > 4) && fragments > flen / 1024 && ret != 0) {
-               if (h2c_report_glitch(h2c, (fragments + 15) / 16), "too many CONTINUATION frames") {
+               if (h2c_report_glitch(h2c, (fragments + 15) / 16, "too many CONTINUATION frames")) {
                        TRACE_STATE("glitch limit reached on CONTINUATION frame", H2_EV_RX_FRAME|H2_EV_RX_HDR|H2_EV_H2C_ERR|H2_EV_PROTO_ERR, h2c->conn);
                        ret = -1;
                }