From: Michael Vrable <mvrable@cs.ucsd.edu>
Date: Fri, 17 Aug 2007 15:08:11 +0000 (-0700)
Subject: Bug fix for the rewritten spawn_filter function.
X-Git-Url: https://git.vrable.net/?a=commitdiff_plain;h=79c9c740d9b78e357dd66cb9a4b77d03a5076f3b;p=cumulus.git

Bug fix for the rewritten spawn_filter function.

We were accidentally using the wrong variable (filter_pid instead of pid)
when looking at the result of a fork call, with the result that both
processes thought they were the parent.
---

diff --git a/store.cc b/store.cc
index caec347..a156a57 100644
--- a/store.cc
+++ b/store.cc
@@ -98,10 +98,10 @@ int spawn_filter(int fd_out, const char *program, pid_t *filter_pid)
 
     /* Create a child process which can exec() the filter program. */
     pid = fork();
-    if (filter_pid < 0)
+    if (pid < 0)
         throw IOException("Unable to fork filter process");
 
-    if (filter_pid > 0) {
+    if (pid > 0) {
         /* Parent process */
         close(fds[0]);
         cloexec(fds[1]);