Page MenuHomeFreeBSD

frag6: fix vnet teardown leak
ClosedPublic

Authored by bz on Oct 16 2019, 9:55 AM.
Tags
None
Referenced Files
F168056578: D22054.id63347.diff
Wed, Aug 26, 4:18 AM
F168018838: D22054.id.diff
Tue, Aug 25, 11:00 PM
Unknown Object (File)
Tue, Aug 11, 11:26 AM
Unknown Object (File)
Tue, Aug 11, 9:42 AM
Unknown Object (File)
Tue, Aug 11, 9:30 AM
Unknown Object (File)
Sun, Aug 9, 2:35 PM
Unknown Object (File)
Sun, Aug 9, 2:14 AM
Unknown Object (File)
Sat, Aug 8, 11:13 PM
Subscribers

Details

Summary

When shutting down a VNET we did not cleanup the fragmentation hashes.
This has multiple problems: (1) leak memory but also (2) leak on the
global counters, which might eventually lead to a problem on a system
starting and stopping a lot of vnets and dealing with a lot of IPv6
fragments that the statistics would be exhausted (maybe memory would
be gone first).

Unfortunately we do not have a useable variable to indicate when
per-VNET initialization of frag6 has happened (or when destroy happened)
so introduce a boolean to flag this. This is needed here as well as
it was in r353635 for ip_reass.c in order to avoid tripping over the
already destroyed locks if interfaces go away after the frag6 destroy.

Reported by: hselasky
Reviewed by:

Diff Detail

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