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

[Peter Collingbourne via llvm-commits]

pcc added inline comments.

================
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
----------------
samsonov wrote:
> These two functions should be static
They are both in an anonymous namespace, so they don't need to be.


Repository:
  rL LLVM

http://reviews.llvm.org/D12790