[PATCH] D55711: Don't trigger sanitizer initialization from `sysctlbyname` interceptor.
Kamil Rytarowski via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 14 15:37:05 PST 2018
krytarowski added a comment.
In D55711#1331469 <https://reviews.llvm.org/D55711#1331469>, @vitalybuka wrote:
> If we are going to support this use-case we need a test.
>
> I think you can clone/update sysctl.cc, and define all possible __*san_default_options as following:
>
> __tsan_default_options() {
> test_sysctl();
> test_sysctlbyname();
> test_sysctlnametomib();
> }
>
>
> This is probably early enough.
> Please check that the new test fail without your patch.
It's a good idea for a test, once it will be shared I will cross-check this for NetBSD.
Repository:
rCRT Compiler Runtime
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55711/new/
https://reviews.llvm.org/D55711
More information about the llvm-commits
mailing list