[PATCH] D43482: Add new interceptor: fgetln(3)

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 19 23:27:26 PST 2018


vitalybuka accepted this revision.
vitalybuka added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/sanitizer_common/sanitizer_common_interceptors.inc:6830
+  COMMON_INTERCEPTOR_ENTER(ctx, fgetln, stream, len);
+  str = REAL(fgetln)(stream, len);
+  if (str && len) {
----------------
Do we need  "unpoison_file(stream);"?


================
Comment at: lib/sanitizer_common/sanitizer_common_interceptors.inc:6830
+  COMMON_INTERCEPTOR_ENTER(ctx, fgetln, stream, len);
+  str = REAL(fgetln)(stream, len);
+  if (str && len) {
----------------
vitalybuka wrote:
> Do we need  "unpoison_file(stream);"?
"char *str ="


Repository:
  rL LLVM

https://reviews.llvm.org/D43482





More information about the llvm-commits mailing list