Page MenuHomeFreeBSD
Differential D28650

pf: Slightly relax pf_rule_addr validation
ClosedPublic
Actions

Authored by kp on Feb 13 2021, 4:33 PM.
Tags
None
Referenced Files
F166524051: D28650.diff
Fri, Aug 14, 12:43 AM
F166374430: D28650.id83872.diff
Thu, Aug 13, 5:49 AM
Unknown Object (File)
Sun, Aug 9, 5:20 PM
Unknown Object (File)
Wed, Aug 5, 5:42 AM
Unknown Object (File)
Wed, Aug 5, 2:26 AM
Unknown Object (File)
Tue, Aug 4, 5:14 AM
Unknown Object (File)
Fri, Jul 24, 6:27 PM
Unknown Object (File)
Thu, Jul 23, 1:36 AM
View All Files
Subscribers
donner
farrokhi
imp
melifaro

Details

Reviewers
donner
Group Reviewers
network
Commits
rG00abeecb4a25: pf: Slightly relax pf_rule_addr validation
rGf8d1f2da0922: pf: Slightly relax pf_rule_addr validation
rGf9a66bb91ae1: pf: Slightly relax pf_rule_addr validation
rG5e42cb139fc1: pf: Slightly relax pf_rule_addr validation
Summary

Ensure we don't reject no-route / urpf-failed addresses.

PR: 253479
MFC after: 3 days

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 36976
Build 33865: arc lint + arc unit

Event Timeline

kp created this revision.Feb 13 2021, 4:33 PM
Herald added subscribers: melifaro, farrokhi, imp. · View Herald TranscriptFeb 13 2021, 4:33 PM
kp requested review of this revision.Feb 13 2021, 4:33 PM
Harbormaster completed remote builds in B36971: Diff 83835.Feb 13 2021, 4:33 PM
donner added a subscriber: donner.Feb 13 2021, 7:52 PM
donner added inline comments.
sys/netpfil/pf/pf_ioctl.c
1564–1569

May give "switch () { case x: case y: ... " a try. It's easier to maintain and offers more flexibility for the compiler.

kp updated this revision to Diff 83848.Feb 13 2021, 8:26 PM

Use switch

Harbormaster completed remote builds in B36976: Diff 83848.Feb 13 2021, 8:27 PM
donner accepted this revision as: donner.Feb 14 2021, 11:16 AM
This revision is now accepted and ready to land.Feb 14 2021, 11:16 AM
Closed by commit rG5e42cb139fc1: pf: Slightly relax pf_rule_addr validation (authored by kp). · Explain WhyFeb 14 2021, 12:09 PM
This revision was automatically updated to reflect the committed changes.
kp added a commit: rG5e42cb139fc1: pf: Slightly relax pf_rule_addr validation.
kp added a commit: rGf9a66bb91ae1: pf: Slightly relax pf_rule_addr validation.Feb 17 2021, 1:05 PM
kp added a commit: rGf8d1f2da0922: pf: Slightly relax pf_rule_addr validation.
kp added a commit: rG00abeecb4a25: pf: Slightly relax pf_rule_addr validation.Feb 17 2021, 4:50 PM

Revision Contents
Changeset List

PathSize
sys/
netpfil/
pf/
47 lines

Diff 83848

View Options

sys/netpfil/pf/pf_ioctl.c

Loading...