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

Phabricator reviews at reviews.llvm.org
Tue Jul 21 07:23:50 PDT 2015


This revision was automatically updated to reflect the committed changes.
Closed by commit rL242787: [asan] Fix the freopen interceptor to allow NULL instead of a filename (authored by kuba.brecka).

Changed prior to commit:
  http://reviews.llvm.org/D11389?vs=30253&id=30255#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D11389

Files:
  compiler-rt/trunk/lib/sanitizer_common/sanitizer_common_interceptors.inc
  compiler-rt/trunk/test/asan/TestCases/Posix/freopen.cc

Index: compiler-rt/trunk/test/asan/TestCases/Posix/freopen.cc
===================================================================
--- compiler-rt/trunk/test/asan/TestCases/Posix/freopen.cc
+++ compiler-rt/trunk/test/asan/TestCases/Posix/freopen.cc
@@ -0,0 +1,12 @@
+// RUN: %clangxx_asan -O0 %s -o %t && %run %t
+
+#include <assert.h>
+#include <stdio.h>
+
+int main() {
+  FILE *fp = fopen("/dev/null", "w");
+  assert(fp);
+  freopen(NULL, "a", fp);
+  fclose(fp);
+  return 0;
+}
Index: compiler-rt/trunk/lib/sanitizer_common/sanitizer_common_interceptors.inc
===================================================================
--- compiler-rt/trunk/lib/sanitizer_common/sanitizer_common_interceptors.inc
+++ compiler-rt/trunk/lib/sanitizer_common/sanitizer_common_interceptors.inc
@@ -4671,7 +4671,7 @@
             __sanitizer_FILE *fp) {
   void *ctx;
   COMMON_INTERCEPTOR_ENTER(ctx, freopen, path, mode, fp);
-  COMMON_INTERCEPTOR_READ_RANGE(ctx, path, REAL(strlen)(path) + 1);
+  if (path) COMMON_INTERCEPTOR_READ_RANGE(ctx, path, REAL(strlen)(path) + 1);
   COMMON_INTERCEPTOR_READ_RANGE(ctx, mode, REAL(strlen)(mode) + 1);
   COMMON_INTERCEPTOR_FILE_CLOSE(ctx, fp);
   __sanitizer_FILE *res = REAL(freopen)(path, mode, fp);
@@ -4702,7 +4702,7 @@
             __sanitizer_FILE *fp) {
   void *ctx;
   COMMON_INTERCEPTOR_ENTER(ctx, freopen64, path, mode, fp);
-  COMMON_INTERCEPTOR_READ_RANGE(ctx, path, REAL(strlen)(path) + 1);
+  if (path) COMMON_INTERCEPTOR_READ_RANGE(ctx, path, REAL(strlen)(path) + 1);
   COMMON_INTERCEPTOR_READ_RANGE(ctx, mode, REAL(strlen)(mode) + 1);
   COMMON_INTERCEPTOR_FILE_CLOSE(ctx, fp);
   __sanitizer_FILE *res = REAL(freopen64)(path, mode, fp);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11389.30255.patch
Type: text/x-patch
Size: 1689 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150721/28cf12b4/attachment.bin>


More information about the llvm-commits mailing list