The ICMP echo is too small
ClosedPublic
Actions

Authored by rrs on Jul 18 2018, 10:46 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Jul 23, 9:00 AM

	Unknown Object (File)
	Fri, Jul 11, 8:51 AM

	Unknown Object (File)
	Jun 28 2025, 10:35 AM

	Unknown Object (File)
	Jun 25 2025, 7:41 PM

	Unknown Object (File)
	Jun 15 2025, 7:07 AM

	Unknown Object (File)
	Jun 10 2025, 8:16 PM

	Unknown Object (File)
	Jun 10 2025, 1:25 AM

	Unknown Object (File)
	Jun 1 2025, 4:56 PM

View All 62 Files

Subscribers

imp

Details

Reviewers

tuexen

Group Reviewers

transport

Commits

rS336465: Bump the ICMP echo limits to match the RFC

Summary

The amount we echo (8 bytes) is from the 1980's and the router requirements document
requires it to be much higher

Test Plan

Grow it and make sure we reflect larger packets back

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

rrs created this revision.Jul 18 2018, 10:46 PM

Add a fullstop at the end of the comment.

This revision is now accepted and ready to land.Jul 18 2018, 10:48 PM

Closed by commit rS336465: Bump the ICMP echo limits to match the RFC (authored by rrs). · Explain WhyJul 18 2018, 10:50 PM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptJul 18 2018, 10:50 PM

Revision Contents
Changeset List

Path

Size

head/

sys/

netinet/

ip_icmp.c

4 lines

tcp_stacks/

rack.c

2 lines

Diff 45502

View Options

head/sys/netinet/ip_icmp.c