From de935f34e5aa0d3c8e12aa56308623a63ed806e8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20L=C3=A9caille?= Date: Fri, 28 May 2021 10:25:07 +0200 Subject: [PATCH] BUG/MINOR: quic: Do not check the acception of a new conn from I/O handler. As the ->conn member of quic_conn struct is reset to NULL value by the ->accept callback potentially run by another thread, this check is irrelevant. --- src/xprt_quic.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/xprt_quic.c b/src/xprt_quic.c index 633767a1a..6e2fde9b5 100644 --- a/src/xprt_quic.c +++ b/src/xprt_quic.c @@ -3267,11 +3267,6 @@ static ssize_t qc_lstnr_pkt_rcv(unsigned char **buf, const unsigned char *end, LIST_APPEND(&l->rx.qpkts, &pkt->rx_list); /* Try to accept a new connection. */ listener_accept(l); - if (!qc->conn) { - TRACE_PROTO("Non accepted connection", QUIC_EV_CONN_LPKT, qc->conn); - goto err; - } - if (!quic_conn_init_timer(qc)) { TRACE_PROTO("Non initialized timer", QUIC_EV_CONN_LPKT, qc->conn); goto err;