Page MenuHomeFreeBSD

pf: merge pf_scrub_ip() and pf_scrub_ip6()
ClosedPublic

Authored by kp on Sep 7 2024, 9:37 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 17, 2:01 PM
Unknown Object (File)
Thu, Jan 2, 7:41 PM
Unknown Object (File)
Thu, Jan 2, 4:50 AM
Unknown Object (File)
Nov 24 2024, 9:32 PM
Unknown Object (File)
Nov 6 2024, 7:11 AM
Unknown Object (File)
Nov 4 2024, 3:00 AM
Unknown Object (File)
Oct 28 2024, 4:41 AM
Unknown Object (File)
Oct 7 2024, 8:10 AM

Details

Summary

Merge pf_scrub_ip() and pf_scrub_ip6() into a single function. Call
pf_scrub with the right arugments in the rule case so that match
rules will work as expected.

OK henning@

Obtained from: OpenBSD, claudio <claudio@openbsd.org>, 48c45e6969
Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

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

Event Timeline

kp requested review of this revision.Sep 7 2024, 9:37 AM
zlei added a subscriber: zlei.
zlei added inline comments.
sys/netpfil/pf/pf_norm.c
2281–2283

Emm, I'd personally prefer big switch case blocks, rather than test pd->af everywhere. But no objection since it is from upstream.

This revision is now accepted and ready to land.Sep 14 2024, 8:32 AM
This revision was automatically updated to reflect the committed changes.