summaryrefslogtreecommitdiffabout
authorMichael Krelin <hacker@klever.net>2008-08-31 11:50:48 (UTC)
committer Michael Krelin <hacker@klever.net>2008-08-31 11:50:48 (UTC)
commit6a68fcae465905e904eea248ffe6b1ed6b8297a4 (patch) (side-by-side diff)
tree5f070058ca6cf9948ae9af229694d3d746d2f1c5
parent26757537328da204e4bf952a80579307440f62c9 (diff)
downloaddudki-6a68fcae465905e904eea248ffe6b1ed6b8297a4.zip
dudki-6a68fcae465905e904eea248ffe6b1ed6b8297a4.tar.gz
dudki-6a68fcae465905e904eea248ffe6b1ed6b8297a4.tar.bz2
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>
Diffstat (more/less context) (ignore whitespace changes)
-rw-r--r--src/process.cc7
1 files changed, 6 insertions, 1 deletions
diff --git a/src/process.cc b/src/process.cc
index 6d3b2a2..4807b98 100644
--- a/src/process.cc
+++ b/src/process.cc
@@ -171,4 +171,10 @@ void process::notify_mailto(const string& email,const string& id,const string& e
}
// 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");
@@ -189,5 +195,4 @@ void process::notify_mailto(const string& email,const string& id,const string& e
description.c_str() );
fclose(mta);
- int status;
waitpid(pid,&status,0);
// TODO: check the return code