Merge pull request #59 from nunofernandes/master

Check fork syscall return value. Log it if we got an error
beave · Nov 23, 2015 · 57569d1 · 57569d1
2 parents e290341 + 300a9e0
commit 57569d1
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/output-plugins/sagan-external.c b/src/output-plugins/sagan-external.c
@@ -129,10 +129,11 @@ Syslog Priority:%s\n\
             Sagan_Log(S_ERROR, "[%s, line %d] Cannot create output pipe!", __FILE__, __LINE__);
         }
 
-
-    if (( pid = fork()) == 0 )
+    pid=fork();
+    if ( pid < 0 ) {
+            Sagan_Log(S_ERROR, "[%s, line %d] Cannot create external program process", __FILE__, __LINE__);
+    } else if ( pid == 0 )
         {
-
             /* Causes problems with alert.log */
 
             close(0);