commit 11a8c35c3f78b414d67d77bfcfa0b0bcbb62af03
author Android Build Coastguard Worker <android-build-coastguard-worker@google.com> Sat Jan 28 13:28:18 2023 +0000
committer Android Build Coastguard Worker <android-build-coastguard-worker@google.com> Sat Jan 28 13:28:18 2023 +0000
tree 72c5dffb275e34c8be6537ab0f49ecfe3d40099d
parent 7c732f4607f065d647006385f88c2325675a21e2
parent 67481a676fa8b04fb0ed926f87320db68b38b71e

Snap for 9537280 from 67481a676fa8b04fb0ed926f87320db68b38b71e to mainline-media-release

Change-Id: I64d6c41689a1a65f089c6e8faf3d8a985af72889

debuggerd/Android.bp

diff --git a/debuggerd/Android.bp b/debuggerd/Android.bp
index ad0231d..5198872 100644
--- a/debuggerd/Android.bp
+++ b/debuggerd/Android.bp
@@ -14,9 +14,15 @@
         "-Wno-nullability-completeness",
         "-Os",
         "-fno-finite-loops",
+        "-DANDROID_DEBUGGABLE=0",
     ],
 
     local_include_dirs: ["include"],
+    product_variables: {
+        debuggable: {
+            cflags: ["-UANDROID_DEBUGGABLE", "-DANDROID_DEBUGGABLE=1"],
+        }
+    },
 }
 
 cc_library_headers {

debuggerd/include/debuggerd/handler.h

diff --git a/debuggerd/include/debuggerd/handler.h b/debuggerd/include/debuggerd/handler.h
index bc08327..68b2e67 100644
--- a/debuggerd/include/debuggerd/handler.h
+++ b/debuggerd/include/debuggerd/handler.h
@@ -62,10 +62,11 @@
 #define DEBUGGER_SIGNAL BIONIC_SIGNAL_DEBUGGER
 
 static void __attribute__((__unused__)) debuggerd_register_handlers(struct sigaction* action) {
+  bool enabled = true;
+#if ANDROID_DEBUGGABLE
   char value[PROP_VALUE_MAX] = "";
-  bool enabled =
-      !(__system_property_get("ro.debuggable", value) > 0 && !strcmp(value, "1") &&
-        __system_property_get("debug.debuggerd.disable", value) > 0 && !strcmp(value, "1"));
+  enabled = !(__system_property_get("debug.debuggerd.disable", value) > 0 && !strcmp(value, "1"));
+#endif
   if (enabled) {
     sigaction(SIGABRT, action, nullptr);
     sigaction(SIGBUS, action, nullptr);