diff options
author | Josh Gao <jmgao@google.com> | 2016-02-16 15:01:43 -0800 |
---|---|---|
committer | The Android Automerger <android-build@google.com> | 2016-03-01 15:27:11 -0800 |
commit | 213af3ac2a9cac2049e930bef6b63f582901c922 (patch) | |
tree | 9a5b7f25e274de2282148109eee76da51679f998 | |
parent | b84295d027c9d86720d93e19eb5c1b644553d647 (diff) | |
download | system_core-213af3ac2a9cac2049e930bef6b63f582901c922.zip system_core-213af3ac2a9cac2049e930bef6b63f582901c922.tar.gz system_core-213af3ac2a9cac2049e930bef6b63f582901c922.tar.bz2 |
Don't create tombstone directory.
Partial backport of cf79748.
Bug: http://b/26403620
Change-Id: Ib877ab6cfab6aef079830c5a50ba81141ead35ee
-rw-r--r-- | debuggerd/tombstone.cpp | 15 |
1 files changed, 1 insertions, 14 deletions
diff --git a/debuggerd/tombstone.cpp b/debuggerd/tombstone.cpp index b0ad274..aeffc66 100644 --- a/debuggerd/tombstone.cpp +++ b/debuggerd/tombstone.cpp @@ -788,21 +788,8 @@ char* engrave_tombstone(pid_t pid, pid_t tid, int signal, int original_si_code, log.current_tid = tid; log.crashed_tid = tid; - if ((mkdir(TOMBSTONE_DIR, 0755) == -1) && (errno != EEXIST)) { - _LOG(&log, logtype::ERROR, "failed to create %s: %s\n", TOMBSTONE_DIR, strerror(errno)); - } - - if (chown(TOMBSTONE_DIR, AID_SYSTEM, AID_SYSTEM) == -1) { - _LOG(&log, logtype::ERROR, "failed to change ownership of %s: %s\n", TOMBSTONE_DIR, strerror(errno)); - } - int fd = -1; - char* path = NULL; - if (selinux_android_restorecon(TOMBSTONE_DIR, 0) == 0) { - path = find_and_open_tombstone(&fd); - } else { - _LOG(&log, logtype::ERROR, "Failed to restore security context, not writing tombstone.\n"); - } + char* path = find_and_open_tombstone(&fd); if (fd < 0) { _LOG(&log, logtype::ERROR, "Skipping tombstone write, nothing to do.\n"); |