]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: quic: do not drop packet on duplicate stream/decoding error
authorAmaury Denoyelle <adenoyelle@haproxy.com>
Tue, 8 Mar 2022 09:48:35 +0000 (10:48 +0100)
committerAmaury Denoyelle <adenoyelle@haproxy.com>
Tue, 8 Mar 2022 13:36:32 +0000 (14:36 +0100)
commit20f89cac95068e988a36d369e499ffdf6ded6a99
tree94557c46595a8c85f7073be38530cb45219f9003
parentb0dfd099c56b8f8fd46686120a6cf1598bd5e470
BUG/MEDIUM: quic: do not drop packet on duplicate stream/decoding error

Change the return value to success in qc_handle_bidi_strm_frm for two
specific cases :
* if STREAM frame is an already received offset
* if application decoding failed

This ensures that the packet is not dropped and properly acknowledged.
Previous to this fix, the return code was set to error which prevented
the ACK to be generated.

The impact of the bug might be noticeable in environment with packet
loss and retransmission. Due to haproxy not generating ACK for packets
containing STREAM frames with already received offset, the client will
probably retransmit them again, which will worsen the network
transmission.
src/xprt_quic.c