From: Frédéric Lécaille <flecaille@haproxy.com>
Date: Wed, 28 Jun 2023 16:28:51 +0000 (+0200)
Subject: CLEANUP: quic: Remove a useless test about discarded pktns (qc_handle_crypto_frm())
X-Git-Tag: v2.9-dev1~5
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=75ae0f7bbc08e8b39ab75bf08f32f306ad6f0fc9;p=thirdparty%2Fhaproxy.git

CLEANUP: quic: Remove a useless test about discarded pktns (qc_handle_crypto_frm())

There is no need to check that the packet number space associated to the encryption
level to handle the CRYPTO frames is available when entering qc_handle_crypto_frm().
This has already been done by the caller: qc_treat_rx_pkts().
---

diff --git a/src/quic_conn.c b/src/quic_conn.c
index 2927058da8..a3b4d3ffeb 100644
--- a/src/quic_conn.c
+++ b/src/quic_conn.c
@@ -2936,11 +2936,6 @@ static int qc_handle_crypto_frm(struct quic_conn *qc,
 	struct ncbuf *ncbuf = &qel->cstream->rx.ncbuf;
 
 	TRACE_ENTER(QUIC_EV_CONN_PRSHPKT, qc);
-	if (unlikely(qel->tls_ctx.flags & QUIC_FL_TLS_SECRETS_DCD)) {
-		TRACE_PROTO("CRYPTO data discarded",
-					QUIC_EV_CONN_RXPKT, qc, pkt, &cfdebug);
-		goto done;
-	}
 
 	if (unlikely(crypto_frm->offset < cstream->rx.offset)) {
 		size_t diff;