Differential D34008

pfctl: improve error reporting for routehost
ClosedPublic
Actions

Authored by kp on Jan 23 2022, 3:05 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Apr 21 2024, 11:06 PM

	Unknown Object (File)
	Apr 21 2024, 10:17 PM

	Unknown Object (File)
	Apr 21 2024, 10:16 PM

	Unknown Object (File)
	Apr 21 2024, 10:16 PM

	Unknown Object (File)
	Mar 30 2024, 2:05 PM

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

	Unknown Object (File)
	Dec 12 2023, 11:58 AM

	Unknown Object (File)
	Nov 15 2023, 12:00 PM

View All 28 Files

Subscribers

Details

Reviewers

None

Group Reviewers

Commits

rGb5f6f687a2ee: pfctl: improve error reporting for routehost
rG8ac3a1785343: pfctl: improve error reporting for routehost
rGe68de6694381: pfctl: improve error reporting for routehost

Summary

If an invalid (i.e. overly long) interface name is specified error out
immediately, rather than in expand_rule() so we point at the incorrect
line.

PR: 260958
MFC after: 3 weeks

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 44057
Build 40945: arc lint + arc unit

Event Timeline

kp created this revision.Jan 23 2022, 3:05 PM

Herald added a subscriber: imp. · View Herald TranscriptJan 23 2022, 3:05 PM

kp requested review of this revision.Jan 23 2022, 3:05 PM

Harbormaster completed remote builds in B44057: Diff 101825.Jan 23 2022, 3:05 PM

This revision was not accepted when it landed; it landed in state Needs Review.Jan 27 2022, 7:51 AM

Closed by commit rGe68de6694381: pfctl: improve error reporting for routehost (authored by kp). · Explain Why

This revision was automatically updated to reflect the committed changes.

kp added a commit: rGe68de6694381: pfctl: improve error reporting for routehost.

kp added a commit: rG8ac3a1785343: pfctl: improve error reporting for routehost.Feb 18 2022, 10:46 AM

kp added a commit: rGb5f6f687a2ee: pfctl: improve error reporting for routehost.

Revision Contents
Changeset List

Path

Size

sbin/

pfctl/

11 lines

Diff 101825

sbin/pfctl/parse.y

Loading...