Page MenuHomeFreeBSD
Differential D16604

Rack sends extra FIN
ClosedPublic
Actions

Authored by rrs on Aug 6 2018, 9:31 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jul 31, 4:16 AM
Unknown Object (File)
Tue, Jul 28, 3:18 PM
Unknown Object (File)
Tue, Jul 21, 10:29 AM
Unknown Object (File)
Sun, Jul 19, 9:15 PM
Unknown Object (File)
Mon, Jul 13, 5:46 AM
Unknown Object (File)
Sun, Jul 12, 1:38 AM
Unknown Object (File)
Sat, Jul 11, 11:30 AM
Unknown Object (File)
Jul 6 2026, 12:51 PM
View All Files
Subscribers
imp

Details

Reviewers
tuexen
Group Reviewers
transport
Commits
rS337455: Fix a small bug in rack where it will
Summary

There is a small bug in rack where when closing a connection it will
send an extra FIN. This is because it checks the wrong variable
and should not allow a goto send unless the FIN has never been sent.
A timer will set the ACK_NOW flag.

Test Plan

Use a the attached packetdrill script to make sure the problem goes away.

D16604.pkt2 KBDownload

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

rrs created this revision.Aug 6 2018, 9:31 AM
tuexen accepted this revision.Aug 6 2018, 7:04 PM
tuexen edited the test plan for this revision. (Show Details)
This revision is now accepted and ready to land.Aug 6 2018, 7:04 PM
Closed by commit rS337455: Fix a small bug in rack where it will (authored by rrs). · Explain WhyAug 8 2018, 1:37 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptAug 8 2018, 1:37 PM

Revision Contents
Changeset List

PathSize
head/
sys/
netinet/
tcp_stacks/
11 lines

Diff 46409

View Options

head/sys/netinet/tcp_stacks/rack.c

Loading...