a62586846f
Summary: Forewarning: This patch looks big in #LOC changed. I promise it's not that bad, it just moves a lot of content from one file to another. I've gone ahead and left inline comments on Phabricator for sections where this has happened. This patch: 1. Introduces the crash handler API (crash_handler_api.h). 2. Moves information required for out-of-process crash handling into an AllocatorState. This is a trivially-copied POD struct that designed to be recovered from a deceased process, and used by the crash handler to create a GWP-ASan report (along with the other trivially-copied Metadata struct). 3. Implements the crash handler API using the AllocatorState and Metadata. 4. Adds tests for the crash handler. 5. Reimplements the (now optionally linked by the supporting allocator) in-process crash handler (i.e. the segv handler) using the new crash handler API. 6. Minor updates Scudo & Scudo Standalone to fix compatibility. 7. Changed capitalisation of errors (e.g. /s/Use after free/Use After Free). Reviewers: cryptoad, eugenis, jfb Reviewed By: eugenis Subscribers: merge_guards_bot, pcc, jfb, dexonsmith, mgorny, cryptoad, #sanitizers, llvm-commits Tags: #sanitizers, #llvm Differential Revision: https://reviews.llvm.org/D73557
21 lines
435 B
C++
21 lines
435 B
C++
// REQUIRES: gwp_asan
|
|
// RUN: %clangxx_gwp_asan %s -o %t
|
|
// RUN: %expect_crash %run %t 2>&1 | FileCheck %s
|
|
|
|
// CHECK: GWP-ASan detected a memory error
|
|
// CHECK: Use After Free at 0x{{[a-f0-9]+}} (0 bytes into a 10-byte allocation
|
|
|
|
#include <cstdlib>
|
|
|
|
int main() {
|
|
char *Ptr = reinterpret_cast<char *>(malloc(10));
|
|
|
|
for (unsigned i = 0; i < 10; ++i) {
|
|
*(Ptr + i) = 0x0;
|
|
}
|
|
|
|
free(Ptr);
|
|
volatile char x = *Ptr;
|
|
return 0;
|
|
}
|