[PATCH] D66582: [Sanitizer] checks ASLR on FreeBSD

Dimitry Andric via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 22 10:50:44 PDT 2019


dim accepted this revision.
dim added a comment.
This revision is now accepted and ready to land.

LGTM. Maybe nice to merge it to 9.0 after a day or two.



================
Comment at: compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp:2027
+  if (aslr_pie > 0) {
+    Printf("This sanitizer is not compatible with enabled ASLR\n");
+    Die();
----------------
Note that the `aslr.pie_enable` sysctl is specifically for PIE executables, so maybe mention that? (This is just a nit.)


Repository:
  rCRT Compiler Runtime

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66582/new/

https://reviews.llvm.org/D66582





More information about the llvm-commits mailing list