Paths

Table of Contentst

Differential D37624

ppp: improve MSS clamping
ClosedPublic
Actions

Authored by tuexen on Dec 7 2022, 5:11 PM.

Tags

None

Referenced Files

	F163531421: D37624.diff
	Fri, Jul 24, 4:56 AM

	Unknown Object (File)
	Wed, Jul 22, 1:01 PM

	Unknown Object (File)
	Mon, Jul 20, 5:57 AM

	Unknown Object (File)
	Sat, Jul 18, 4:18 AM

	Unknown Object (File)
	Mon, Jul 13, 2:50 AM

	Unknown Object (File)
	Fri, Jul 10, 7:34 PM

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

	Unknown Object (File)
	Thu, Jul 9, 4:49 AM

Subscribers

Details

Reviewers

Commits

rG301bff9bdd62: ppp: improve MSS clamping
rGcef3c4e0bab8: ppp: improve MSS clamping

Summary

ppp supports MSS clamping for TCP/IPv4. This patch

improves MSS clamping by correctly computing the MSS to be reported as specified in RFC 6691.
adds support for MSS clamping when using TCP/IPv6.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tuexen created this revision.Dec 7 2022, 5:11 PM

Herald added a subscriber: imp. · View Herald TranscriptDec 7 2022, 5:11 PM

tuexen requested review of this revision.Dec 7 2022, 5:11 PM

thj added inline comments.Dec 7 2022, 8:19 PM

usr.sbin/ppp/tcpmss.c
165	This should move into to switch statement below to be consistent with the v6 path

tuexen added inline comments.Dec 7 2022, 10:01 PM

usr.sbin/ppp/tcpmss.c
165	The reason I added the check here is that from the next line on I assume that pip points to an IP header and in the line after that I look at the first component of it. Don't I need to make sure I have at least enough data to look at the ip_v field? I think only checking in the switch statement is too late, since I have already accessed a component. Or am I wrong?

thj accepted this revision.Dec 7 2022, 10:16 PM

This revision is now accepted and ready to land.Dec 7 2022, 10:16 PM

Closed by commit rGcef3c4e0bab8: ppp: improve MSS clamping (authored by tuexen). · Explain WhyDec 8 2022, 9:10 AM

This revision was automatically updated to reflect the committed changes.

tuexen added a commit: rGcef3c4e0bab8: ppp: improve MSS clamping.

tuexen added a commit: rG301bff9bdd62: ppp: improve MSS clamping.Feb 2 2023, 12:04 AM

Revision Contents
Changeset List

Path

Size

usr.sbin/

ppp/

76 lines

Diff 113936

usr.sbin/ppp/tcpmss.c

Loading...