Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e5288f29 authored by Josh Gao's avatar Josh Gao
Browse files

debuggerd_handler: remove PR_SET_DUMPABLE check. 

crash_dump has CAP_SYS_PTRACE and this was never obeyed by debuggerd.

Change-Id: Ifee5e94b97b1f6440ad0be79758f0db2d2aaba2e
parent fc267f86

debuggerd/handler/debuggerd_handler.cpp

+0 −7
Original line number Diff line number Diff line
@@ -297,13 +297,6 @@ static void debuggerd_signal_handler(int signal_number, siginfo_t* info, void*) 
  }



  log_signal_summary(signal_number, info);

  if (prctl(PR_GET_DUMPABLE, 0, 0, 0, 0) == 0) {

    // 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.