HomeFreeBSD

MFC 9febbc454190:

Description

MFC 9febbc454190:
Fix for natd(8) sending wrong sequence number after TCP retransmission,
terminating a TCP connection.

If a TCP packet must be retransmitted and the data length has changed in the
retransmitted packet, due to the internal workings of TCP, typically when ACK
packets are lost, then there is a 30% chance that the logic in GetDeltaSeqOut()
will find the correct length, which is the last length received.

This can be explained as follows:

If a "227 Entering Passive Mode" packet must be retransmittet and the length
changes from 51 to 50 bytes, for example, then we have three cases for the
list scan in GetDeltaSeqOut(), depending on how many prior packets were
received modulus N_LINK_TCP_DATA=3:

case 1:  index 0:   original packet        51
         index 1:   retransmitted packet   50
         index 2:   not relevant

case 2:  index 0:   not relevant
         index 1:   original packet        51
         index 2:   retransmitted packet   50

case 3:  index 0:   retransmitted packet   50
         index 1:   not relevant
         index 2:   original packet        51

This patch simply changes the searching order for TCP packets, always starting
at the last received packet instead of any received packet, in
GetDeltaAckIn() and GetDeltaSeqOut().

Else no functional changes.

Discussed with: rscheff@
Submitted by: Andreas Longwitz <longwitz@incore.de>
PR: 230755
Sponsored by: Mellanox Technologies // NVIDIA Networking

(cherry picked from commit 9febbc4541903bb8e6b0f1c84988c98b2f7c96ef)

Details

Provenance
hselaskyAuthored on Feb 22 2021, 10:58 AM
Parents
rG68b5d0bea6d2: rmlock: Add a required compiler membar to the rlock slow path
Branches
Unknown
Tags
Unknown