commit 196b82f81abd882d1cdcb92123b59bae078fe596
author Josh Gao <jmgao@google.com> Wed Jan 25 20:08:51 2017 +0000
committer android-build-merger <android-build-merger@google.com> Wed Jan 25 20:08:51 2017 +0000
tree a24f7811d43d21d5bf70151b59edfcd8e1167c88
parent e1d76012ff58c157bd33f0697d546d4d328fbdd7
parent 63e77221092387fac3eb46495088f960275b9e4e

debuggerd_handler: don't dump PR_NO_NEW_PRIVS processes. am: 7e14d020f1
am: 44212f19a5

Change-Id: If49cce292c620b9dca8e89b22544f594960c1979

debuggerd/handler/debuggerd_handler.cpp

diff --git a/debuggerd/handler/debuggerd_handler.cpp b/debuggerd/handler/debuggerd_handler.cpp
index 1af0bd1..c009869 100644
--- a/debuggerd/handler/debuggerd_handler.cpp
+++ b/debuggerd/handler/debuggerd_handler.cpp
@@ -301,7 +301,14 @@
     // The process has disabled core dumps and PTRACE_ATTACH, and does not want to be dumped.
     __libc_format_log(ANDROID_LOG_INFO, "libc",
                       "Suppressing debuggerd output because prctl(PR_GET_DUMPABLE)==0");
+    resend_signal(info, false);
+    return;
+  }
 
+  if (prctl(PR_GET_NO_NEW_PRIVS, 0, 0, 0, 0) == 1) {
+    // The process has NO_NEW_PRIVS enabled, so we can't transition to the crash_dump context.
+    __libc_format_log(ANDROID_LOG_INFO, "libc",
+                      "Suppressing debuggerd output because prctl(PR_GET_NO_NEW_PRIVS)==1");
     resend_signal(info, false);
     return;
   }