[PATCH] D42066: Add new interceptor: acct(2)

Kamil Rytarowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 16 17:05:46 PST 2018


krytarowski added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_common_interceptors.inc:6467
+#if SANITIZER_INTERCEPT_ACCT
+INTERCEPTOR(int, acct, char *file) {
+  void *ctx;
----------------
vitalybuka wrote:
> "const char *"?
I try to omit const in order to prevent "const pollution" phenomena. It might be fine for so short interceptors thought.

This is my motivation and decision is up to you.


Repository:
  rL LLVM

https://reviews.llvm.org/D42066





More information about the llvm-commits mailing list