From: Frédéric Lécaille <flecaille@haproxy.com>
Date: Mon, 27 Dec 2021 17:09:22 +0000 (+0100)
Subject: MINOR: quic: Wrong first packet number space computation
X-Git-Tag: v2.6-dev1~182
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=1f6cf18183e46ca88d4ba928d4e1f5c3b2219b69;p=thirdparty%2Fhaproxy.git

MINOR: quic: Wrong first packet number space computation

I really do not know where does these inversion come from.
---

diff --git a/include/haproxy/quic_loss.h b/include/haproxy/quic_loss.h
index 092046f12a..04b5084bd2 100644
--- a/include/haproxy/quic_loss.h
+++ b/include/haproxy/quic_loss.h
@@ -111,7 +111,7 @@ static inline struct quic_pktns *quic_loss_pktns(struct quic_conn *qc)
 	TRACE_PROTO("pktns", QUIC_EV_CONN_SPTO, qc, pktns);
 	for (i = QUIC_TLS_PKTNS_HANDSHAKE; i < QUIC_TLS_PKTNS_MAX; i++) {
 		TRACE_PROTO("pktns", QUIC_EV_CONN_SPTO, qc, &qc->pktns[i]);
-		if (tick_isset(pktns->tx.loss_time) &&
+		if (!tick_isset(pktns->tx.loss_time) ||
 		    qc->pktns[i].tx.loss_time < pktns->tx.loss_time)
 			pktns = &qc->pktns[i];
 	}