[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