[all-commits] [llvm/llvm-project] e1d84c: [compiler-rt][sanitizer] Have all OOM-related erro...

PiJoules via All-commits all-commits at lists.llvm.org
Tue Jun 7 16:46:30 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e1d84c421df1bd496918bc4dd30f040d47906a77
      https://github.com/llvm/llvm-project/commit/e1d84c421df1bd496918bc4dd30f040d47906a77
  Author: Leonard Chan <leonardchan at google.com>
  Date:   2022-06-07 (Tue, 07 Jun 2022)

  Changed paths:
    M compiler-rt/lib/asan/asan_errors.cpp
    M compiler-rt/lib/sanitizer_common/sanitizer_allocator_report.cpp
    M compiler-rt/lib/sanitizer_common/sanitizer_common.cpp
    M compiler-rt/lib/sanitizer_common/sanitizer_common.h
    M compiler-rt/lib/sanitizer_common/sanitizer_fuchsia.cpp
    M compiler-rt/lib/sanitizer_common/sanitizer_posix.cpp
    M compiler-rt/lib/sanitizer_common/sanitizer_win.cpp
    M compiler-rt/test/hwasan/TestCases/sizes.cpp
    M compiler-rt/test/lsan/TestCases/realloc_too_big.c

  Log Message:
  -----------
  [compiler-rt][sanitizer] Have all OOM-related error messages start with the same format

This way downstream tools that read sanitizer output can differentiate between OOM errors
reported by sanitizers from other sanitizer errors.

Changes:

- Introduce ErrorIsOOM for checking if a platform-specific error code from an "mmap" is an OOM err.
- Add ReportOOMError which just prepends this error message to the start of a Report call.
- Replace some Reports for OOMs with calls to ReportOOMError.
- Update necessary tests.

Differential Revision: https://reviews.llvm.org/D127161




More information about the All-commits mailing list