diff options
author | Ryan Prichard <rprichard@google.com> | 2021-06-09 20:59:13 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2021-06-09 20:59:13 +0000 |
commit | 6b456c631754aeeeaaf433818a14ce98960b4aae (patch) | |
tree | e6c7a1503de028c6373fce0790cef398ac7b5b3f | |
parent | a33ab3c5c61f0f5b9c541ffef4bc24a45c034a31 (diff) | |
parent | 1c0c59d81658530697d0a4dee184cf5bb2eed502 (diff) | |
download | libcxxabi-6b456c631754aeeeaaf433818a14ce98960b4aae.tar.gz |
cxa_guard: avoid sys/syscall.h and SYS_gettid am: 45ad00158c am: f73a312a36 am: 1c0c59d816
Original change: https://android-review.googlesource.com/c/platform/external/libcxxabi/+/1729852
Change-Id: I1129d2d967968a63b5364280aaf5abee1307029a
-rw-r--r-- | src/cxa_guard_impl.h | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/cxa_guard_impl.h b/src/cxa_guard_impl.h index 58c5dc5..552c454 100644 --- a/src/cxa_guard_impl.h +++ b/src/cxa_guard_impl.h @@ -41,7 +41,10 @@ #include "include/atomic_support.h" #include <unistd.h> #include <sys/types.h> -#if defined(__has_include) +// Android Trusty: sys/syscall.h tries to include bits/syscall.h, which is +// missing. Trusty seems to define _LIBCXXABI_HAS_NO_THREADS, and gettid isn't +// needed in that case, so skip sys/syscall.h. +#if defined(__has_include) && !defined(_LIBCXXABI_HAS_NO_THREADS) # if __has_include(<sys/syscall.h>) # include <sys/syscall.h> # endif @@ -113,7 +116,10 @@ uint32_t PlatformThreadID() { return static_cast<uint32_t>( pthread_mach_thread_np(std::__libcpp_thread_get_current_id())); } -#elif defined(SYS_gettid) && defined(_LIBCPP_HAS_THREAD_API_PTHREAD) +#elif defined(SYS_gettid) && defined(_LIBCPP_HAS_THREAD_API_PTHREAD) && \ + !defined(__BIONIC__) +// Bionic: Disable the SYS_gettid feature for now. Some processes on Android +// block SYS_gettid using seccomp. uint32_t PlatformThreadID() { static_assert(sizeof(pid_t) == sizeof(uint32_t), ""); return static_cast<uint32_t>(syscall(SYS_gettid)); |