]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
tcp: Fix bugs related to the PUSH bit and rack and an ack war
authorRandall Stewart <rrs@FreeBSD.org>
Tue, 25 May 2021 17:23:31 +0000 (13:23 -0400)
committerMichael Tuexen <tuexen@FreeBSD.org>
Wed, 9 Jun 2021 00:13:32 +0000 (02:13 +0200)
commit12e181b6727326f79173182d04eb799eab3bdec7
tree479c4d5d815c0c321bb8059f1dcd6e329fa97fba
parente99fa57b981eaeac287a1d0d8c0b351740f7a30b
tcp: Fix bugs related to the PUSH bit and rack and an ack war

Michaels testing with UDP tunneling found an issue with the push bit, which was only partly fixed
in the last commit. The problem is the left edge gets transmitted before the adjustments are done
to the send_map, this means that right edge bits must be considered to be added only if
the entire RSM is being retransmitted.

Now syzkaller also continued to find a crash, which Michael sent me the reproducer for. Turns
out that the reproducer on default (freebsd) stack made the stack get into an ack-war with itself.
After fixing the reference issues in rack the same ack-war was found in rack (and bbr). Basically
what happens is we go into the reassembly code and lose the FIN bit. The trick here is we
should not be going into the reassembly code if tlen == 0 i.e. the peer never sent you anything.
That then gets the proper action on the FIN bit but then you end up in LAST_ACK with no
timers running. This is because the usrclosed function gets called and the FIN's and such have
already been exchanged. So when we should be entering FIN_WAIT2 (or even FIN_WAIT1) we get
stuck in LAST_ACK. Fixing this means tweaking the usrclosed function so that we properly
recognize the condition and drop into FIN_WAIT2 where a timer will allow at least TP_MAXIDLE
before closing (to allow time for the peer to retransmit its FIN if the ack is lost). Setting the fast_finwait2
timer can speed this up in testing.

Reviewed by: mtuexen,rscheff
Sponsored by: Netflix Inc
Differential Revision: https://reviews.freebsd.org/D30451

(cherry picked from commit 13c0e198ca275447f9a60a03f730c38c98f19009)
sys/netinet/tcp_input.c
sys/netinet/tcp_stacks/bbr.c
sys/netinet/tcp_stacks/rack.c
sys/netinet/tcp_usrreq.c