diff options
| author | Josh Gao <jmgao@google.com> | 2017-02-16 19:17:28 -0800 |
|---|---|---|
| committer | Josh Gao <jmgao@google.com> | 2017-02-16 19:26:09 -0800 |
| commit | 2a18b822d50f360b83e9ce3fd822d2650ed3a4db (patch) | |
| tree | df1f9c23f7425a836ab8095859e89638d6be2321 /debuggerd/handler/debuggerd_handler.cpp | |
| parent | 325eeebc47d9f8ac07c5e390f46c8f58a1621180 (diff) | |
crash_dump: remove unneeded/faulty checks.
We already check our /proc/`getppid()` fd every time we attach a thread, so
these were unneeded at best. The one that happened after dropping
capabilities was actively wrong, though, because /proc/pid access
checks happen on every operation. (only on some kernels?)
Also, add a check that getppid() doesn't change after opening
/proc/getppid().
Bug: http://b/35241370
Test: debuggerd -b `pidof com.android.bluetooth`
Change-Id: I807439d8c2afd027f3c382face50167a8a7946c4
Diffstat (limited to 'debuggerd/handler/debuggerd_handler.cpp')
0 files changed, 0 insertions, 0 deletions