[compiler-rt] r369708 - [Sanitizer] checks ASLR on FreeBSD

David Carlier via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 22 14:36:35 PDT 2019


Author: devnexen
Date: Thu Aug 22 14:36:35 2019
New Revision: 369708

URL: http://llvm.org/viewvc/llvm-project?rev=369708&view=rev
Log:
[Sanitizer] checks ASLR on FreeBSD

- Especially MemorySanitizer fails if those sysctl configs are enabled.

Reviewers: vitalybuka, emaste, dim

Reviewed By: dim

Differential Revision: https://reviews.llvm.org/D66582

Modified:
    compiler-rt/trunk/lib/asan/asan_rtl.cpp
    compiler-rt/trunk/lib/msan/msan.cpp
    compiler-rt/trunk/lib/sanitizer_common/sanitizer_linux.cpp

Modified: compiler-rt/trunk/lib/asan/asan_rtl.cpp
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/asan/asan_rtl.cpp?rev=369708&r1=369707&r2=369708&view=diff
==============================================================================
--- compiler-rt/trunk/lib/asan/asan_rtl.cpp (original)
+++ compiler-rt/trunk/lib/asan/asan_rtl.cpp Thu Aug 22 14:36:35 2019
@@ -402,7 +402,6 @@ static void AsanInitInternal() {
   asan_init_is_running = true;
 
   CacheBinaryName();
-  CheckASLR();
 
   // Initialize flags. This must be done early, because most of the
   // initialization steps look at flags().
@@ -450,6 +449,7 @@ static void AsanInitInternal() {
   SetLowLevelAllocateCallback(OnLowLevelAllocate);
 
   InitializeAsanInterceptors();
+  CheckASLR();
 
   // Enable system log ("adb logcat") on Android.
   // Doing this before interceptors are initialized crashes in:

Modified: compiler-rt/trunk/lib/msan/msan.cpp
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/msan/msan.cpp?rev=369708&r1=369707&r2=369708&view=diff
==============================================================================
--- compiler-rt/trunk/lib/msan/msan.cpp (original)
+++ compiler-rt/trunk/lib/msan/msan.cpp Thu Aug 22 14:36:35 2019
@@ -403,7 +403,6 @@ void __msan_init() {
   AvoidCVE_2016_2143();
 
   CacheBinaryName();
-  CheckASLR();
   InitializeFlags();
 
   // Install tool-specific callbacks in sanitizer_common.
@@ -412,6 +411,7 @@ void __msan_init() {
   __sanitizer_set_report_path(common_flags()->log_path);
 
   InitializeInterceptors();
+  CheckASLR();
   InitTlsSize();
   InstallDeadlySignalHandlers(MsanOnDeadlySignal);
   InstallAtExitHandler(); // Needs __cxa_atexit interceptor.

Modified: compiler-rt/trunk/lib/sanitizer_common/sanitizer_linux.cpp
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_linux.cpp?rev=369708&r1=369707&r2=369708&view=diff
==============================================================================
--- compiler-rt/trunk/lib/sanitizer_common/sanitizer_linux.cpp (original)
+++ compiler-rt/trunk/lib/sanitizer_common/sanitizer_linux.cpp Thu Aug 22 14:36:35 2019
@@ -2011,6 +2011,33 @@ void CheckASLR() {
     CHECK_NE(personality(old_personality | ADDR_NO_RANDOMIZE), -1);
     ReExec();
   }
+#elif SANITIZER_FREEBSD
+  int aslr_pie;
+  uptr len = sizeof(aslr_pie);
+#if SANITIZER_WORDSIZE == 64
+  if (UNLIKELY(internal_sysctlbyname("kern.elf64.aslr.pie_enable",
+      &aslr_pie, &len, NULL, 0) == -1)) {
+    // We're making things less 'dramatic' here since
+    // the OID is not necessarily guaranteed to be here
+    // just yet regarding FreeBSD release
+    return;
+  }
+
+  if (aslr_pie > 0) {
+    Printf("This sanitizer is not compatible with enabled ASLR and binaries compiled with PIE\n");
+    Die();
+  }
+#endif
+  // there might be 32 bits compat for 64 bits
+  if (UNLIKELY(internal_sysctlbyname("kern.elf32.aslr.pie_enable",
+      &aslr_pie, &len, NULL, 0) == -1)) {
+    return;
+  }
+
+  if (aslr_pie > 0) {
+    Printf("This sanitizer is not compatible with enabled ASLR and binaries compiled with PIE\n");
+    Die();
+  }
 #else
   // Do nothing
 #endif




More information about the llvm-commits mailing list