[launchd-changes] [23214] trunk/launchd/src/launchd_core_logic.c

[source_changes at macosforge.org]

Revision: 23214
          http://trac.macosforge.org/projects/launchd/changeset/23214
Author:   zarzycki at apple.com
Date:     2007-04-11 11:22:27 -0700 (Wed, 11 Apr 2007)

Log Message:
-----------
It is safer to assume that a program crashed.

Modified Paths:
--------------
    trunk/launchd/src/launchd_core_logic.c

Modified: trunk/launchd/src/launchd_core_logic.c
===================================================================
--- trunk/launchd/src/launchd_core_logic.c	2007-04-11 16:29:37 UTC (rev 23213)
+++ trunk/launchd/src/launchd_core_logic.c	2007-04-11 18:22:27 UTC (rev 23214)
@@ -1752,7 +1752,7 @@
 {
 	struct timeval tve, tvd;
 	struct rusage ru;
-	int status = 0;
+	int status;
 
 	job_log(j, LOG_DEBUG, "Reaping");
 
@@ -1778,14 +1778,10 @@
 	}
 
 	if (!j->anonymous && !job_assumes(j, wait4(j->p, &status, 0, &ru) != -1)) {
-		/*
-		 * wait4() then kill() is still racy.
-		 * Then again, we never should have got here in the first place...
-		 */
-		if (kill(j->p, 0) == 0) {
-			job_log(j, LOG_DEBUG, "Working around 5020256");
-		}
+		job_log(j, LOG_NOTICE, "Working around 5020256. Assuming the job crashed.");
 
+		status = W_EXITCODE(0, SIGSEGV);
+
 		memset(&ru, 0, sizeof(ru));
 	}
 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.macosforge.org/pipermail/launchd-changes/attachments/20070411/75c4188c/attachment.html