Differential D16175

Use appropriate MSS when populating the TCP fast open client cookie cache
ClosedPublic
Actions

Authored by tuexen on Jul 8 2018, 9:48 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Feb 2 2024, 5:26 PM

	Unknown Object (File)
	Dec 22 2023, 9:58 PM

	Unknown Object (File)
	Nov 17 2023, 6:32 AM

	Unknown Object (File)
	Sep 29 2023, 2:29 PM

	Unknown Object (File)
	Aug 27 2023, 4:28 AM

	Unknown Object (File)
	Jul 11 2023, 7:02 PM

	Unknown Object (File)
	Jul 11 2023, 6:56 PM

	Unknown Object (File)
	Jul 11 2023, 5:01 PM

View All 14 Files

Subscribers

Details

Reviewers

pkelsey
jason_eggnet.com
rrs
kbowling
jtl

Group Reviewers

Commits

rS336167: Use appropriate MSS value when populating the TCP FO client cookie cache

Summary

When a client receives a SYN-ACK segment with a TFP fast open cookie, but without an MSS option, an MSS value from uninitialised stack memory is used. This patch ensures that in case no MSS option is included in the SYN-ACK, the appropriate value as given in the TCP fast open RFC is used.

Test Plan

Use a packetdrill script to trigger the condition and use D14554 to verify the correct MSS is used.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tuexen created this revision.Jul 8 2018, 9:48 AM

Herald added 1 blocking reviewer(s): transport. · View Herald TranscriptJul 8 2018, 9:48 AM

Herald added a subscriber: imp. · View Herald Transcript

Harbormaster completed remote builds in B17903: Diff 45008.Jul 8 2018, 9:48 AM

kbowling accepted this revision.Jul 9 2018, 9:23 AM

This revision is now accepted and ready to land.Jul 9 2018, 9:23 AM

Closed by commit rS336167: Use appropriate MSS value when populating the TCP FO client cookie cache (authored by tuexen). · Explain WhyJul 10 2018, 10:42 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

netinet/

15 lines

tcp_stacks/

30 lines

15 lines

Diff 45106

head/sys/netinet/tcp_input.c

Loading...

head/sys/netinet/tcp_stacks/fastpath.c

Loading...

head/sys/netinet/tcp_stacks/rack.c

Loading...