diff --git a/cmds/incidentd/src/Section.cpp b/cmds/incidentd/src/Section.cpp index 93875cde4e24e..0e72fc0768fd9 100644 --- a/cmds/incidentd/src/Section.cpp +++ b/cmds/incidentd/src/Section.cpp @@ -902,11 +902,16 @@ status_t TombstoneSection::BlockingCall(int pipeWriteFd) const { // Read from the pipe concurrently to avoid blocking the child. FdBuffer buffer; err = buffer.readFully(dumpPipe.readFd().get()); + // Wait on the child to avoid it becoming a zombie process. + status_t cStatus = wait_child(child); if (err != NO_ERROR) { ALOGW("TombstoneSection '%s' failed to read stack dump: %d", this->name.string(), err); dumpPipe.readFd().reset(); break; } + if (cStatus != NO_ERROR) { + ALOGE("TombstoneSection '%s' child had an issue: %s\n", this->name.string(), strerror(-cStatus)); + } auto dump = std::make_unique(buffer.size()); auto iterator = buffer.data();