commit 9967ac4debf2361529aacb53f35d4e213cf3bec5
author Christopher Ferris <cferris@google.com> Fri Apr 25 01:59:52 2025 +0000
committer repo sync -j5 <cferris@google.com> Wed Apr 30 23:15:18 2025 +0000
tree 3044d28143d03b5905463a09684a22ed0f63f0dc
parent d436c6a1b4d3dfa2fe222747b8d63ce99404bb5c

Remove SEGV_MTEAERR check.

SEGV_MTEARR/SEGV_MTESERR are always defined now, so no need to
check before using them.

Add the signal.h include so that this actually works properly.

Test: Builds/Treehugger
Change-Id: Iec6e7c69cd80a002ea972f32834bdc6701af5063

debuggerd/libdebuggerd/tombstone_proto_to_text.cpp

diff --git a/debuggerd/libdebuggerd/tombstone_proto_to_text.cpp b/debuggerd/libdebuggerd/tombstone_proto_to_text.cpp
index 832db48..ff12660 100644
--- a/debuggerd/libdebuggerd/tombstone_proto_to_text.cpp
+++ b/debuggerd/libdebuggerd/tombstone_proto_to_text.cpp
@@ -19,6 +19,7 @@
 
 #include <ctype.h>
 #include <inttypes.h>
+#include <signal.h>
 
 #include <algorithm>
 #include <functional>
@@ -453,11 +454,9 @@
     CBL("signal %d (%s), code %d (%s%s), fault addr %s", signal_info.number(),
         signal_info.name().c_str(), signal_info.code(), signal_info.code_name().c_str(),
         sender_desc.c_str(), fault_addr_desc.c_str());
-#ifdef SEGV_MTEAERR
     is_async_mte_crash = signal_info.number() == SIGSEGV && signal_info.code() == SEGV_MTEAERR;
     is_mte_crash = is_async_mte_crash ||
                    (signal_info.number() == SIGSEGV && signal_info.code() == SEGV_MTESERR);
-#endif
   }
 
   if (tombstone.causes_size() == 1) {