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

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 16 17:11:57 PST 2018


vitalybuka added a comment.

Oh, maybe test?



================
Comment at: lib/sanitizer_common/sanitizer_common_interceptors.inc:6467
+#if SANITIZER_INTERCEPT_ACCT
+INTERCEPTOR(int, acct, char *file) {
+  void *ctx;
----------------
krytarowski wrote:
> 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.
function is declared with const


Repository:
  rL LLVM

https://reviews.llvm.org/D42066





More information about the llvm-commits mailing list