pf: Rationalize the ip_divert_ptr test
ClosedPublic
Actions

Authored by markj on Jan 23 2026, 8:55 PM.

Details

Reviewers

kp
igoro
glebius

Commits

rG4775399b4e87: pf: Rationalize the ip_divert_ptr test
rG39878d24a690: pf: Rationalize the ip_divert_ptr test

Summary

If a rule has a divert port set, then we can reasonably predict that
ipdivert.ko is loaded, and in particular that ip_divert_ptr is set.

Moreover, in this case, if ipdivert.ko is not loaded we should just drop
the packet instead of ignoring the divert rule.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Jan 23 2026, 8:55 PM

Herald added subscribers: vegeta_tuxpowered.net, melifaro, farrokhi, imp. · View Herald TranscriptJan 23 2026, 8:56 PM

markj requested review of this revision.Jan 23 2026, 8:56 PM

Harbormaster completed remote builds in B70084: Diff 170289.Jan 23 2026, 8:56 PM

markj added a child revision: D54846: pf tests: Set require.kmods in divert-to tests.Jan 23 2026, 8:56 PM

glebius accepted this revision.Jan 23 2026, 9:11 PM

This revision is now accepted and ready to land.Jan 23 2026, 9:11 PM

kp accepted this revision.Jan 24 2026, 11:41 AM

igoro accepted this revision.Jan 24 2026, 3:44 PM

Closed by commit rG39878d24a690: pf: Rationalize the ip_divert_ptr test (authored by markj). · Explain WhyJan 26 2026, 5:25 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG39878d24a690: pf: Rationalize the ip_divert_ptr test.

markj added a commit: rG4775399b4e87: pf: Rationalize the ip_divert_ptr test.Feb 9 2026, 6:05 PM

Revision Contents
Changeset List

Path

Size

sys/

netpfil/

pf/

pf.c

15 lines

Diff 170474

View Options

pf: Rationalize the ip_divert_ptr testClosedPublicActions