[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