From 140e0522a2ad42ec78d8eeba7414cb22af2a924b Mon Sep 17 00:00:00 2001 From: Gaurav Singh Date: Fri, 3 Jan 2025 17:59:20 +0530 Subject: [PATCH] New Largest Packet Number check shifted --- src/core/ack_tracker.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/core/ack_tracker.c b/src/core/ack_tracker.c index 538930546e..5ba46d3810 100644 --- a/src/core/ack_tracker.c +++ b/src/core/ack_tracker.c @@ -239,7 +239,7 @@ QuicAckTrackerAckPacket( Tracker->AckElicitingPacketsToAcknowledge++; - if ((Connection->Send.SendFlags & QUIC_CONN_SEND_FLAG_ACK) || !NewLargestPacketNumber) { + if (Connection->Send.SendFlags & QUIC_CONN_SEND_FLAG_ACK) { goto Exit; // Already queued to send an ACK, no more work to do. } @@ -262,7 +262,8 @@ QuicAckTrackerAckPacket( if (AckType == QUIC_ACK_TYPE_ACK_IMMEDIATE || Connection->Settings.MaxAckDelayMs == 0 || (Tracker->AckElicitingPacketsToAcknowledge >= (uint16_t)Connection->PacketTolerance) || - QuicAckTrackerDidHitReorderingThreshold(Tracker, Connection->ReorderingThreshold)) { + (NewLargestPacketNumber && + QuicAckTrackerDidHitReorderingThreshold(Tracker, Connection->ReorderingThreshold))) { // // Send the ACK immediately. //