]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: quic: fix FD for quic_cc_conn
authorAmaury Denoyelle <adenoyelle@haproxy.com>
Mon, 13 Nov 2023 10:30:36 +0000 (11:30 +0100)
committerAmaury Denoyelle <adenoyelle@haproxy.com>
Mon, 13 Nov 2023 10:55:07 +0000 (11:55 +0100)
commit954b5b756a7dd4cdabbc46402055f4673650a250
tree541784b3dbe41856eec1d7c6582f7da21a352583
parent78d244e9f7788bdd719ececca1f7e9308416a3b4
BUG/MEDIUM: quic: fix FD for quic_cc_conn

Since following commit, quic_conn closes its owned socket before
transition to quic_cc_conn for closing state. This allows to save FDs as
quic_cc_conn could use the listener socket for their I/O.

  commit 150c0da8895be50a39fd8e44f1db28e52c938569
  MEDIUM: quic: release conn socket before using quic_cc_conn

This patch is incomplete as it removes initialization of <fd> member for
quic_cc_conn. Thus, if sending is done on closing state, <fd> value is
undefined which in most cases will result in a crash. Fix this by simply
initializing <fd> member with qc_init_fd() in qc_new_cc_conn().

This bug should fix recent issue from #2095. Thanks to Tristan for its
reporting and then testing of this patch.

No need to backport.
src/quic_conn.c