Page MenuHomeFreeBSD
Differential D41008

pf: Fix duplicate storage of direction
ClosedPublic
Actions

Authored by vegeta_tuxpowered.net on Jul 12 2023, 4:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 20 2023, 7:34 AM
Unknown Object (File)
Dec 10 2023, 8:16 PM
Unknown Object (File)
Dec 9 2023, 3:31 AM
Unknown Object (File)
Sep 12 2023, 7:10 PM
Unknown Object (File)
Aug 14 2023, 9:43 PM
Unknown Object (File)
Aug 14 2023, 10:43 AM
Unknown Object (File)
Jul 15 2023, 8:22 PM
Unknown Object (File)
Jul 12 2023, 6:07 PM
View All 9 Files
Subscribers
ae
farrokhi
glebius
imp
melifaro

Details

Reviewers
kp
Commits
rGf2064dd1f170: pf: Fix duplicate storage of direction
Summary

The variable storing the direction of a processed packet is passed around to many functions. Most of those functions already have a pointer to struct pf_pdesc which also contains the direction. By using the one in struct pf_pdesc we can reduce the amount of arguments passed around.

Sponsored by: InnGames GmbH

Diff Detail

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

Event Timeline

vegeta_tuxpowered.net created this revision.Jul 12 2023, 4:34 PM
Herald added subscribers: glebius, melifaro, farrokhi and 2 others. · View Herald TranscriptJul 12 2023, 4:34 PM
vegeta_tuxpowered.net requested review of this revision.Jul 12 2023, 4:34 PM
vegeta_tuxpowered.net edited the summary of this revision. (Show Details)
kp added a comment.Jul 12 2023, 4:52 PM

Looks sensible at first glance. I'm test-building and running the tests just in case, but I expect you've done that already.

This revision was not accepted when it landed; it landed in state Needs Review.Jul 12 2023, 6:07 PM
Closed by commit rGf2064dd1f170: pf: Fix duplicate storage of direction (authored by vegeta_tuxpowered.net, committed by kp). · Explain Why
This revision was automatically updated to reflect the committed changes.
kp added a commit: rGf2064dd1f170: pf: Fix duplicate storage of direction.

Revision Contents
Changeset List

PathSize
sys/
net/
4 lines
14 lines
netpfil/
pf/
8 lines
287 lines
22 lines
38 lines

Diff 124580

View Options

sys/net/if_pflog.h

Loading...
View Options

sys/net/pfvar.h

Loading...
View Options

sys/netpfil/pf/if_pflog.c

Loading...
View Options

sys/netpfil/pf/pf.c

Loading...
View Options

sys/netpfil/pf/pf_lb.c

Loading...
View Options

sys/netpfil/pf/pf_norm.c

Loading...