diff options
| author | Josh Gao <jmgao@google.com> | 2017-01-26 15:03:29 -0800 |
|---|---|---|
| committer | Josh Gao <jmgao@google.com> | 2017-01-26 15:08:18 -0800 |
| commit | e5288f292a2565e276098b5f0e60ee93bb7e10a5 (patch) | |
| tree | 95408035288b241aaf305248c0e212e782d79dcf | |
| parent | fc267f865e1475030f777a3640bd1cf46dc766b0 (diff) | |
| download | core-e5288f292a2565e276098b5f0e60ee93bb7e10a5.tar.gz core-e5288f292a2565e276098b5f0e60ee93bb7e10a5.tar.bz2 core-e5288f292a2565e276098b5f0e60ee93bb7e10a5.zip | |
debuggerd_handler: remove PR_SET_DUMPABLE check.
crash_dump has CAP_SYS_PTRACE and this was never obeyed by debuggerd.
Change-Id: Ifee5e94b97b1f6440ad0be79758f0db2d2aaba2e
| -rw-r--r-- | debuggerd/handler/debuggerd_handler.cpp | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/debuggerd/handler/debuggerd_handler.cpp b/debuggerd/handler/debuggerd_handler.cpp index c009869c5..501d0898c 100644 --- a/debuggerd/handler/debuggerd_handler.cpp +++ b/debuggerd/handler/debuggerd_handler.cpp @@ -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. |