diff options
author | Kuba Brecka <kuba.brecka@gmail.com> | 2016-03-15 15:53:39 +0000 |
---|---|---|
committer | Kuba Brecka <kuba.brecka@gmail.com> | 2016-03-15 15:53:39 +0000 |
commit | 08eefe1431281e59392c9bc4289872041b5585f7 (patch) | |
tree | 166d9476ff25713550d587e607d6405e5ca72059 | |
parent | d7724fcbaf8d9f71fe17d2414741b2596379e098 (diff) |
Revert r263551 due to a test failure.
git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@263553 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/sanitizer_common/sanitizer_mac.cc | 15 | ||||
-rw-r--r-- | test/lit.common.cfg | 8 | ||||
-rw-r--r-- | test/tsan/Darwin/dlopen.cc | 41 |
3 files changed, 0 insertions, 64 deletions
diff --git a/lib/sanitizer_common/sanitizer_mac.cc b/lib/sanitizer_common/sanitizer_mac.cc index 0b025fcb9..25c1c683d 100644 --- a/lib/sanitizer_common/sanitizer_mac.cc +++ b/lib/sanitizer_common/sanitizer_mac.cc @@ -627,21 +627,6 @@ void MaybeReexec() { CHECK("execv failed" && 0); } - // Verify that interceptors really work. We'll use dlsym to locate - // "pthread_create", if interceptors are working, it should really point to - // "wrap_pthread_create" within our own dylib. - Dl_info info_pthread_create; - void *dlopen_addr = dlsym(RTLD_DEFAULT, "pthread_create"); - CHECK(dladdr(dlopen_addr, &info_pthread_create)); - if (internal_strcmp(info.dli_fname, info_pthread_create.dli_fname) != 0) { - Report( - "ERROR: Interceptors are not working. This may be because %s is " - "loaded too late (e.g. via dlopen). Please launch the executable " - "with:\n%s=%s\n", - SanitizerToolName, kDyldInsertLibraries, info.dli_fname); - CHECK("interceptors not installed" && 0); - } - if (!lib_is_in_env) return; diff --git a/test/lit.common.cfg b/test/lit.common.cfg index 595a9e919..5603dbec2 100644 --- a/test/lit.common.cfg +++ b/test/lit.common.cfg @@ -118,14 +118,6 @@ if config.can_symbolize: lit.util.usePlatformSdkOnDarwin(config, lit_config) -if config.host_os == 'Darwin': - ld_cmd = subprocess.Popen(["bash", "-c", "sw_vers -productVersion | awk -F '.' '{print $1 \".\" $2}'"], stdout = subprocess.PIPE) - ld_out = ld_cmd.stdout.read().decode() - ld_cmd.wait() - osx_version = float(ld_out) - if osx_version >= 10.11: - config.available_features.add('osx-autointerception') - sancovcc_path = os.path.join(llvm_tools_dir, "sancov") if os.path.exists(sancovcc_path): config.available_features.add("has_sancovcc") diff --git a/test/tsan/Darwin/dlopen.cc b/test/tsan/Darwin/dlopen.cc deleted file mode 100644 index 116d69f14..000000000 --- a/test/tsan/Darwin/dlopen.cc +++ /dev/null @@ -1,41 +0,0 @@ -// Checks that on OS X 10.11+ (where we do not re-exec anymore, because -// interceptors work automatically), dlopen'ing a TSanified library from a -// non-instrumented program exits with a user-friendly message. - -// REQUIRES: osx-autointerception - -// RUN: %clangxx_tsan %s -o %t.so -shared -DSHARED_LIB -// RUN: %clangxx_tsan -fno-sanitize=thread %s -o %t - -// RUN: TSAN_DYLIB_PATH=`%clangxx_tsan %s -### 2>&1 \ -// RUN: | grep "libclang_rt.tsan_osx_dynamic.dylib" \ -// RUN: | sed -e 's/.*"\(.*libclang_rt.tsan_osx_dynamic.dylib\)".*/\1/'` - -// Launching a non-instrumented binary that dlopen's an instrumented library should fail. -// RUN: not %run %t-noninstr %t.so 2>&1 | FileCheck %s --check-prefix=CHECK-FAIL -// Launching a non-instrumented binary with an explicit DYLD_INSERT_LIBRARIES should work. -// RUN: DYLD_INSERT_LIBRARIES=$TSAN_DYLIB_PATH %run %t-noninstr %t.so 2>&1 | FileCheck %s - -#include <dlfcn.h> -#include <pthread.h> -#include <stdio.h> - -#if defined(SHARED_LIB) -void foo() { - fprintf(stderr, "Hello world.\n"); -} -#else // defined(SHARED_LIB) -int main(int argc, char *argv[]) { - void *handle = dlopen(argv[1], RTLD_NOW); - fprintf(stderr, "handle = %p\n", handle); - void (*foo)() = (void (*)())dlsym(handle, "foo"); - fprintf(stderr, "foo = %p\n", foo); - foo(); -} -#endif // defined(SHARED_LIB) - -// CHECK: Hello world. -// CHECK-NOT: ERROR: Interceptors are not working. - -// CHECK-FAIL-NOT: Hello world. -// CHECK-FAIL: ERROR: Interceptors are not working. |