]> git.ipfire.org Git - thirdparty/glibc.git/commit
malloc: Fix tcache leak after thread destruction [BZ #22111]
authorCarlos O'Donell <carlos@systemhalted.org>
Thu, 28 Sep 2017 17:05:18 +0000 (11:05 -0600)
committerCarlos O'Donell <carlos@systemhalted.org>
Fri, 6 Oct 2017 16:31:52 +0000 (09:31 -0700)
commit1e26d35193efbb29239c710a4c46a64708643320
tree711bdaefe5af9f9566c3a9e101b7328f565faa61
parentd13867625894fda6c6a5034dadfa8ff86983ea12
malloc: Fix tcache leak after thread destruction [BZ #22111]

The malloc tcache added in 2.26 will leak all of the elements remaining
in the cache and the cache structure itself when a thread exits. The
defect is that we do not set tcache_shutting_down early enough, and the
thread simply recreates the tcache and places the elements back onto a
new tcache which is subsequently lost as the thread exits (unfreed
memory). The fix is relatively simple, move the setting of
tcache_shutting_down earlier in tcache_thread_freeres. We add a test
case which uses mallinfo and some heuristics to look for unaccounted for
memory usage between the start and end of a thread start/join loop. It
is very reliable at detecting that there is a leak given the number of
iterations.  Without the fix the test will consume 122MiB of leaked
memory.
ChangeLog
malloc/Makefile
malloc/malloc.c
malloc/tst-malloc-tcache-leak.c [new file with mode: 0644]