From 11a6f4007b908b49ecd3abd5cd10fba177f07c11 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20L=C3=A9caille?= Date: Tue, 16 Aug 2022 14:48:59 +0200 Subject: [PATCH] BUG/MINOR: quic: Wrong status returned by qc_pkt_decrypt() This bug came with this big commit: "MEDIUM: quic: xprt traces rework" This is the variable value which must be returned by most of the xprt functions. This leaded packets which could not be decrypted to be parsed, with weird frames to be parsed as found by Tristan in GH #1808. To be backported where the commit above was backported. --- src/xprt_quic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/xprt_quic.c b/src/xprt_quic.c index 762a3725b..bcd99ae70 100644 --- a/src/xprt_quic.c +++ b/src/xprt_quic.c @@ -1453,7 +1453,7 @@ static int qc_pkt_decrypt(struct quic_rx_packet *pkt, struct quic_enc_level *qel ret = 1; leave: TRACE_LEAVE(QUIC_EV_CONN_RXPKT, qc); - return 1; + return ret; }