Page MenuHomeFreeBSD
Differential D29560

pfctl: Use the new DIOCGETRULENV ioctl
ClosedPublic
Actions

Authored by kp on Apr 2 2021, 6:55 PM.
Tags
None
Referenced Files
F157230157: D29560.id87026.diff
Tue, May 19, 12:51 PM
F157215307: D29560.id.diff
Tue, May 19, 10:05 AM
F157183900: D29560.id87185.diff
Tue, May 19, 2:31 AM
Unknown Object (File)
Mon, May 18, 10:07 AM
Unknown Object (File)
Sun, May 17, 4:55 PM
Unknown Object (File)
Sat, May 16, 8:16 PM
Unknown Object (File)
Mon, May 11, 3:57 AM
Unknown Object (File)
Mon, May 11, 3:57 AM
View All Files
Subscribers
imp
takahiro.kurosawa_gmail.com

Details

Reviewers
None
Group Reviewers
network
Commits
rGad208b077153: pfctl: Use the new DIOCGETRULENV ioctl
rGd14cf045873b: pfctl: Use the new DIOCGETRULENV ioctl
rG0d6c8174ef2f: pfctl: Use the new DIOCGETRULENV ioctl
Summary

Create wrapper functions to handle the parsing of the nvlist and move
that code into pfctl_ioctl.c.
At some point this should be moved into a libpfctl.

MFC after: 4 weeks
Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

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

Event Timeline

kp created this revision.Apr 2 2021, 6:55 PM
Herald added a subscriber: imp. · View Herald TranscriptApr 2 2021, 6:55 PM
kp requested review of this revision.Apr 2 2021, 6:55 PM
kp added a parent revision: D29559: pf: Implement nvlist variant of DIOCGETRULE.
Harbormaster completed remote builds in B38282: Diff 86762.Apr 2 2021, 6:55 PM
kp added a child revision: D29561: libnv: Build PIC.Apr 2 2021, 6:55 PM
kp mentioned this in D29468: pf: Implement the NAT source port selection of MAP-E Customer Edge.Apr 2 2021, 6:59 PM
takahiro.kurosawa_gmail.com added a subscriber: takahiro.kurosawa_gmail.com.Apr 2 2021, 7:52 PM
kp updated this revision to Diff 87026.Apr 8 2021, 12:31 PM

Rebase

  • remove rt_listid
  • Return qid/pqid (which pfctl can display for debugging)
Harbormaster completed remote builds in B38406: Diff 87026.Apr 8 2021, 12:32 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 10 2021, 9:17 AM
Closed by commit rG0d6c8174ef2f: pfctl: Use the new DIOCGETRULENV ioctl (authored by kp). · Explain Why
This revision was automatically updated to reflect the committed changes.
kp added a commit: rG0d6c8174ef2f: pfctl: Use the new DIOCGETRULENV ioctl.
kp added a commit: rGd14cf045873b: pfctl: Use the new DIOCGETRULENV ioctl.May 7 2021, 3:27 PM
kp added a commit: rGad208b077153: pfctl: Use the new DIOCGETRULENV ioctl.

Revision Contents
Changeset List

PathSize
sbin/
pfctl/
2 lines
12 lines
43 lines
338 lines
7 lines

Diff 86762

View Options

sbin/pfctl/Makefile

Loading...
View Options

sbin/pfctl/pfctl.c

Loading...
View Options

sbin/pfctl/pfctl_ioctl.h

Loading...
View Options

sbin/pfctl/pfctl_ioctl.c

Loading...
View Options

sbin/pfctl/pfctl_optimize.c

Loading...