[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