diff --git a/cmds/incidentd/src/Section.cpp b/cmds/incidentd/src/Section.cpp index 6d5737d3389fd..78876a37f7301 100644 --- a/cmds/incidentd/src/Section.cpp +++ b/cmds/incidentd/src/Section.cpp @@ -864,11 +864,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("[%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();