[PATCH] D54779: [Sanitizer] Adding setvbuf in supported platforms and other stream buffer functions

Kamil Rytarowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 20 20:00:31 PST 2018


krytarowski added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_common_interceptors.inc:7319
+  if (buf) {
+    COMMON_INTERCEPTOR_WRITE_RANGE(ctx, buf, BUFSIZ);
+    unpoison_file(stream);
----------------
Please store it as __sanitizer_bufsiz and read its value in platform code.

The value is specific to libc implementation (it can be changed too) and shall not be hardcoded.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D54779





More information about the llvm-commits mailing list