Page MenuHomeFreeBSD
Differential D41772

pf: fix state leak
ClosedPublic
Actions

Authored by kp on Sep 7 2023, 3:39 PM.
Tags
None
Referenced Files
F150493039: D41772.diff
Wed, Apr 1, 4:51 PM
Unknown Object (File)
Thu, Mar 19, 5:24 AM
Unknown Object (File)
Wed, Mar 18, 4:29 AM
Unknown Object (File)
Sun, Mar 15, 4:39 AM
Unknown Object (File)
Fri, Mar 13, 6:10 PM
Unknown Object (File)
Mar 2 2026, 8:06 PM
Unknown Object (File)
Dec 29 2025, 6:27 AM
Unknown Object (File)
Dec 24 2025, 8:02 PM
View All 98 Files
Subscribers
farrokhi
glebius
imp
melifaro

Details

Reviewers
None
Group Reviewers
pfsense
network
Commits
rGddca690373e5: pf: fix state leak
rGed82385135cc: pf: fix state leak
rGb6ce41118bb1: pf: fix state leak
Summary

If we hit the csfailed case in pf_create_state() we may have allocated
a state, so we must also free it. While here reduce the amount of
duplicated cleanup code.

MFC after: 1 week
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 2023, 3:39 PM
Herald added subscribers: glebius, melifaro, farrokhi, imp. · View Herald TranscriptSep 7 2023, 3:39 PM
kp requested review of this revision.Sep 7 2023, 3:39 PM
Harbormaster completed remote builds in B53493: Diff 127065.Sep 7 2023, 3:39 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 10 2023, 8:40 PM
Closed by commit rGb6ce41118bb1: pf: fix state leak (authored by kp). · Explain Why
This revision was automatically updated to reflect the committed changes.
kp added a commit: rGb6ce41118bb1: pf: fix state leak.
kp added a commit: rGed82385135cc: pf: fix state leak.Sep 25 2023, 12:27 PM
kp added a commit: rGddca690373e5: pf: fix state leak.

Revision Contents
Changeset List

PathSize
sys/
netpfil/
pf/
26 lines

Diff 127209

View Options

sys/netpfil/pf/pf.c

Loading...