[PATCH] D32457: [asan] Don't reset non-default user handler if allow_user_signal_handler is true.

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 25 11:15:42 PDT 2017


vitalybuka added inline comments.


================
Comment at: test/asan/TestCases/Linux/preinstalled_signal.cc:48
+
+  void *asan = dlopen(asan_rt, RTLD_NOW);
+  assert(asan);
----------------
eugenis wrote:
> I don't like this at all. This is a mode that is not supposed to work.
> 
> Could you set a signal handler in preinit_array?
> On Linux (and new-ish Android) preinit_array ctors get argc/argv/envp just as main().
> 
No, asan also uses preinit_array and will be executed before this.


https://reviews.llvm.org/D32457





More information about the llvm-commits mailing list