Differential D46596

pf: do not pass af to PFLOG_PACKET
ClosedPublic
Actions

Authored by kp on Sep 7 2024, 9:37 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Oct 7, 1:48 PM

	Unknown Object (File)
	Mon, Oct 7, 4:00 AM

	Unknown Object (File)
	Thu, Oct 3, 4:28 AM

	Unknown Object (File)
	Fri, Sep 27, 11:53 PM

	Unknown Object (File)
	Mon, Sep 23, 3:02 AM

	Unknown Object (File)
	Thu, Sep 19, 1:37 PM

	Unknown Object (File)
	Tue, Sep 17, 6:21 AM

	Unknown Object (File)
	Mon, Sep 16, 8:59 PM

View All 19 Files

Subscribers

vegeta_tuxpowered.net

Details

Reviewers

Group Reviewers

network
pfsense

Commits

rGbc4b0defb012: pf: do not pass af to PFLOG_PACKET

Summary

Do not pass AF specific information to pf_test_rule() and PFLOG_PACKET()
because either the info is already available in struct pd or easy
to figure out. Makes pf_test() and pf_test6() even more similar
(with the target to remove one of them in the near future).
OK henning@

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

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kp created this revision.Sep 7 2024, 9:37 AM

Herald added subscribers: vegeta_tuxpowered.net, glebius, melifaro and 3 others. · View Herald TranscriptSep 7 2024, 9:37 AM

kp requested review of this revision.Sep 7 2024, 9:37 AM

Harbormaster completed remote builds in B59384: Diff 143092.Sep 7 2024, 9:37 AM

kp added a parent revision: D46595: pf tests: ensure we don't NAT to link-local addresses.Sep 7 2024, 9:37 AM

kp added a child revision: D46597: pf: merge pf_scrub_ip() and pf_scrub_ip6().

Looks good to me.

This revision is now accepted and ready to land.Sat, Sep 14, 8:24 AM

Closed by commit rGbc4b0defb012: pf: do not pass af to PFLOG_PACKET (authored by kp). · Explain WhyThu, Sep 19, 8:22 PM

This revision was automatically updated to reflect the committed changes.

kp added a commit: rGbc4b0defb012: pf: do not pass af to PFLOG_PACKET.

Revision Contents
Changeset List

Path

Size

sys/

net/

4 lines

2 lines

netpfil/

pf/

6 lines

20 lines

10 lines

Diff 143493

sys/net/if_pflog.h

Loading...

sys/net/pfvar.h

Loading...

sys/netpfil/pf/if_pflog.c

Loading...

sys/netpfil/pf/pf.c

Loading...

sys/netpfil/pf/pf_norm.c

Loading...