Merge pull request #241 from TestPlanB/master

dump过程提高成功率
KwaiAppTeam · May 3, 2023 · a5430e2 · a5430e2
2 parents f0a2815 + c0acb1f
commit a5430e2
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/koom-fast-dump/src/main/cpp/hprof_dump.cpp b/koom-fast-dump/src/main/cpp/hprof_dump.cpp
@@ -136,14 +136,18 @@ bool HprofDump::ResumeAndWait(pid_t pid) {
   }
   int status;
   for (;;) {
-    if (waitpid(pid, &status, 0) != -1 || errno != EINTR) {
+    if (waitpid(pid, &status, 0) != -1) {
       if (!WIFEXITED(status)) {
         ALOGE("Child process %d exited with status %d, terminated by signal %d",
               pid, WEXITSTATUS(status), WTERMSIG(status));
         return false;
       }
       return true;
     }
+    // if waitpid is interrupted by the signal,just call it again
+    if (errno == EINTR){
+      continue;
+    }
     return false;
   }
 }