]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MAJOR: quic: Possible crash with full congestion control window
authorFrédéric Lécaille <flecaille@haproxy.com>
Sun, 13 Mar 2022 18:19:12 +0000 (19:19 +0100)
committerWilly Tarreau <w@1wt.eu>
Tue, 15 Mar 2022 09:38:48 +0000 (10:38 +0100)
commite9a974a37a58eb3a23316bd03e0b95e00d951cb8
tree378f216c7d52d7878e3cb80109e89a77cec4479d
parent2ee5c8b3dd22e58af338a8e993c10f7b35dff1ff
BUG/MAJOR: quic: Possible crash with full congestion control window

This commit reverts this one:
  "d5066dd9d BUG/MEDIUM: quic: qc_prep_app_pkts() retries on qc_build_pkt() failures"

After having filled the congestion control window, qc_build_pkt() always fails.
Then depending on the relative position of the writer and  reader indexes for the
TX buffer, this could lead this function to try to reuse the buffer even if not full.
In such case, we do not always mark the end of the data in this TX buffer. This
is something the reader cannot understand: it reads a false datagram length,
then a wrong packet address from the TX buffer, leading to an invalid pointer
dereferencing.
src/xprt_quic.c