Page MenuHomeFreeBSD
Differential D59015

pf: Fix fallout from the STATE_LOOKUP macro removal
ClosedPublic
Actions

Authored by markj on Aug 19 2026, 10:00 PM.
Tags
None
Referenced Files
F174384641: D59015.id185217.diff
Fri, Oct 2, 8:20 PM
F174335299: D59015.diff
Fri, Oct 2, 12:03 PM
F174335294: D59015.diff
Fri, Oct 2, 12:03 PM
Unknown Object (File)
Thu, Oct 1, 9:54 PM
Unknown Object (File)
Thu, Oct 1, 9:19 PM
Unknown Object (File)
Tue, Sep 29, 6:38 PM
Unknown Object (File)
Sun, Sep 27, 5:23 PM
Unknown Object (File)
Sun, Sep 27, 5:22 PM
View All 73 Files
Subscribers
farrokhi
glebius
imp
melifaro
vegeta_tuxpowered.net

Details

Reviewers
kp
Commits
rG971d24f203dd: pf: Fix fallout from the STATE_LOOKUP macro removal
rG00a793def743: pf: Fix fallout from the STATE_LOOKUP macro removal
Summary

Commit 8572367b6814 ("pf: remove STATE_LOOKUP") introduced two seemingly
unintentional changes with respect to divert(4)-injected packets (i.e.,
the PACKET_LOOPED case): we no longer return the matching state, and
direct callers of pf_find_state() now treat matches of diverted packets
the same as having no matching state at all.

This seems inadvertent, and breaks certain rulesets which use divert-to.
Fix them, and add a regression test case.

Fixes: 8572367b6814 ("pf: remove STATE_LOOKUP")

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
sys/
netpfil/
pf/
17 lines
tests/
sys/
netpfil/
pf/
82 lines

Diff 185217

View Options

sys/netpfil/pf/pf.c

Loading...
View Options

tests/sys/netpfil/pf/route_to.sh

Loading...