[PATCH] [sanitizer] Syscall hooks

Alexander Potapenko glider at google.com
Thu Apr 11 07:25:39 PDT 2013



================
Comment at: include/sanitizer/linux_syscall_hooks.h:1
@@ +1,2 @@
+//===-- linux_syscalls.h ---------------------------------------------------===//
+//
----------------
Please fix the file name in the comments as well.

================
Comment at: include/sanitizer/linux_syscall_hooks.h:275
@@ +274,2 @@
+
+#endif // SANITIZER_LINUX_SYSCALLS_H
----------------
And the sentry.

================
Comment at: lib/msan/lit_tests/Linux/syscalls.cc:10
@@ +9,3 @@
+
+#include <sanitizer/linux_syscalls.h>
+#include <sanitizer/msan_interface.h>
----------------
The include filename is incorrect. Does the test still work?


http://llvm-reviews.chandlerc.com/D655



More information about the llvm-commits mailing list