BUG/MINOR: quic: Missing padding in very short probe packets
authorFrédéric Lécaille <flecaille@haproxy.com>
Tue, 28 Mar 2023 13:39:11 +0000 (15:39 +0200)
committerWilly Tarreau <w@1wt.eu>
Tue, 28 Mar 2023 16:26:57 +0000 (18:26 +0200)
commit9c317b1d35efe7f957ad101d902168aa77fa9117
tree5d572437e9b8b0fc1b0eb8bcf5850db8e62f2d68
parent21fb6bdab4d1924748d4c0541c3badb353d5a819
BUG/MINOR: quic: Missing padding in very short probe packets

This bug arrived with this commit:
   MINOR: quic: Send PING frames when probing Initial packet number space

This may happen when haproxy needs to probe the peer with very short packets
(only one PING frame). In this case, the packet must be padded. There was clearly
a case which was removed by the mentionned commit above. That said, there was
an extra byte which was added to the PADDING frame before the mentionned commit
above. This is no more the case with this patch.

Thank you to @tatsuhiro-t (ngtcp2 manager) for having reported this issue which
was revealed by the keyupdate test (on client side).

Must be backported to 2.7 and 2.6.
src/quic_conn.c