commit df841da2f78ef5dca876a2fde67e1e39f4e1ca24
author Mitch Phillips <31459023+hctim@users.noreply.github.com> Mon Jul 27 05:46:43 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Sep 30 11:43:08 2020 -0700
tree 324841a4e185fe7c4c7a8fe52af673005778356e
parent 32dc40fd1771f7f8cea82d6ef68d98e1919405aa

[GWP-ASan] Fix uninitialized memory use in sigaction.

Fix up a small bug where we used a partially-uninitialized sigaction
struct in the optional signal handler. Shouldn't be a user-visible
change.

GitOrigin-RevId: 005eee8b3ef7f244daf12f574a191079bfc1918b

optional/segv_handler_posix.cpp

diff --git a/optional/segv_handler_posix.cpp b/optional/segv_handler_posix.cpp
index 1bd7a60..9a80436 100644
--- a/optional/segv_handler_posix.cpp
+++ b/optional/segv_handler_posix.cpp
@@ -144,7 +144,7 @@
   PrintBacktraceForSignalHandler = PrintBacktrace;
   BacktraceForSignalHandler = SegvBacktrace;
 
-  struct sigaction Action;
+  struct sigaction Action = {};
   Action.sa_sigaction = sigSegvHandler;
   Action.sa_flags = SA_SIGINFO;
   sigaction(SIGSEGV, &Action, &PreviousHandler);