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)
Thu, Jun 1, 6:21 PM
Unknown Object (File)
Sat, May 27, 7:27 AM
Unknown Object (File)
Sat, May 27, 7:22 AM
Unknown Object (File)
Sat, May 27, 1:09 AM
Unknown Object (File)
Sat, May 27, 1:06 AM
Unknown Object (File)
Fri, May 26, 2:02 PM
Unknown Object (File)
Mon, May 8, 7:33 PM
Unknown Object (File)
May 3 2023, 1:40 AM
View All 18 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...