commit 28f8cf0f975500d754e259e19dcf9762f0969060
author Josh Gao <jmgao@google.com> Thu May 03 14:31:08 2018 -0700
committer Josh Gao <jmgao@google.com> Thu May 03 14:31:08 2018 -0700
tree 086b700620066ba2238c85226f3ee55bbf91fbc1
parent abf6a2e1ade1d89b3bb83d8f0b7d77a62a3b31c5

tombstoned: don't bail out if we fail to unlink a file that isn't there.

Test: crasher with no tombstones
Change-Id: I20e0537a347dd1f83877150ab13f53441dd65d95

debuggerd/tombstoned/tombstoned.cpp

diff --git a/debuggerd/tombstoned/tombstoned.cpp b/debuggerd/tombstoned/tombstoned.cpp
index 5dffa5b..2f16eb2 100644
--- a/debuggerd/tombstoned/tombstoned.cpp
+++ b/debuggerd/tombstoned/tombstoned.cpp
@@ -352,7 +352,7 @@
     std::string fd_path = StringPrintf("/proc/self/fd/%d", crash->crash_tombstone_fd.get());
     std::string tombstone_path = CrashQueue::for_crash(crash)->get_next_artifact_path();
     int rc = unlink(tombstone_path.c_str());
-    if (rc != 0) {
+    if (rc != 0 && errno != ENOENT) {
       PLOG(ERROR) << "failed to unlink tombstone at " << tombstone_path;
       goto fail;
     }