]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: quic: fix race-condition on trace for CID retrieval
authorAmaury Denoyelle <adenoyelle@haproxy.com>
Thu, 27 Jun 2024 16:52:23 +0000 (18:52 +0200)
committerAmaury Denoyelle <adenoyelle@haproxy.com>
Fri, 28 Jun 2024 14:28:33 +0000 (16:28 +0200)
commitbbb9f8248e29e89c288ad55a0fb7c71280a335a0
tree8f88f3c1576f43fa2cf3849ad6584922ffc043f3
parent05f59a51ac5ba193ef37447ac88f74d3019c3399
BUG/MINOR: quic: fix race-condition on trace for CID retrieval

quic_rx_pkt_retrieve_conn() is used when parsing a received datagram
from the listener socket. It returned the quic_conn instance
corresponding to the first packet DCID, unless it is mapped to another
thread.

As expected, global CID tree access is protected by a lock in the
function. However, there is a race condition due to the final trace
where qc instance is dereferenced outside of the lock. Fix this by
adding a new trace under lock protection and remove qc deferencement at
function end.

This may fix first crash of github issue #2607.

This must be backported up to 2.8.
src/quic_cid.c