From 6a68fcae465905e904eea248ffe6b1ed6b8297a4 Mon Sep 17 00:00:00 2001
From: Michael Krelin <hacker@klever.net>
Date: Sun, 31 Aug 2008 11:50:48 +0000
Subject: don't break in the absence of sendmail

If for whatever reason notify child isn't properly spawned, don't die and still
try to launch process in question

Signed-off-by: Michael Krelin <hacker@klever.net>
---
diff --git a/src/process.cc b/src/process.cc
index 6d3b2a2..4807b98 100644
--- a/src/process.cc
+++ b/src/process.cc
@@ -170,6 +170,12 @@ void process::notify_mailto(const string& email,const string& id,const string& e
 	_exit(-1);
     }
     // parent
+    int status;
+    if(waitpid(pid,&status,WNOHANG)) {
+	close(files[0]);
+	close(files[1]);
+	throw runtime_error("vfork()ed sendmail child exited unexpectedly");
+    }
     close(files[0]);
     FILE *mta = fdopen(files[1],"w");
     for(headers_t::const_iterator i=mailto_headers.begin();i!=mailto_headers.end();++i) {
@@ -188,7 +194,6 @@ void process::notify_mailto(const string& email,const string& id,const string& e
 	    id.c_str(), event.c_str(),
 	    description.c_str() );
     fclose(mta);
-    int status;
     waitpid(pid,&status,0);
     // TODO: check the return code
 }
--
cgit v0.9.0.2