BUG/MINOR: quic: Crash from trace dumping SSL eary data status (AWS-LC)
authorFrederic Lecaille <flecaille@haproxy.com>
Mon, 2 Sep 2024 07:36:19 +0000 (09:36 +0200)
committerFrederic Lecaille <flecaille@haproxy.com>
Thu, 5 Sep 2024 14:54:16 +0000 (16:54 +0200)
commit763a0367a1d7944dce11e18dfa5733ddf1a0c8d7
tree64526d4574dc7be54a15419fb06ff38e5f69464f
parente875aa59a9216d42639b802b5008afc733e4c940
BUG/MINOR: quic: Crash from trace dumping SSL eary data status (AWS-LC)

This bug follows this patch:
     MINOR: quic: Add trace for QUIC_EV_CONN_IO_CB event.
where a new third variable was added to be dumped from QUIC_EV_CONN_IO_CB trace
event. The quic_trace() code did not reveal there was already another variable
passed as third argument but not dumped. This leaded to crash when dereferencing
a point to an int in place of a point to an SSL object.

This issue was reproduced only by handshakecorruption aws-lc interop test with
s2n-quic as client.

Note that this patch must be backported with this one:
     BUG/MEDIUM: quic: always validate sender address on 0-RTT
which depends on the commit mentionned above.

(cherry picked from commit db13df3d6e64e9993b90f048734538491082cbed)
Signed-off-by: Frederic Lecaille <flecaille@haproxy.com>
src/quic_ssl.c
src/quic_trace.c