]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
MINOR: quic: Trace fix in qc_release_frm()
authorFrédéric Lécaille <flecaille@haproxy.com>
Fri, 19 Aug 2022 10:08:13 +0000 (12:08 +0200)
committerFrédéric Lécaille <flecaille@haproxy.com>
Fri, 19 Aug 2022 10:15:05 +0000 (12:15 +0200)
This wrong trace came with this commit:
  "BUG/MINOR: quic: Possible crashes when dereferencing ->pkt quic_frame struct member"
In qc_release_frm() we mark frames as acked. Nothing to see with references
to frames.

Thank you to Willy for having caught this one.

Must be backported to 2.6 as these traces arrived with a bug fix to be backported
to 2.6.

src/xprt_quic.c

index baee8e242b920671d99943797d3501ecc0daeef0..9c42e524b201770b29ccbb6c6df95797780693de 100644 (file)
@@ -1510,7 +1510,7 @@ void qc_release_frm(struct quic_conn *qc, struct quic_frame *frm)
                }
                else {
                        /* XXX TODO: what must be done for such a frame */
-                       TRACE_DEVEL("remove frame reference for unsent frame",
+                       TRACE_DEVEL("mark frame as acked for unsent frame",
                                    QUIC_EV_CONN_PRSAFRM, qc, f);
                }
                f->flags |= QUIC_FL_TX_FRAME_ACKED;