Forráskód Böngészése

Unix: Fix retrieval of PID exit code

Haoyu Qiu 2 hete
szülő
commit
d5738b5cdd
1 módosított fájl, 5 hozzáadás és 10 törlés
  1. 5 10
      drivers/unix/os_unix.cpp

+ 5 - 10
drivers/unix/os_unix.cpp

@@ -815,7 +815,7 @@ int OS_Unix::_wait_for_pid_completion(const pid_t p_pid, int *r_status, int p_op
 	while (true) {
 	while (true) {
 		pid_t pid = waitpid(p_pid, r_status, p_options);
 		pid_t pid = waitpid(p_pid, r_status, p_options);
 		if (pid != -1) {
 		if (pid != -1) {
-			// Thread exited normally.
+			// When `p_options` has `WNOHANG`, 0 can be returned if the process is still running.
 			if (r_pid) {
 			if (r_pid) {
 				*r_pid = pid;
 				*r_pid = pid;
 			}
 			}
@@ -845,24 +845,19 @@ bool OS_Unix::_check_pid_is_running(const pid_t p_pid, int *r_status) const {
 	pid_t pid = -1;
 	pid_t pid = -1;
 	int status = 0;
 	int status = 0;
 	const int result = _wait_for_pid_completion(p_pid, &status, WNOHANG, &pid);
 	const int result = _wait_for_pid_completion(p_pid, &status, WNOHANG, &pid);
-	if (result == 0) {
+	if (result == 0 && pid == 0) {
 		// Thread is still running.
 		// Thread is still running.
-		if (pi && pid == p_pid) {
-			pi->exit_code = WIFEXITED(status) ? WEXITSTATUS(status) : status;
-		}
 		return true;
 		return true;
 	}
 	}
 
 
-	ERR_FAIL_COND_V_MSG(result == -1, false, vformat("Thread %d exited with errno: %d", (int)p_pid, errno));
-	// Thread exited normally.
+	ERR_FAIL_COND_V_MSG(result != 0, false, vformat("Thread %d exited with errno: %d", (int)p_pid, errno));
 
 
+	// Thread exited normally.
 	status = WIFEXITED(status) ? WEXITSTATUS(status) : status;
 	status = WIFEXITED(status) ? WEXITSTATUS(status) : status;
 
 
 	if (pi) {
 	if (pi) {
 		pi->is_running = false;
 		pi->is_running = false;
-		if (pid == p_pid) {
-			pi->exit_code = status;
-		}
+		pi->exit_code = status;
 	}
 	}
 
 
 	if (r_status) {
 	if (r_status) {