]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: quic: CRYPTO frame freeing without eb_delete()
authorFrederic Lecaille <flecaille@haproxy.com>
Mon, 1 Sep 2025 08:39:00 +0000 (10:39 +0200)
committerFrederic Lecaille <flecaille@haproxy.com>
Mon, 1 Sep 2025 08:39:00 +0000 (10:39 +0200)
commit800ba73a9ca39dc0e0c33230c87ececd540f5541
tree2ccb06017814a3d3447dc3da1cc52aa067e3a116
parent90126ec9b733694d4c5ad0f20c221cff639db95f
BUG/MEDIUM: quic: CRYPTO frame freeing without eb_delete()

Since this commit:

BUG/MINOR: quic: reorder fragmented RX CRYPTO frames by their offsets

when they are parsed, the CRYPTO frames are ordered by their offsets into an ebtree.
Then their data are provided to the ncbufs.

But in case of error, when qc_handle_crypto_frm() returns QUIC_RX_RET_FRM_FATAL
or QUIC_RX_RET_FRM_AGAIN), they remain attached to their tree. Then
from <err> label, they are deteleted and deleted (with a while(node) { eb_delete();
qc_frm_free();} loop). But before this loop, these statements directly
free the frame without deleting it from its tree, if this is a CRYPTO frame,
leading to a use after free when running the loop:

     if (frm)
    qc_frm_free(qc, &frm);

This issue was detected by the interop tests, with quic-go as client. Weirdly, this
client sends CRYPTO frames by packet with holes.
Must be backported as far as 2.6 as the commit mentioned above.
src/quic_rx.c