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

Aleksey Shlyapnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 25 08:40:51 PDT 2017


alekseyshl added a comment.

Let's have a cl per sanitizer.



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


https://reviews.llvm.org/D35818





More information about the llvm-commits mailing list