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

	Unknown Object (File)
	Thu, Oct 31, 11:04 AM

	Unknown Object (File)
	Tue, Oct 22, 3:36 AM

	Unknown Object (File)
	Fri, Oct 18, 4:14 AM

	Unknown Object (File)
	Fri, Oct 18, 3:01 AM

	Unknown Object (File)
	Sep 24 2024, 5:56 AM

	Unknown Object (File)
	Sep 17 2024, 4:55 PM

	Unknown Object (File)
	Sep 17 2024, 4:54 PM

	Unknown Object (File)
	Sep 17 2024, 4:54 PM

View All 65 Files

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 Skipped

Unit

Tests Skipped

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 113907

usr.sbin/ppp/tcpmss.c

Loading...