[PATCH] D35818: [scudo] Check for pvalloc overflow

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 24 22:34:31 PDT 2017


cryptoad added a comment.

Do you prefer the other sanitizers in the same CL or a different one?



================
Comment at: test/scudo/memalign.cpp:93
+    // Size passed to pvalloc overflows when rounded up.
+    p = pvalloc((size_t)-1);
+    assert(!p);
----------------
alekseyshl wrote:
> Spaces around -
Just making sure: do you want spaces around the unary minus? I feel it's usually not the case.


https://reviews.llvm.org/D35818





More information about the llvm-commits mailing list