[PATCH] D35275: [Sanitizers] ASan/MSan/LSan allocators set errno on failure.

[Matt Morehouse via Phabricator via llvm-commits]

morehouse added inline comments.


================
Comment at: lib/msan/msan_allocator.cc:287
+  if (UNLIKELY(!IsPowerOfTwo(alignment) ||
+               (alignment % sizeof(void *)) != 0)) {  // NOLINT
+    Allocator::FailureHandler::OnBadRequest();
----------------
vitalybuka wrote:
> maybe just, pointer should be always power of two?
> if (UNLIKELY((alignment % sizeof(void *)))) { 
> 
> 
From the posix_memalign man page:

> The function posix_memalign()  allocates  size  bytes  and  places  the
> address  of  the allocated memory in *memptr.  The address of the allo‐
> cated memory will be a multiple of alignment, which must be a power  of
> two  and  a  multiple  of sizeof(void *).




https://reviews.llvm.org/D35275