From ca3fff7b8c46e27247fe1ef813c51af6c5dda2a6 Mon Sep 17 00:00:00 2001 From: Kostya Serebryany Date: Fri, 8 Jan 2016 22:48:19 +0000 Subject: [tsan] don't crash on closedir(0) git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@257223 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/tsan/rtl/tsan_interceptors.cc | 6 ++++-- test/sanitizer_common/TestCases/Linux/closedir.c | 5 +++++ 2 files changed, 9 insertions(+), 2 deletions(-) create mode 100644 test/sanitizer_common/TestCases/Linux/closedir.c diff --git a/lib/tsan/rtl/tsan_interceptors.cc b/lib/tsan/rtl/tsan_interceptors.cc index 62c96cb42..7c835c6dc 100644 --- a/lib/tsan/rtl/tsan_interceptors.cc +++ b/lib/tsan/rtl/tsan_interceptors.cc @@ -1892,8 +1892,10 @@ TSAN_INTERCEPTOR(int, rmdir, char *path) { TSAN_INTERCEPTOR(int, closedir, void *dirp) { SCOPED_TSAN_INTERCEPTOR(closedir, dirp); - int fd = dirfd(dirp); - FdClose(thr, pc, fd); + if (dirp) { + int fd = dirfd(dirp); + FdClose(thr, pc, fd); + } return REAL(closedir)(dirp); } diff --git a/test/sanitizer_common/TestCases/Linux/closedir.c b/test/sanitizer_common/TestCases/Linux/closedir.c new file mode 100644 index 000000000..990628db4 --- /dev/null +++ b/test/sanitizer_common/TestCases/Linux/closedir.c @@ -0,0 +1,5 @@ +// Check that closedir(NULL) is ok. +// RUN: %clang -O2 %s -o %t && %run %t +#include +#include +int main() { closedir(0); } -- cgit v1.2.1