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

	F175047147: D59015.diff
	Wed, Oct 7, 9:21 PM

	F175042682: D59015.id185217.diff
	Wed, Oct 7, 8:40 PM

	Unknown Object (File)
	Tue, Oct 6, 7:08 PM

	Unknown Object (File)
	Tue, Oct 6, 7:24 AM

	Unknown Object (File)
	Mon, Oct 5, 1:51 AM

	Unknown Object (File)
	Mon, Oct 5, 1:04 AM

	Unknown Object (File)
	Sun, Oct 4, 1:02 AM

	Unknown Object (File)
	Sat, Oct 3, 10:10 AM

View All 82 Files

Subscribers

vegeta_tuxpowered.net

Details

Reviewers

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

Event Timeline

markj created this revision.Aug 19 2026, 10:00 PM

Herald added subscribers: vegeta_tuxpowered.net, glebius, melifaro and 2 others. · View Herald TranscriptAug 19 2026, 10:00 PM

markj requested review of this revision.Aug 19 2026, 10:00 PM

Harbormaster completed remote builds in B75769: Diff 184408.Aug 19 2026, 10:00 PM

Improve the test a bit: actually transmit some data, and verify that it was received.

Harbormaster completed remote builds in B75773: Diff 184412.Aug 19 2026, 10:15 PM

kp accepted this revision.Aug 24 2026, 3:19 PM

This revision is now accepted and ready to land.Aug 24 2026, 3:19 PM

Closed by commit rG00a793def743: pf: Fix fallout from the STATE_LOOKUP macro removal (authored by markj). · Explain WhyAug 27 2026, 1:33 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG00a793def743: pf: Fix fallout from the STATE_LOOKUP macro removal.

markj added a commit: rG971d24f203dd: pf: Fix fallout from the STATE_LOOKUP macro removal.Thu, Sep 10, 9:14 AM

Revision Contents
Changeset List

Path

Size

sys/

netpfil/

pf/

17 lines

tests/

sys/

netpfil/

pf/

82 lines

Diff 185217

sys/netpfil/pf/pf.c

Loading...

tests/sys/netpfil/pf/route_to.sh

Loading...