[PATCH] Fix the region overflow condition in SanitizerAllocator64::PopulateFreeList().

Kostya Serebryany kcc at google.com
Thu May 16 04:52:16 PDT 2013



================
Comment at: lib/sanitizer_common/tests/sanitizer_allocator_test.cc:725
@@ +724,3 @@
+  ASSERT_GT(3 * kAllocationSize, kRegionSize);
+  ASSERT_DEATH(cache.Allocate(a, kClassID) && cache.Allocate(a, kClassID) &&
+                   cache.Allocate(a, kClassID),
----------------
you should move the first two calls out of the ASSERT_DEATH
Also, why not EXPECT_DEATH? 


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



More information about the llvm-commits mailing list