Check return value of a few write calls and retry on EINTR, and fix a few other warnings, mostly by printing error messages before giving up.

darcs-hash:20090222202852-ac50b-b0e79142af5b7a99e55271d4001fa252d9684a1d.gz

1 files changed, 8 insertions, 2 deletions

diff --git a/proc.c b/proc.c
index 0b9c4bae..35dff1d4 100644
--- a/proc.c
+++ b/proc.c
@@ -356,6 +356,8 @@ static void mark_process_status( job_t *j,
 	}
 	else
 	{
+		ssize_t ignore;
+		
 		/* This should never be reached */
 		p->completed = 1;
 
@@ -364,8 +366,12 @@ static void mark_process_status( job_t *j,
 				  MESS_SIZE,
 				  "Process %d exited abnormally\n",
 				  (int) p->pid );
-		
-		write( 2, mess, strlen(mess) );
+		/*
+		  If write fails, do nothing. We're in a signal handlers error
+		  handler. If things aren't working properly, it's safer to
+		  give up.
+		 */
+		ignore = write( 2, mess, strlen(mess) );
 	}
 }