Paths

Table of Contentst

ipfw: Add ICMPv6 type(s) to ipfw syntax summary usage
Needs ReviewPublic
Actions

Authored by jlduran on Mar 1 2025, 4:57 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Aug 7, 5:38 PM

	Unknown Object (File)
	Tue, Aug 4, 5:20 AM

	Unknown Object (File)
	Mon, Aug 3, 6:29 AM

	Unknown Object (File)
	Fri, Jul 31, 6:41 AM

	Unknown Object (File)
	Fri, Jul 24, 7:36 PM

	Unknown Object (File)
	Wed, Jul 22, 4:11 PM

	Unknown Object (File)
	Sun, Jul 12, 5:12 PM

	Unknown Object (File)
	Jul 4 2026, 1:07 AM

Subscribers

Details

Reviewers

emaste
pouria

Group Reviewers

network

Summary

Account for the IPv6 ICMP types in the usage. While here account for
both singular and plural forms as well.

PR: 284865

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 73178
Build 70061: arc lint + arc unit

Event Timeline

jlduran created this revision.Mar 1 2025, 4:57 PM

Herald added subscribers: donner, imp. · View Herald TranscriptMar 1 2025, 4:57 PM

jlduran requested review of this revision.Mar 1 2025, 4:57 PM

Harbormaster completed remote builds in B62712: Diff 151718.Mar 1 2025, 4:57 PM

Adding both singular and plural form is good to me.
But adding another icmp6types is redundant.

sbin/ipfw/main.c
74	there is one `icmp6types` right here.

This revision now requires changes to proceed.Feb 25 2026, 1:42 PM

pouria retitled this revision from ipfw: Add IPv6 ICMP type(s) to ipfw: Add ICMPv6 type(s) to ipfw syntax summary usage.Feb 25 2026, 1:45 PM

Address sugesstions:

Update the commit title
Remove duplicate icmp6types entry

Harbormaster completed remote builds in B73178: Diff 177992.May 18 2026, 2:21 AM

jlduran marked an inline comment as done.May 18 2026, 2:22 AM

pouria accepted this revision as: pouria.May 18 2026, 5:19 PM

Revision Contents
Changeset List

Path

Size

sbin/

ipfw/

main.c

6 lines

Diff 177992

View Options

ipfw: Add ICMPv6 type(s) to ipfw syntax summary usageNeeds ReviewPublicActions