Page MenuHomeFreeBSD
Files F160938300

D57851.diff
No OneTemporary
Actions

D57851.diff
View Options

diff --git a/contrib/pf/pflogd/pflogd.c b/contrib/pf/pflogd/pflogd.c
--- a/contrib/pf/pflogd/pflogd.c
+++ b/contrib/pf/pflogd/pflogd.c
@@ -701,10 +701,15 @@
argc -= optind;
argv += optind;
- /* does interface exist */
- if (!if_exists(interface)) {
- warn("Failed to initialize: %s", interface);
- logmsg(LOG_ERR, "Failed to initialize: %s", interface);
+ /* filter will be used by the privileged process */
+ if (argc) {
+ filter = copy_argv(argv);
+ if (filter == NULL)
+ logmsg(LOG_NOTICE, "Failed to form filter expression");
+ }
+
+ /* initialize pcap before dropping privileges */
+ if (init_pcap()) {
logmsg(LOG_ERR, "Exiting, init failure");
exit(1);
}
@@ -721,19 +726,6 @@
tzset();
(void)umask(S_IRWXG | S_IRWXO);
- /* filter will be used by the privileged process */
- if (argc) {
- filter = copy_argv(argv);
- if (filter == NULL)
- logmsg(LOG_NOTICE, "Failed to form filter expression");
- }
-
- /* initialize pcap before dropping privileges */
- if (init_pcap()) {
- logmsg(LOG_ERR, "Exiting, init failure");
- exit(1);
- }
-
/* Privilege separation begins here */
if (priv_init()) {
logmsg(LOG_ERR, "unable to privsep");
diff --git a/libexec/rc/rc.d/pflog b/libexec/rc/rc.d/pflog
--- a/libexec/rc/rc.d/pflog
+++ b/libexec/rc/rc.d/pflog
@@ -27,18 +27,11 @@
{
load_kld pflog || return 1
- # create pflog_dev interface if needed
- if ! ifconfig $pflog_dev > /dev/null 2>&1; then
- if ! ifconfig $pflog_dev create; then
- warn "could not create $pflog_dev."
- return 1
- fi
- fi
-
- # set pflog_dev interface to up state
- if ! ifconfig $pflog_dev up; then
- warn "could not bring up $pflog_dev."
- return 1
+ # Do we need to create more devices?
+ unit=${pflog_dev#pflog}
+ max=$(sysctl -n net.pflog.if_count)
+ if [ "$unit" -ge "$max" ]; then
+ sysctl net.pflog.if_count=$(expr ${unit} + 1)
fi
# -p flag requires stripping pidfile's leading /var/run and trailing .pid
@@ -53,11 +46,6 @@
pflog_poststop()
{
- if ! ifconfig $pflog_dev down; then
- warn "could not bring down $pflog_dev."
- return 1
- fi
-
if [ "$pflog_instances" ] && [ -n "$pflog_instances" ]; then
rm $pidfile
fi
diff --git a/sbin/ifconfig/ifclone.c b/sbin/ifconfig/ifclone.c
--- a/sbin/ifconfig/ifclone.c
+++ b/sbin/ifconfig/ifclone.c
@@ -126,6 +126,11 @@
"in FreeBSD 16.0");
return;
}
+ if (sscanf(ctx->ifname, "pflog%u", &u) == 1) {
+ warnx("pflog(4) logging does not need interface creation "
+ "in FreeBSD 16.0");
+ return;
+ }
strlcpy(ifr.ifr_name, ctx->ifname, sizeof(ifr.ifr_name));
diff --git a/share/man/man4/pflog.4 b/share/man/man4/pflog.4
--- a/share/man/man4/pflog.4
+++ b/share/man/man4/pflog.4
@@ -23,18 +23,18 @@
.\" (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
.\" THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
.\"
-.Dd October 29, 2021
+.Dd June 295 2026
.Dt PFLOG 4
.Os
.Sh NAME
.Nm pflog
-.Nd packet filter logging interface
+.Nd packet filter logging BPF tap
.Sh SYNOPSIS
.Cd "device pflog"
.Sh DESCRIPTION
The
.Nm pflog
-interface is a device which makes visible all packets logged by
+is a BPF tap which makes visible all packets logged by
the packet filter,
.Xr pf 4 .
Logged packets can easily be monitored in real
@@ -42,10 +42,10 @@
.Xr tcpdump 1
on the
.Nm
-interface, or stored to disk using
+BPF tap, or stored to disk using
.Xr pflogd 8 .
.Pp
-The pflog0 interface is created when the
+The pflog0 BPF tap is created when the
.Nm
module is loaded;
further instances can be created using
@@ -58,7 +58,7 @@
.Xr pflogd 8
are enabled.
.Pp
-Each packet retrieved on this interface has a header associated
+Each packet retrieved on this BPF tap has a header associated
with it of length
.Dv PFLOG_HDRLEN .
This header documents the address family, interface name, rule
@@ -85,16 +85,14 @@
u_int32_t ridentifier;
};
.Ed
+.Pp
+The number of pflog devices can be configured with the
+.Va net.pflog.if_count
+sysctl.
.Sh EXAMPLES
-Create a
-.Nm
-interface
-and monitor all packets logged on it:
+Monitor all packets logged on pflog0:
.Bd -literal -offset indent
-# ifconfig pflog create
-pflog1
-# ifconfig pflog1 up
-# tcpdump -n -e -ttt -i pflog1
+# tcpdump -n -e -ttt -i pflog0
.Ed
.Sh SEE ALSO
.Xr tcpdump 1 ,
diff --git a/sys/net/if_pflog.h b/sys/net/if_pflog.h
--- a/sys/net/if_pflog.h
+++ b/sys/net/if_pflog.h
@@ -63,6 +63,8 @@
/* minus pad, also used as a signature */
#define PFLOG_REAL_HDRLEN offsetof(struct pfloghdr, pad2)
+#define PFLOG_MAX_DEVS 256
+
#ifdef _KERNEL
struct pf_rule;
struct pf_ruleset;
diff --git a/sys/netpfil/pf/if_pflog.c b/sys/netpfil/pf/if_pflog.c
--- a/sys/netpfil/pf/if_pflog.c
+++ b/sys/netpfil/pf/if_pflog.c
@@ -49,11 +49,11 @@
#include <sys/proc.h>
#include <sys/socket.h>
#include <sys/sockio.h>
+#include <sys/sysctl.h>
#include <net/bpf.h>
#include <net/if.h>
#include <net/if_var.h>
-#include <net/if_clone.h>
#include <net/if_pflog.h>
#include <net/if_private.h>
#include <net/if_types.h>
@@ -85,159 +85,110 @@
#define DPRINTF(x)
#endif
-static int pflogoutput(struct ifnet *, struct mbuf *,
- const struct sockaddr *, struct route *);
static void pflogattach(void);
-static int pflogifs_resize(size_t);
-static int pflogioctl(struct ifnet *, u_long, caddr_t);
-static void pflogstart(struct ifnet *);
-static int pflog_clone_create(struct if_clone *, char *, size_t,
- struct ifc_data *, struct ifnet **);
-static int pflog_clone_destroy(struct if_clone *, struct ifnet *, uint32_t);
+static int pflog_create(int);
+static void pflog_destroy(int);
+static int sysctl_pflog_if_count(SYSCTL_HANDLER_ARGS);
+static bool bpf_pflog_chkdir(void *, const struct mbuf *, int);
static const char pflogname[] = "pflog";
-VNET_DEFINE_STATIC(struct if_clone *, pflog_cloner);
-#define V_pflog_cloner VNET(pflog_cloner)
+static const struct bif_methods bpf_pflog_methods = {
+ .bif_chkdir = bpf_pflog_chkdir,
+};
-VNET_DEFINE_STATIC(int, npflogifs) = 0;
+struct pflog_dev {
+ struct bpf_if *pflog_bpf;
+ char pflog_name[IFNAMSIZ];
+};
+VNET_DEFINE_STATIC(uint8_t, npflogifs) = 8;
#define V_npflogifs VNET(npflogifs)
-VNET_DEFINE(struct ifnet **, pflogifs); /* for fast access */
+VNET_DEFINE(struct pflog_dev, pflogifs[256]); /* for fast access */
#define V_pflogifs VNET(pflogifs)
+SYSCTL_NODE(_net, OID_AUTO, pflog, CTLFLAG_RW | CTLFLAG_MPSAFE, 0,
+ "PFLOG");
+SYSCTL_PROC(_net_pflog, OID_AUTO, if_count,
+ CTLTYPE_U8 | CTLFLAG_RW | CTLFLAG_VNET,
+ 0, 0, sysctl_pflog_if_count, "CU",
+ "Number of pflog(4) interfaces");
+static struct sx pflog_dev_lock;
+SX_SYSINIT(pflog_dev_lock, &pflog_dev_lock, "pflog(4) device lock");
+
static void
pflogattach(void)
{
- struct if_clone_addreq req = {
- .create_f = pflog_clone_create,
- .destroy_f = pflog_clone_destroy,
- .flags = IFC_F_AUTOUNIT,
- };
- V_pflog_cloner = ifc_attach_cloner(pflogname, &req);
- struct ifc_data ifd = { .unit = 0 };
- ifc_create_ifp(pflogname, &ifd, NULL);
-}
-
-static int
-pflogifs_resize(size_t n)
-{
- struct ifnet **p;
- int i;
+ int ret __diagused;
- if (n > SIZE_MAX / sizeof(struct ifnet *))
- return (EINVAL);
- if (n == 0)
- p = NULL;
- else if ((p = malloc(n * sizeof(struct ifnet *), M_DEVBUF,
- M_NOWAIT | M_ZERO)) == NULL)
- return (ENOMEM);
- for (i = 0; i < n; i++) {
- if (i < V_npflogifs)
- p[i] = V_pflogifs[i];
- else
- p[i] = NULL;
+ sx_xlock(&pflog_dev_lock);
+ for (int i = 0; i < V_npflogifs; i++) {
+ ret = pflog_create(i);
+ MPASS(ret == 0);
}
-
- if (V_pflogifs)
- free(V_pflogifs, M_DEVBUF);
- V_pflogifs = p;
- V_npflogifs = n;
-
- return (0);
+ sx_xunlock(&pflog_dev_lock);
}
static int
-pflog_clone_create(struct if_clone *ifc, char *name, size_t maxlen,
- struct ifc_data *ifd, struct ifnet **ifpp)
+pflog_create(int unit)
{
- struct ifnet *ifp;
-
- ifp = if_alloc(IFT_PFLOG);
- if_initname(ifp, pflogname, ifd->unit);
- ifp->if_mtu = PFLOGMTU;
- ifp->if_ioctl = pflogioctl;
- ifp->if_output = pflogoutput;
- ifp->if_start = pflogstart;
- ifp->if_snd.ifq_maxlen = ifqmaxlen;
- ifp->if_hdrlen = PFLOG_HDRLEN;
- if_attach(ifp);
-
- bpfattach(ifp, DLT_PFLOG, PFLOG_HDRLEN);
-
- if (ifd->unit + 1 > V_npflogifs &&
- pflogifs_resize(ifd->unit + 1) != 0) {
- pflog_clone_destroy(ifc, ifp, IFC_F_FORCE);
- return (ENOMEM);
- }
- V_pflogifs[ifd->unit] = ifp;
- *ifpp = ifp;
-
- return (0);
-}
+ sx_assert(&pflog_dev_lock, SX_XLOCKED);
-static int
-pflog_clone_destroy(struct if_clone *ifc, struct ifnet *ifp, uint32_t flags)
-{
- int i;
-
- if (ifp->if_dunit == 0 && (flags & IFC_F_FORCE) == 0)
+ if (unit < 0)
+ return (EINVAL);
+ if (unit > PFLOG_MAX_DEVS)
return (EINVAL);
- for (i = 0; i < V_npflogifs; i++)
- if (V_pflogifs[i] == ifp)
- V_pflogifs[i] = NULL;
+ if (V_pflogifs[unit].pflog_bpf != NULL)
+ return (EEXIST);
- bpfdetach(ifp);
- if_detach(ifp);
- if_free(ifp);
+ snprintf(V_pflogifs[unit].pflog_name, IFNAMSIZ, "pflog%d", unit);
+
+ V_pflogifs[unit].pflog_bpf = bpf_attach(V_pflogifs[unit].pflog_name,
+ DLT_PFLOG, PFLOG_HDRLEN, &bpf_pflog_methods, NULL);
return (0);
}
-/*
- * Start output on the pflog interface.
- */
static void
-pflogstart(struct ifnet *ifp)
+pflog_destroy(int unit)
{
- struct mbuf *m;
+ sx_assert(&pflog_dev_lock, SX_XLOCKED);
- for (;;) {
- IF_LOCK(&ifp->if_snd);
- _IF_DEQUEUE(&ifp->if_snd, m);
- IF_UNLOCK(&ifp->if_snd);
-
- if (m == NULL)
- return;
- else
- m_freem(m);
- }
+ bpf_detach(V_pflogifs[unit].pflog_bpf);
+ V_pflogifs[unit].pflog_bpf = NULL;
}
static int
-pflogoutput(struct ifnet *ifp, struct mbuf *m, const struct sockaddr *dst,
- struct route *rt)
+sysctl_pflog_if_count(SYSCTL_HANDLER_ARGS)
{
- m_freem(m);
- return (0);
+ uint8_t n = V_npflogifs;
+ int error;
+
+ error = sysctl_handle_8(oidp, &n, 0, req);
+
+ if (n != V_npflogifs) {
+ sx_xlock(&pflog_dev_lock);
+ if (n > V_npflogifs) {
+ for (int i = V_npflogifs; i < n; i++) {
+ pflog_create(i);
+ }
+ } else {
+ for (int i = V_npflogifs - 1; i >= n; i--) {
+ pflog_destroy(i);
+ }
+ }
+ V_npflogifs = n;
+ sx_xunlock(&pflog_dev_lock);
+ }
+
+ return (error);
}
-/* ARGSUSED */
-static int
-pflogioctl(struct ifnet *ifp, u_long cmd, caddr_t data)
+static bool
+bpf_pflog_chkdir(void *arg __unused, const struct mbuf *m, int dir)
{
- switch (cmd) {
- case SIOCSIFFLAGS:
- if (ifp->if_flags & IFF_UP)
- ifp->if_drv_flags |= IFF_DRV_RUNNING;
- else
- ifp->if_drv_flags &= ~IFF_DRV_RUNNING;
- break;
- default:
- return (ENOTTY);
- }
-
- return (0);
+ return ((dir == BPF_D_IN && m_rcvif(m) == NULL) ||
+ (dir == BPF_D_OUT && m_rcvif(m) != NULL));
}
static int
@@ -246,9 +197,11 @@
struct pf_kruleset *ruleset, struct pf_pdesc *pd, int lookupsafe,
struct pf_krule *trigger)
{
- struct ifnet *ifn;
+ struct bpf_if *ifn;
struct pfloghdr hdr;
+ NET_EPOCH_ASSERT();
+
if (rm == NULL || pd == NULL)
return (1);
if (trigger == NULL)
@@ -257,8 +210,8 @@
if (trigger->logif > V_npflogifs)
return (0);
- ifn = V_pflogifs[trigger->logif];
- if (ifn == NULL || !bpf_peers_present(ifn->if_bpf))
+ ifn = V_pflogifs[trigger->logif].pflog_bpf;
+ if (ifn == NULL)
return (0);
bzero(&hdr, sizeof(hdr));
@@ -305,9 +258,7 @@
}
#endif /* INET */
- if_inc_counter(ifn, IFCOUNTER_OPACKETS, 1);
- if_inc_counter(ifn, IFCOUNTER_OBYTES, pd->m->m_pkthdr.len);
- bpf_mtap2(ifn->if_bpf, &hdr, PFLOG_HDRLEN, pd->m);
+ bpf_mtap2(ifn, &hdr, PFLOG_HDRLEN, pd->m);
return (0);
}
@@ -324,9 +275,12 @@
static void
vnet_pflog_uninit(const void *unused __unused)
{
-
- ifc_detach_cloner(V_pflog_cloner);
+ sx_xlock(&pflog_dev_lock);
+ for (int i = 0; i < V_npflogifs; i++)
+ pflog_destroy(i);
+ sx_xunlock(&pflog_dev_lock);
}
+
/*
* Detach after pf is gone; otherwise we might touch pflog memory
* from within pf after freeing pflog.
diff --git a/tests/sys/netpfil/pf/nat64.py b/tests/sys/netpfil/pf/nat64.py
--- a/tests/sys/netpfil/pf/nat64.py
+++ b/tests/sys/netpfil/pf/nat64.py
@@ -33,7 +33,7 @@
from atf_python.sys.net.vnet import VnetTestTemplate
class TestNAT64(VnetTestTemplate):
- REQUIRED_MODULES = [ "pf", "pflog" ]
+ REQUIRED_MODULES = [ "pf", "pflog", "pfsync" ]
TOPOLOGY = {
"vnet1": {"ifaces": ["if1"]},
"vnet2": {"ifaces": ["if1", "if2"]},
@@ -101,7 +101,7 @@
"pass in on %s inet6 af-to inet from 192.0.2.1" % ifname,
])
- vnet.pipe.send(socket.if_nametoindex("pflog0"))
+ vnet.pipe.send(socket.if_nametoindex("pfsync0"))
@pytest.mark.require_user("root")
@pytest.mark.require_progs(["scapy"])
diff --git a/tests/sys/netpfil/pf/pflog.sh b/tests/sys/netpfil/pf/pflog.sh
--- a/tests/sys/netpfil/pf/pflog.sh
+++ b/tests/sys/netpfil/pf/pflog.sh
@@ -459,6 +459,41 @@
pft_cleanup
}
+atf_test_case "create_destroy" "cleanup"
+create_destroy_head()
+{
+ atf_set descr 'Test adding/remove pflog interfaces'
+ atf_set require.user root
+}
+
+create_destroy_body()
+{
+ pflog_init
+
+ # Make sure pflog15 doesn't exist
+ atf_check -s exit:1 -e ignore timeout 3 \
+ tcpdump -i pflog15
+
+ # Create more devices
+ sysctl net.pflog.if_count=16
+
+ # It does now
+ atf_check -s exit:124 -o ignore -e ignore timeout 3 \
+ tcpdump -i pflog15
+
+ # Back down to the default 8 devices:
+ sysctl net.pflog.if_count=8
+
+ # And pflog15 no longer exists
+ atf_check -s exit:1 -e ignore timeout 3 \
+ tcpdump -i pflog15
+}
+
+create_destroy_cleanup()
+{
+ pft_cleanup
+}
+
atf_init_test_cases()
{
atf_add_test_case "malformed"
@@ -469,4 +504,5 @@
atf_add_test_case "unspecified_v6"
atf_add_test_case "rdr_action"
atf_add_test_case "rule_number"
+ atf_add_test_case "create_destroy"
}

File Metadata

Mime Type
text/plain
Expires
Tue, Jun 30, 6:27 AM (4 h, 27 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
34487003
Default Alt Text
D57851.diff (13 KB)

Event Timeline