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
Unknown Object (File)
Wed, Oct 8, 7:09 AM
Unknown Object (File)
Sat, Oct 4, 2:49 AM
Unknown Object (File)
Thu, Sep 18, 3:27 PM
Unknown Object (File)
Wed, Sep 17, 3:32 PM
Unknown Object (File)
Wed, Sep 17, 8:43 AM
Unknown Object (File)
Mon, Sep 15, 1:06 PM
Unknown Object (File)
Aug 20 2025, 9:09 AM
Unknown Object (File)
Jul 10 2025, 6:02 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
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 83872

View Options

sys/netpfil/pf/pf_ioctl.c

Loading...