Page MenuHomeFreeBSD
Differential D10040

pf: Fix memory leak on vnet shutdown or unload
ClosedPublic
Actions

Authored by kp on Mar 17 2017, 5:37 AM.
Tags
None
Referenced Files
F157017034: D10040.diff
Sun, May 17, 11:32 PM
Unknown Object (File)
Sun, May 17, 12:27 PM
Unknown Object (File)
Sun, May 17, 4:49 AM
Unknown Object (File)
Sun, May 17, 4:49 AM
Unknown Object (File)
Sun, May 17, 4:36 AM
Unknown Object (File)
Sat, May 16, 6:38 PM
Unknown Object (File)
Thu, May 14, 12:47 PM
Unknown Object (File)
Mon, May 4, 4:16 PM
View All Files
Subscribers
bz

Details

Reviewers
eri
bz
Group Reviewers
network
Commits
rS315469: pf: Fix memory leak on vnet shutdown or unload
Summary

Rules are unlinked in shutdown_pf(), so we must call
pf_unload_vnet_purge(), which frees unlinked rules, after that, not
before.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kp created this revision.Mar 17 2017, 5:37 AM
kp added reviewers: network, eri.Mar 17 2017, 5:54 AM
kp set the repository for this revision to rS FreeBSD src repository - subversion.
eri accepted this revision.Mar 17 2017, 10:02 PM

Good catch.

This revision is now accepted and ready to land.Mar 17 2017, 10:02 PM
bz accepted this revision.Mar 17 2017, 10:19 PM
bz added a subscriber: bz.

It is unclear to me why I had chosen the different order in r302156 but on a quick glance this looks OK.

Closed by commit rS315469: pf: Fix memory leak on vnet shutdown or unload (authored by kp). · Explain WhyMar 18 2017, 1:37 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
netpfil/
pf/
4 lines

Diff 26379

View Options

head/sys/netpfil/pf/pf_ioctl.c

Loading...