author | Michael Krelin <hacker@klever.net> | 2008-08-31 11:50:48 (UTC) |
---|---|---|
committer | Michael Krelin <hacker@klever.net> | 2008-08-31 11:50:48 (UTC) |
commit | 6a68fcae465905e904eea248ffe6b1ed6b8297a4 (patch) (side-by-side diff) | |
tree | 5f070058ca6cf9948ae9af229694d3d746d2f1c5 | |
parent | 26757537328da204e4bf952a80579307440f62c9 (diff) | |
download | dudki-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>
-rw-r--r-- | src/process.cc | 7 |
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 @@ -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 } |