Blame SOURCES/mod_revocator-waitpid.patch

3515f1
diff -rupN mod_revocator-1.0.3.orig/exec-client.cpp mod_revocator-1.0.3/exec-client.cpp
3515f1
--- mod_revocator-1.0.3.orig/exec-client.cpp	2006-10-16 11:16:35.000000000 -0700
3515f1
+++ mod_revocator-1.0.3/exec-client.cpp	2012-10-08 13:36:11.093881000 -0700
3515f1
@@ -36,6 +36,8 @@
3515f1
 #ifdef XP_UNIX
3515f1
 #include <sys/types.h>
3515f1
 #include <unistd.h>
3515f1
+#include <sys/wait.h>
3515f1
+#include <syslog.h>
3515f1
 #endif
3515f1
 
3515f1
 #include "client.h"
3515f1
@@ -50,6 +52,7 @@ PR_IMPLEMENT(void *)exec_client(const ch
3515f1
     char * separator;
3515f1
     int pfildes[2];
3515f1
     int pid;
3515f1
+    int status;
3515f1
     char * tmpurl = NULL;
3515f1
     int ldap = 0;
3515f1
     char * arg;
3515f1
@@ -105,12 +108,10 @@ PR_IMPLEMENT(void *)exec_client(const ch
3515f1
         int done = 0;
3515f1
 
3515f1
         close(pfildes[1]);
3515f1
-        dup2(pfildes[0],0);
3515f1
-        close(pfildes[0]);
3515f1
 
3515f1
         totalread = 0;
3515f1
         do {
3515f1
-            numbytes = read(0, buffer, (BIG_LINE * 4) - 1);
3515f1
+            numbytes = read(pfildes[0], buffer, (BIG_LINE * 4) - 1);
3515f1
             if (numbytes > 0) {
3515f1
                 buffer[numbytes] = '\0';
3515f1
                 totalread += numbytes;
3515f1
@@ -125,6 +126,8 @@ PR_IMPLEMENT(void *)exec_client(const ch
3515f1
                         }
3515f1
                         *errnum = CL_OUT_OF_MEMORY;
3515f1
                         free(tmpurl);
3515f1
+                        close(pfildes[0]);
3515f1
+                        waitpid(pid, &status, WNOHANG);
3515f1
                         return NULL;
3515f1
                     }
3515f1
                     sz += (BIG_LINE * 4);
3515f1
@@ -135,6 +138,8 @@ PR_IMPLEMENT(void *)exec_client(const ch
3515f1
                 if (data) free(data);
3515f1
                 data = 0;
3515f1
                 free(tmpurl);
3515f1
+                close(pfildes[0]);
3515f1
+                waitpid(pid, &status, WNOHANG);
3515f1
                 return NULL;
3515f1
             } else if (numbytes == 0) {
3515f1
                 done = 1;
3515f1
@@ -148,6 +153,8 @@ PR_IMPLEMENT(void *)exec_client(const ch
3515f1
         }
3515f1
         free(tmpurl);
3515f1
         *len = totalread;
3515f1
+        close(pfildes[0]);
3515f1
+        waitpid(pid, &status, 0);
3515f1
         return data;
3515f1
     }
3515f1
 }