[PATCH] D31978: Fix memory leaks in address sanitizer darwin tests

[Francis Ricci via Phabricator via llvm-commits]

fjricci added inline comments.


================
Comment at: test/asan/TestCases/Darwin/malloc_set_zone_name-mprotect.cc:52
+  // but it's leaked unless cleared
+  malloc_set_zone_name(zone, NULL);
+  malloc_destroy_zone(zone);
----------------
fjricci wrote:
> alekseyshl wrote:
> > Who's holding (and leaking) the buffer?
> From looking at the only malloc_set_zone_name source I could find (https://opensource.apple.com/source/Libc/Libc-320.1.3/gen/malloc.c), it looks like libc itself is holding a copy, which it frees whenever the name is overwritten. For some reason, malloc_destroy_zone doesn't free that buffer (I assume it should).
Another alternative which would probably work would be calling 'free(malloc_get_zone_name(zone))`, but that seemed hackier.


https://reviews.llvm.org/D31978