pf tests: check that we clean up unused kifs
ClosedPublic
Actions

Authored by kp on Nov 30 2022, 4:16 PM.

Details

Reviewers

None

Group Reviewers

network
pfsense

Commits

rG16603f60156e: pf tests: check that we clean up unused kifs
rGe0724177b584: pf tests: check that we clean up unused kifs
rG654e8d84ec8b: pf tests: check that we clean up unused kifs

Summary

The previous commit fixed a memory leak, where we'd fail to clean up
removed groups (and interfaces).
Check that we now clean those up as expected.

MFC after: 2 weeks
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.Nov 30 2022, 4:16 PM

Herald added subscribers: glebius, melifaro, farrokhi, imp. · View Herald TranscriptNov 30 2022, 4:16 PM

kp requested review of this revision.Nov 30 2022, 4:16 PM

Harbormaster completed remote builds in B48571: Diff 113679.Nov 30 2022, 4:16 PM

kp added a parent revision: D37569: pf: fix pfi_ifnet leak on interface removal.Nov 30 2022, 4:16 PM

This revision was not accepted when it landed; it landed in state Needs Review.Dec 14 2022, 9:21 AM

Closed by commit rG654e8d84ec8b: pf tests: check that we clean up unused kifs (authored by kp). · Explain Why

This revision was automatically updated to reflect the committed changes.

kp added a commit: rG654e8d84ec8b: pf tests: check that we clean up unused kifs.

kp added a commit: rGe0724177b584: pf tests: check that we clean up unused kifs.Dec 28 2022, 9:04 AM

kp added a commit: rG16603f60156e: pf tests: check that we clean up unused kifs.Dec 28 2022, 9:08 AM

Revision Contents
Changeset List

Path

Size

tests/

sys/

netpfil/

pf/

names.sh

44 lines

Diff 114136

View Options

tests/sys/netpfil/pf/names.sh

pf tests: check that we clean up unused kifsClosedPublicActions