[PATCH] D11389: [asan] Fix the freopen interceptor to allow NULL instead of a filename

Alexander Potapenko glider at google.com
Tue Jul 21 07:16:58 PDT 2015

glider accepted this revision.
glider added a comment.
This revision is now accepted and ready to land.

The case of path==NULL is not covered in the Ubuntu manpage for freopen, but this behavior is mandated by POSIX and is in fact implemented in Glibc.
Therefore LGTM

Comment at: test/asan/TestCases/Posix/freopen.cc:10
@@ +9,3 @@
+  freopen(NULL, "a", fp);
+  fclose(fp);
Please add the return statement.


More information about the llvm-commits mailing list