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.
This commit is contained in:
Frédéric Lécaille 2021-05-28 10:25:07 +02:00 committed by Amaury Denoyelle
parent ecb5872012
commit de935f34e5

View File

@ -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); LIST_APPEND(&l->rx.qpkts, &pkt->rx_list);
/* Try to accept a new connection. */ /* Try to accept a new connection. */
listener_accept(l); 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)) { if (!quic_conn_init_timer(qc)) {
TRACE_PROTO("Non initialized timer", QUIC_EV_CONN_LPKT, qc->conn); TRACE_PROTO("Non initialized timer", QUIC_EV_CONN_LPKT, qc->conn);
goto err; goto err;