[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