[llvm-branch-commits] [compiler-rt] eceac8e - Revert "Revert "[sanitizer] Add CHECKs to validate calculated TLS range" (#10…"

[via llvm-branch-commits]

Author: Vitaly Buka
Date: 2024-09-10T17:46:05-07:00
New Revision: eceac8e53357b8d91f047f7a5fe954d987e05e53

URL: https://github.com/llvm/llvm-project/commit/eceac8e53357b8d91f047f7a5fe954d987e05e53
DIFF: https://github.com/llvm/llvm-project/commit/eceac8e53357b8d91f047f7a5fe954d987e05e53.diff

LOG: Revert "Revert "[sanitizer] Add CHECKs to validate calculated TLS range" (#10…"

This reverts commit 5804193e38680683b370cb3ced46c018d4dbd1b2.

Added: 
    

Modified: 
    compiler-rt/lib/sanitizer_common/sanitizer_tls_get_addr.cpp

Removed: 
    


################################################################################
diff  --git a/compiler-rt/lib/sanitizer_common/sanitizer_tls_get_addr.cpp b/compiler-rt/lib/sanitizer_common/sanitizer_tls_get_addr.cpp
index 666e6f3b351067..a1107ff7d24737 100644
--- a/compiler-rt/lib/sanitizer_common/sanitizer_tls_get_addr.cpp
+++ b/compiler-rt/lib/sanitizer_common/sanitizer_tls_get_addr.cpp
@@ -151,6 +151,10 @@ DTLS::DTV *DTLS_on_tls_get_addr(void *arg_void, void *res,
     // This may happen inside the DTOR of main thread, so just ignore it.
     tls_size = 0;
   }
+  if (tls_size) {
+    CHECK_LE(tls_beg, reinterpret_cast<uptr>(res));
+    CHECK_LT(reinterpret_cast<uptr>(res), tls_beg + tls_size);
+  }
   dtv->beg = tls_beg;
   dtv->size = tls_size;
   return dtv;