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

[Kamil Rytarowski via Phabricator via llvm-commits]

krytarowski added inline comments.


================
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:
> vitalybuka wrote:
> > Do we need  "unpoison_file(stream);"?
> "char *str ="
It looks like we don't need `unpoison_file(stream);`

Right now we don't sanitize `FILE` on NetBSD anyway. Once I will sanitize this struct I might revisit it.


Repository:
  rL LLVM

https://reviews.llvm.org/D43482