[PATCH] D12790: ubsan: Implement memory permission validation for vtables.

Alexey Samsonov via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 11 15:20:00 PDT 2015


samsonov added a subscriber: samsonov.
samsonov accepted this revision.
samsonov added a reviewer: samsonov.
samsonov added a comment.

LGTM. Thanks!


================
Comment at: lib/ubsan/ubsan_type_hash_itanium.cc:197
@@ +196,3 @@
+#if SANITIZER_LINUX
+bool isValidVptr(void *Vtable) {
+  // Validate the memory permissions of the vtable pointer and the first
----------------
These two functions should be static


http://reviews.llvm.org/D12790





More information about the llvm-commits mailing list