[PATCH] Call __asan_free_hook() before marking the chunk quarantinned

Alexey Samsonov samsonov at google.com
Tue Jun 4 04:50:39 PDT 2013


  Done


================
Comment at: lib/asan/asan_allocator2.cc:497
@@ -502,1 +496,3 @@
   AsanChunk *m = reinterpret_cast<AsanChunk *>(chunk_beg);
+  // FIXME: if the free hook produces an ASan report (e.g. due to a bug),
+  // printing the report may crash as the AsanChunk free-related fields have
----------------
Timur Iskhodzhanov wrote:
> This comment should be removed.
Done

================
Comment at: lib/asan/asan_allocator2.cc:522
@@ -527,1 +521,3 @@
+    // If realloc() races with free(), we may start copying freed memory.
+    // However, we will return racy double-free later anyway.
     REAL(memcpy)(new_ptr, old_ptr, memcpy_size);
----------------
Timur Iskhodzhanov wrote:
> s/return/report/
Done


http://llvm-reviews.chandlerc.com/D913

BRANCH
  svn

ARCANIST PROJECT
  compiler-rt



More information about the llvm-commits mailing list