diff options
author | Kostya Kortchinsky <kostyak@google.com> | 2018-06-29 14:56:25 +0000 |
---|---|---|
committer | Kostya Kortchinsky <kostyak@google.com> | 2018-06-29 14:56:25 +0000 |
commit | ff1f01dfe070e713fd8fb5e3a94fa7c69f092fbb (patch) | |
tree | a9b4714cbc85ea3843c64d03b1f89b29bf74ad7c /test/scudo | |
parent | edb018532fec4577c481cf7d4fbea1f21bd137e5 (diff) |
[scudo] Add some runtime tests for the minimal runtime
Summary:
As well as some tests to ensure that various combinations of the clang command
line flags work (shared/static/minimal).
Reviewers: eugenis, alekseyshl, vitalybuka
Reviewed By: vitalybuka
Subscribers: srhines, delcypher, #sanitizers, llvm-commits
Differential Revision: https://reviews.llvm.org/D48553
git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@335981 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/scudo')
-rw-r--r-- | test/scudo/fsanitize.c | 28 | ||||
-rw-r--r-- | test/scudo/lit.cfg | 8 | ||||
-rw-r--r-- | test/scudo/preload.cpp | 3 | ||||
-rw-r--r-- | test/scudo/symbols.test | 8 |
4 files changed, 43 insertions, 4 deletions
diff --git a/test/scudo/fsanitize.c b/test/scudo/fsanitize.c new file mode 100644 index 000000000..7e5d5451f --- /dev/null +++ b/test/scudo/fsanitize.c @@ -0,0 +1,28 @@ +// Test various -fsanitize= additional flags combinations. + +// RUN: %clang_scudo %s -o %t +// RUN: not %run %t 2>&1 | FileCheck %s + +// RUN: %clang_scudo -shared-libsan %s -o %t +// RUN: env LD_LIBRARY_PATH=`dirname %shared_libscudo`:$LD_LIBRARY_PATH not %run %t 2>&1 | FileCheck %s +// RUN: %clang_scudo -shared-libsan -fsanitize-minimal-runtime %s -o %t +// RUN: env LD_LIBRARY_PATH=`dirname %shared_minlibscudo`:$LD_LIBRARY_PATH not %run %t 2>&1 | FileCheck %s + +// RUN: %clang_scudo -static-libsan %s -o %t +// RUN: not %run %t 2>&1 | FileCheck %s +// RUN: %clang_scudo -static-libsan -fsanitize-minimal-runtime %s -o %t +// RUN: not %run %t 2>&1 | FileCheck %s + +#include <assert.h> +#include <stdlib.h> + +int main(int argc, char *argv[]) { + unsigned long *p = (unsigned long *)malloc(sizeof(unsigned long)); + assert(p); + *p = 0; + free(p); + free(p); + return 0; +} + +// CHECK: ERROR: invalid chunk state diff --git a/test/scudo/lit.cfg b/test/scudo/lit.cfg index b7f814ebc..df78d5f9d 100644 --- a/test/scudo/lit.cfg +++ b/test/scudo/lit.cfg @@ -10,9 +10,10 @@ config.test_source_root = os.path.dirname(__file__) # Path to the shared library shared_libscudo = os.path.join(config.compiler_rt_libdir, "libclang_rt.scudo%s.so" % config.target_suffix) +shared_minlibscudo = os.path.join(config.compiler_rt_libdir, "libclang_rt.scudo_minimal-%s.so" % config.target_arch) # Test suffixes. -config.suffixes = ['.c', '.cc', '.cpp'] +config.suffixes = ['.c', '.cc', '.cpp', '.test'] # C & CXX flags. c_flags = ([config.target_cflags] + @@ -32,14 +33,15 @@ cxx_flags = (c_flags + config.cxx_mode_flags + ["-std=c++11"]) scudo_flags = ["-fsanitize=scudo"] -def build_invocation(compile_flags): +def build_invocation(compile_flags): return " " + " ".join([config.clang] + compile_flags) + " " -# Add clang substitutions. +# Add substitutions. config.substitutions.append(("%clang ", build_invocation(c_flags))) config.substitutions.append(("%clang_scudo ", build_invocation(c_flags + scudo_flags))) config.substitutions.append(("%clangxx_scudo ", build_invocation(cxx_flags + scudo_flags))) config.substitutions.append(("%shared_libscudo", shared_libscudo)) +config.substitutions.append(("%shared_minlibscudo", shared_minlibscudo)) # Platform-specific default SCUDO_OPTIONS for lit tests. default_scudo_opts = '' diff --git a/test/scudo/preload.cpp b/test/scudo/preload.cpp index b41a70e47..7fa8df4c6 100644 --- a/test/scudo/preload.cpp +++ b/test/scudo/preload.cpp @@ -1,7 +1,8 @@ // Test that the preloaded runtime works without linking the static library. // RUN: %clang %s -lstdc++ -o %t -// RUN: env LD_PRELOAD=%shared_libscudo not %run %t 2>&1 | FileCheck %s +// RUN: env LD_PRELOAD=%shared_libscudo not %run %t 2>&1 | FileCheck %s +// RUN: env LD_PRELOAD=%shared_minlibscudo not %run %t 2>&1 | FileCheck %s // This way of setting LD_PRELOAD does not work with Android test runner. // REQUIRES: !android diff --git a/test/scudo/symbols.test b/test/scudo/symbols.test new file mode 100644 index 000000000..0425e62ba --- /dev/null +++ b/test/scudo/symbols.test @@ -0,0 +1,8 @@ +UNSUPPORTED: android + +Verify that various functions are *not* present in the minimal binary. Presence +of those symbols in the minimal runtime would mean that the split code made it +back into the core Sanitizer runtime library. + +RUN: nm %shared_minlibscudo | not grep Symbolizer +RUN: nm %shared_minlibscudo | not grep Coverage |