aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDimitry Andric <dim@FreeBSD.org>2017-12-28 23:58:28 +0000
committerDimitry Andric <dim@FreeBSD.org>2017-12-28 23:58:28 +0000
commit0d8e7490d6e8a13a8f0977d9b7771803b9f64ea0 (patch)
tree21b1390c80adf149dbf0a3cf182485942499dd65
parentce2df4ef53e14dc3b8585123e49729f1cdf4c0af (diff)
downloadsrc-vendor/compiler-rt/compiler-rt-trunk-r321545.tar.gz
src-vendor/compiler-rt/compiler-rt-trunk-r321545.zip
Vendor import of compiler-rt trunk r321545:vendor/compiler-rt/compiler-rt-trunk-r321545
-rw-r--r--lib/sanitizer_common/sanitizer_common_interceptors.inc15
-rw-r--r--lib/sanitizer_common/sanitizer_platform_interceptors.h2
-rw-r--r--lib/tsan/rtl/tsan_libdispatch_mac.cc5
-rw-r--r--test/msan/pthread_getname_np.cc34
4 files changed, 56 insertions, 0 deletions
diff --git a/lib/sanitizer_common/sanitizer_common_interceptors.inc b/lib/sanitizer_common/sanitizer_common_interceptors.inc
index 592a8c7b032a..32342682a595 100644
--- a/lib/sanitizer_common/sanitizer_common_interceptors.inc
+++ b/lib/sanitizer_common/sanitizer_common_interceptors.inc
@@ -4551,6 +4551,20 @@ INTERCEPTOR(int, pthread_setname_np, uptr thread, const char *name, void *arg) {
#define INIT_PTHREAD_SETNAME_NP
#endif
+#if SANITIZER_INTERCEPT_PTHREAD_GETNAME_NP
+INTERCEPTOR(int, pthread_getname_np, uptr thread, char *name, SIZE_T len) {
+ void *ctx;
+ COMMON_INTERCEPTOR_ENTER(ctx, pthread_getname_np, thread, name, len);
+ int res = REAL(pthread_getname_np)(thread, name, len);
+ if (!res)
+ COMMON_INTERCEPTOR_WRITE_RANGE(ctx, name, internal_strnlen(name, len) + 1);
+ return res;
+}
+#define INIT_PTHREAD_GETNAME_NP COMMON_INTERCEPT_FUNCTION(pthread_getname_np);
+#else
+#define INIT_PTHREAD_GETNAME_NP
+#endif
+
#if SANITIZER_INTERCEPT_SINCOS
INTERCEPTOR(void, sincos, double x, double *sin, double *cos) {
void *ctx;
@@ -6541,6 +6555,7 @@ static void InitializeCommonInterceptors() {
INIT_TTYNAME_R;
INIT_TEMPNAM;
INIT_PTHREAD_SETNAME_NP;
+ INIT_PTHREAD_GETNAME_NP;
INIT_SINCOS;
INIT_REMQUO;
INIT_LGAMMA;
diff --git a/lib/sanitizer_common/sanitizer_platform_interceptors.h b/lib/sanitizer_common/sanitizer_platform_interceptors.h
index 7e33fa59b312..b99ac4480586 100644
--- a/lib/sanitizer_common/sanitizer_platform_interceptors.h
+++ b/lib/sanitizer_common/sanitizer_platform_interceptors.h
@@ -343,6 +343,8 @@
#define SANITIZER_INTERCEPT_PHTREAD_MUTEX SI_POSIX
#define SANITIZER_INTERCEPT_PTHREAD_SETNAME_NP \
(SI_FREEBSD || SI_NETBSD || SI_LINUX_NOT_ANDROID || SI_SOLARIS)
+#define SANITIZER_INTERCEPT_PTHREAD_GETNAME_NP \
+ (SI_FREEBSD || SI_LINUX_NOT_ANDROID || SI_SOLARIS)
#define SANITIZER_INTERCEPT_TLS_GET_ADDR \
(SI_FREEBSD || SI_NETBSD || SI_LINUX_NOT_ANDROID || SI_SOLARIS)
diff --git a/lib/tsan/rtl/tsan_libdispatch_mac.cc b/lib/tsan/rtl/tsan_libdispatch_mac.cc
index eb22e4baa710..d6c1ca6622f4 100644
--- a/lib/tsan/rtl/tsan_libdispatch_mac.cc
+++ b/lib/tsan/rtl/tsan_libdispatch_mac.cc
@@ -25,6 +25,11 @@
#include <dispatch/dispatch.h>
#include <pthread.h>
+// DISPATCH_NOESCAPE is not defined prior to XCode 8.
+#ifndef DISPATCH_NOESCAPE
+#define DISPATCH_NOESCAPE
+#endif
+
typedef long long_t; // NOLINT
namespace __tsan {
diff --git a/test/msan/pthread_getname_np.cc b/test/msan/pthread_getname_np.cc
new file mode 100644
index 000000000000..ca27d8b6fd6c
--- /dev/null
+++ b/test/msan/pthread_getname_np.cc
@@ -0,0 +1,34 @@
+// RUN: %clangxx_msan -std=c++11 -O0 %s -o %t && %run %t
+// UNSUPPORTED: android, netbsd
+
+// Regression test for a deadlock in pthread_getattr_np
+
+#include <assert.h>
+#include <pthread.h>
+#include <string.h>
+#include <sanitizer/msan_interface.h>
+
+#include <stdio.h>
+
+void *ThreadFn(void *) {
+ return nullptr;
+}
+
+int main(void) {
+ pthread_t t;
+ int res = pthread_create(&t, 0, ThreadFn, 0);
+ assert(!res);
+
+ const char *kMyThreadName = "my-thread-name";
+ res = pthread_setname_np(t, kMyThreadName);
+ assert(!res);
+
+ char buf[100];
+ res = pthread_getname_np(t, buf, sizeof(buf));
+ assert(!res);
+ assert(strcmp(buf, kMyThreadName) == 0);
+
+ res = pthread_join(t, 0);
+ assert(!res);
+ return 0;
+}