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
Unknown Object (File)
Mon, Jun 29, 7:51 PM
Unknown Object (File)
Sun, Jun 28, 3:19 PM
Unknown Object (File)
Tue, Jun 23, 7:05 PM
Unknown Object (File)
Mon, Jun 22, 7:34 PM
Unknown Object (File)
Sun, Jun 21, 1:09 PM
Unknown Object (File)
Sat, Jun 20, 12:00 AM
Unknown Object (File)
Tue, Jun 16, 5:22 AM
Unknown Object (File)
May 25 2026, 2:08 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
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
339 lines
7 lines

Diff 87185

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...