[PATCH] D50940: [sanitizer] Change Mmap*NoAccess to return MMAP_FAILED (~(uptr)0) on error

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 20 11:37:22 PDT 2018


cryptoad updated this revision to Diff 161523.
cryptoad added a comment.

Updated proposal: make `Mmap*NoAccess` return `nullptr` on failure (like the
other Mmap functions).
Modify callers that were checking for `~(uptr)0` to now check for `nullptr`.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D50940

Files:
  lib/asan/asan_linux.cc
  lib/asan/asan_premap_shadow.cc
  lib/hwasan/hwasan_dynamic_shadow.cc
  lib/sanitizer_common/sanitizer_allocator_primary64.h
  lib/sanitizer_common/sanitizer_mac_libcdep.cc
  lib/sanitizer_common/sanitizer_posix_libcdep.cc
  lib/scudo/scudo_allocator_secondary.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50940.161523.patch
Type: text/x-patch
Size: 4208 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180820/23ec0a96/attachment.bin>


More information about the llvm-commits mailing list