diff options
| author | Kuba Brecka <kuba.brecka@gmail.com> | 2015-11-12 11:29:21 +0000 |
|---|---|---|
| committer | Kuba Brecka <kuba.brecka@gmail.com> | 2015-11-12 11:29:21 +0000 |
| commit | ba85da8482315fa1f5c364120004cd20dfa5c365 (patch) | |
| tree | 7a2c4b45bd781805a10c155281881612f1156278 /compiler-rt | |
| parent | 317e98b1a23512ffe813189d614e3c660502f755 (diff) | |
| download | bcm5719-llvm-ba85da8482315fa1f5c364120004cd20dfa5c365.tar.gz bcm5719-llvm-ba85da8482315fa1f5c364120004cd20dfa5c365.zip | |
[asan] Remove system_malloc_zone
This seems to be dead code – `system_malloc_zone` is initialized as null and we never assign to it. This code is apparently currently never executed (in ASan), but is causes trouble for the TSan OS X port. Let's replace the checks with `COMMON_MALLOC_ENTER` (`ENSURE_ASAN_INITED()`).
Differential Revision: http://reviews.llvm.org/D14334
llvm-svn: 252876
Diffstat (limited to 'compiler-rt')
| -rw-r--r-- | compiler-rt/lib/sanitizer_common/sanitizer_malloc_mac.inc | 17 |
1 files changed, 3 insertions, 14 deletions
diff --git a/compiler-rt/lib/sanitizer_common/sanitizer_malloc_mac.inc b/compiler-rt/lib/sanitizer_common/sanitizer_malloc_mac.inc index 33e9d96b8ce..17d3a1349f5 100644 --- a/compiler-rt/lib/sanitizer_common/sanitizer_malloc_mac.inc +++ b/compiler-rt/lib/sanitizer_common/sanitizer_malloc_mac.inc @@ -29,8 +29,6 @@ // Similar code is used in Google Perftools, // http://code.google.com/p/google-perftools. -// TODO(glider): do we need both zones? -static malloc_zone_t *system_malloc_zone = 0; static malloc_zone_t sanitizer_zone; INTERCEPTOR(malloc_zone_t *, malloc_create_zone, @@ -155,10 +153,7 @@ size_t __sanitizer_mz_size(malloc_zone_t* zone, const void* ptr) { extern "C" SANITIZER_INTERFACE_ATTRIBUTE void *__sanitizer_mz_malloc(malloc_zone_t *zone, uptr size) { - if (UNLIKELY(!COMMON_MALLOC_SANITIZER_INITIALIZED)) { - CHECK(system_malloc_zone); - return malloc_zone_malloc(system_malloc_zone, size); - } + COMMON_MALLOC_ENTER(); COMMON_MALLOC_MALLOC(size); return p; } @@ -184,10 +179,7 @@ void *__sanitizer_mz_calloc(malloc_zone_t *zone, size_t nmemb, size_t size) { extern "C" SANITIZER_INTERFACE_ATTRIBUTE void *__sanitizer_mz_valloc(malloc_zone_t *zone, size_t size) { - if (UNLIKELY(!COMMON_MALLOC_SANITIZER_INITIALIZED)) { - CHECK(system_malloc_zone); - return malloc_zone_valloc(system_malloc_zone, size); - } + COMMON_MALLOC_ENTER(); COMMON_MALLOC_VALLOC(size); return p; } @@ -246,10 +238,7 @@ void __sanitizer_mz_destroy(malloc_zone_t* zone) { extern "C" SANITIZER_INTERFACE_ATTRIBUTE void *__sanitizer_mz_memalign(malloc_zone_t *zone, size_t align, size_t size) { - if (UNLIKELY(!COMMON_MALLOC_SANITIZER_INITIALIZED)) { - CHECK(system_malloc_zone); - return malloc_zone_memalign(system_malloc_zone, align, size); - } + COMMON_MALLOC_ENTER(); COMMON_MALLOC_MEMALIGN(align, size); return p; } |

