]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: quic: fix BUG_ON() on Tx pkt alloc failure
authorAmaury Denoyelle <adenoyelle@haproxy.com>
Thu, 20 Jun 2024 15:54:04 +0000 (17:54 +0200)
committerAmaury Denoyelle <adenoyelle@haproxy.com>
Mon, 24 Jun 2024 12:40:38 +0000 (14:40 +0200)
commitd5376b7a874776b4d5d79f9b746d4654df796f85
tree02f86213f70fb8c960998b3930184e7f4cbcb64a
parent5718c67c19766c87bb68b7624e1873a887fbbaf1
BUG/MINOR: quic: fix BUG_ON() on Tx pkt alloc failure

On quic_tx_packet allocation failure, it is possible to trigger BUG_ON()
crash on INITIAL packet building. This statement is responsible to
ensure INITIAL packets are padded to 1.200 bytes as required. If a
packet on higher encryption level allocation fails, PADDING frame cannot
properly encoded, despite the INITIAL packet properly built.

This crash happens due to qc_txb_store() invokation after quic_tx_packet
allocation failure to validate already built packets. However, this
statement is unneeded as qc_purge_tx_buf() is called just after. Simply
remove qc_txb_store() to fix this issue.

This was detected using -dMfail.

This should be backported up to 2.6.
src/quic_tx.c