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

	Unknown Object (File)
	Nov 18 2024, 5:57 AM

	Unknown Object (File)
	Nov 18 2024, 5:50 AM

	Unknown Object (File)
	Oct 24 2024, 4:29 PM

	Unknown Object (File)
	Oct 5 2024, 2:59 PM

	Unknown Object (File)
	Oct 5 2024, 10:54 AM

	Unknown Object (File)
	Oct 2 2024, 10:06 PM

	Unknown Object (File)
	Oct 2 2024, 7:04 PM

	Unknown Object (File)
	Sep 28 2024, 5:33 AM

View All 43 Files

Subscribers

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.

Good catch.

This revision is now accepted and ready to land.Mar 17 2017, 10:02 PM

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

Path

Size

head/

sys/

netpfil/

pf/

pf_ioctl.c

4 lines

Diff 26379

View Options

head/sys/netpfil/pf/pf_ioctl.c

pf: Fix memory leak on vnet shutdown or unloadClosedPublicActions