[PATCH] D47995: [ASAN] fix startup crash in dlsym for long paths since glibc 2.27
Dan Liew via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 14 04:16:12 PDT 2018
delcypher added inline comments.
================
Comment at: compiler-rt/trunk/lib/asan/asan_malloc_linux.cc:53
+static void DeallocateFromLocalPool(const void *ptr) {
+ // Hack: since glibc 2.27, dlsym longer use stack-allocated memory to store
+ // error messages and instead use malloc followed by free. To avoid pool
----------------
Typo? s/dlsym longer/dlsym no longer/
================
Comment at: compiler-rt/trunk/lib/asan/asan_malloc_linux.cc:54
+ // Hack: since glibc 2.27, dlsym longer use stack-allocated memory to store
+ // error messages and instead use malloc followed by free. To avoid pool
+ // exhaustion due to long object filenames, handle that special case here.
----------------
Typo. s/use/uses/
Repository:
rL LLVM
https://reviews.llvm.org/D47995
More information about the llvm-commits
mailing list