Paths

Table of Contentst

sack_filter: guard against unsigned underflow (tcp_fixed_maxseg)
AbandonedPublic
Actions

Authored by thebugfixers_pm.me on Jun 24 2026, 11:14 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Aug 13, 6:16 AM

	Unknown Object (File)
	Tue, Aug 11, 6:46 PM

	Unknown Object (File)
	Sun, Aug 9, 3:55 PM

	Unknown Object (File)
	Sun, Aug 9, 3:24 AM

	Unknown Object (File)
	Sat, Aug 8, 10:06 PM

	Unknown Object (File)
	Sat, Aug 8, 10:57 AM

	Unknown Object (File)
	Fri, Aug 7, 9:55 AM

	Unknown Object (File)
	Thu, Aug 6, 11:41 PM

Subscribers

Details

Reviewers

tuexen

Group Reviewers

Src Committers
Contributor Reviews (src)
transport

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

thebugfixers_pm.me created this revision.Jun 24 2026, 11:14 AM

Herald added a reviewer: transport. · View Herald TranscriptJun 24 2026, 11:14 AM

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

thebugfixers_pm.me requested review of this revision.Jun 24 2026, 11:14 AM

thebugfixers_pm.me added a parent revision: D57813: sack_filter: Brand-new path in sack_filter_blks.Jun 26 2026, 5:25 PM

tuexen requested changes to this revision.Jun 28 2026, 12:06 PM

tuexen added a reviewer: tuexen.

This revision now requires changes to proceed.Jun 28 2026, 12:06 PM

tcp_fixed_maxseg() is only used when to code is compiled for being tested in userland. So it might make more sense to add a check when processing the -S option to ensure that the value provided is at least 12. But again, this has no impact for the code being used inside the kernel.

thebugfixers_pm.me abandoned this revision.Jun 28 2026, 3:32 PM

Revision Contents
Changeset List

Path

Size

sys/

netinet/

tcp_stacks/

sack_filter.c

2 lines

Diff 180523

View Options

sys/netinet/tcp_stacks/sack_filter.c

sack_filter: guard against unsigned underflow (tcp_fixed_maxseg)AbandonedPublicActions