Page MenuHomeFreeBSD
Differential D41772

pf: fix state leak
ClosedPublic
Actions

Authored by kp on Sep 7 2023, 3:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 13, 7:59 PM
Unknown Object (File)
Wed, May 13, 7:59 PM
Unknown Object (File)
Wed, May 13, 7:52 PM
Unknown Object (File)
Sat, May 9, 4:19 PM
Unknown Object (File)
Fri, May 8, 9:45 PM
Unknown Object (File)
Apr 21 2026, 9:19 AM
Unknown Object (File)
Apr 21 2026, 8:37 AM
Unknown Object (File)
Apr 21 2026, 3:36 AM
View All 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...