commit 2495d6540fe0c7072438605c77efe8b66948a8d8
author Mitch Phillips <31459023+hctim@users.noreply.github.com> Fri Jul 17 12:59:09 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Aug 27 23:58:44 2020 -0700
tree ef21fbc5aadca555016f82bc63df9d668f2de265
parent 59b98c7062c3c58ee16cc08246aa2999f7de7eb8

[GWP-ASan] Split the unwinder into segv/non-segv.

Summary:
Splits the unwinder into a non-segv (for allocation/deallocation traces) and a
segv unwinder. This ensures that implementations can select an accurate, slower
unwinder in the segv handler (if they choose to use the GWP-ASan provided one).
This is important as fast frame-pointer unwinders (like the sanitizer unwinder)
don't like unwinding through signal handlers.

Reviewers: morehouse, cryptoad

Reviewed By: morehouse, cryptoad

Subscribers: cryptoad, mgorny, eugenis, pcc, #sanitizers

Tags: #sanitizers

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

GitOrigin-RevId: 502f0cc0e3889229e923e187f38dda91324ae139

combined.h

diff --git a/combined.h b/combined.h
index 3bb41ec..ae085be 100644
--- a/combined.h
+++ b/combined.h
@@ -192,7 +192,7 @@
     if (Opt.InstallSignalHandlers)
       gwp_asan::crash_handler::installSignalHandlers(
           &GuardedAlloc, Printf, gwp_asan::options::getPrintBacktraceFunction(),
-          Opt.Backtrace);
+          gwp_asan::crash_handler::getSegvBacktraceFunction());
 #endif // GWP_ASAN_HOOKS
   }