The ICMP echo is too small
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F81587438: D16333.diff
	Thu, Apr 18, 1:39 PM

	Unknown Object (File)
	Tue, Apr 16, 9:42 AM

	Unknown Object (File)
	Feb 15 2024, 8:47 AM

	Unknown Object (File)
	Dec 20 2023, 12:16 AM

	Unknown Object (File)
	Dec 14 2023, 1:39 PM

	Unknown Object (File)
	Nov 13 2023, 10:50 AM

	Unknown Object (File)
	Oct 19 2023, 7:06 PM

	Unknown Object (File)
	Sep 11 2023, 6:55 AM

View All 20 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