Differential D25909

Fix handling of TTL/Hop Limit and TOS/Traffic Class in the TCP SYN Cache
ClosedPublic
Actions

Authored by tuexen on Jul 31 2020, 10:03 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Jul 19, 3:13 PM

	Unknown Object (File)
	Sun, Jul 19, 12:59 AM

	Unknown Object (File)
	Sat, Jul 18, 6:37 AM

	Unknown Object (File)
	Fri, Jul 17, 7:41 PM

	Unknown Object (File)
	Sun, Jul 12, 1:01 PM

	Unknown Object (File)
	Sat, Jul 11, 10:14 PM

	Unknown Object (File)
	Thu, Jul 9, 6:57 AM

	Unknown Object (File)
	Mon, Jul 6, 1:01 PM

Subscribers

Details

Reviewers

rscheff
rrs
bz
donner

Group Reviewers

Commits

rS364089: Fix the following issues related to the TCP SYN-cache:

Summary

This patch fixes the following issues:

Let the accepted TCP/IPv4 socket inherit the configured TTL and TOS value.
Let the accepted TCP/IPv6 socket inherit the configured Hop Limit.
Use the configured Hop Limit and Traffic Class when sending IPv6 packets.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 32698

Event Timeline

tuexen created this revision.Jul 31 2020, 10:03 PM

Herald added a reviewer: transport. · View Herald TranscriptJul 31 2020, 10:03 PM

Herald added subscribers: melifaro, imp. · View Herald Transcript

tuexen requested review of this revision.Jul 31 2020, 10:03 PM

Harbormaster completed remote builds in B32687: Diff 75234.Jul 31 2020, 10:03 PM

donner added a subscriber: donner.Aug 1 2020, 7:57 PM

donner added inline comments.

sys/netinet/tcp_syncache.c
1391–1392	May this check for IPv6 become a single test, which covers the tests below? This would improve the reading of the whole construct.
1404–1413	A bit superfluous. isn't it?
1415–1416	This could be a single #ifdef INET (in the #ifdef INET6 scope)

tuexen added inline comments.Aug 1 2020, 9:53 PM

sys/netinet/tcp_syncache.c
1415–1416	I'm using the pattern also used at tcp_syncache.c:1849 for consistency within the same file.

Simplify code as suggested by Lutz.

Harbormaster completed remote builds in B32698: Diff 75258.Aug 1 2020, 10:05 PM

tuexen added inline comments.Aug 1 2020, 10:06 PM

sys/netinet/tcp_syncache.c
1391–1392	Done in latest revision.
1404–1413	Should also be fixed by the latest revision.

donner accepted this revision as: donner.Aug 2 2020, 11:11 AM

rrs accepted this revision.Aug 10 2020, 6:34 PM

This revision is now accepted and ready to land.Aug 10 2020, 6:34 PM

Closed by commit rS364089: Fix the following issues related to the TCP SYN-cache: (authored by tuexen). · Explain WhyAug 10 2020, 8:25 PM

This revision was automatically updated to reflect the committed changes.

tuexen added a commit: rS364089: Fix the following issues related to the TCP SYN-cache:.

Revision Contents
Changeset List

Path

Size

sys/

netinet/

4 lines

41 lines

Diff 75258

sys/netinet/tcp_syncache.h

Loading...

sys/netinet/tcp_syncache.c

Loading...