]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: quic: Leak in qc_release_lost_pkts() for non in flight TX packets
authorFrédéric Lécaille <flecaille@haproxy.com>
Wed, 24 Aug 2022 15:06:04 +0000 (17:06 +0200)
committerFrédéric Lécaille <flecaille@haproxy.com>
Wed, 24 Aug 2022 16:13:30 +0000 (18:13 +0200)
commit277c4629e7896276f3a54b939471bde59d31fff6
treee50bfdd7de243c8fbc541b8492d36eb542dc33b4
parent5f6c25e44739a273ed2c38db30fdffe32a0a6112
BUG/MINOR: quic: Leak in qc_release_lost_pkts() for non in flight TX packets

Packets with null "in flight" lengths are kept as the others packets as sent
but not already acknowledeged in the by packet number space trees.
But qc_release_lost_pkts() relied on this in fligh length to release the
memory allocated for this packets. We must release the memory allocated for
all the lost packets regardless of their in fligh lengths.

Modify this function to do nothing if the list of lost packets passed
as argument is empty. Stop using <lost_bytes> variable to decide if some packets
memory must be released or not.
Modify the callers to stop checking if this list is empty.

Should helping in fixing memory leak as reported by Tristan in GH #1801.

Must be backported to 2.6.
src/xprt_quic.c