Page MenuHomeFreeBSD

frag6: fix vnet teardown leak
ClosedPublic

Authored by bz on Oct 16 2019, 9:55 AM.
Tags
None
Referenced Files
F154392090: D22054.id63347.diff
Tue, Apr 28, 7:38 AM
F154349364: D22054.id63509.diff
Tue, Apr 28, 12:40 AM
Unknown Object (File)
Mon, Apr 27, 4:32 PM
Unknown Object (File)
Sun, Apr 26, 2:14 AM
Unknown Object (File)
Fri, Apr 24, 7:33 PM
Unknown Object (File)
Sat, Apr 18, 4:51 AM
Unknown Object (File)
Tue, Apr 14, 9:20 PM
Unknown Object (File)
Tue, Apr 7, 1:54 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