only selectively kill child, not entire process group

Author: Daniel Kroening

src/util/run.cpp

@@ -181,17 +181,23 @@ int run(
     }
     else /* fork() returns new pid to the parent process */
     {
+      // must do before resuming signals to avoid race
+      register_child(childpid);
+
       // resume signals
       sigprocmask(SIG_SETMASK, &old_mask, nullptr);
 
       int status;     /* parent process: child's exit status */
 
       /* wait for child to exit, and store its status */
       while(waitpid(childpid, &status, 0)==-1)
+      {
         if(errno==EINTR)
           continue; // try again
         else
         {
+          unregister_child();
+
           perror("Waiting for child process failed");
           if(stdin_fd!=STDIN_FILENO)
             close(stdin_fd);
@@ -201,6 +207,9 @@ int run(
             close(stderr_fd);
           return 1;
         }
+      }
+
+      unregister_child();
 
       if(stdin_fd!=STDIN_FILENO)
         close(stdin_fd);

src/util/signal_catcher.cpp

@@ -11,21 +11,27 @@ Author: Daniel Kroening, [email protected]
 #include "signal_catcher.h"
 
 #if defined(_WIN32)
-#include <process.h>
 #else
 #include <cstdlib>
-#include <csignal>
 #endif
 
-#include <vector>
-
 // Here we have an instance of an ugly global object.
 // It keeps track of any child processes that we'll kill
-// when we are told to terminate.
+// when we are told to terminate. "No child" is indicated by '0'.
 
 #ifdef _WIN32
 #else
-std::vector<pid_t> pids_of_children;
+pid_t child_pid = 0;
+
+void register_child(pid_t pid)
+{
+  child_pid = pid;
+}
+
+void unregister_child()
+{
+  child_pid = 0;
+}
 #endif
 
 void install_signal_catcher()
@@ -66,13 +72,13 @@ void signal_catcher(int sig)
   #if defined(_WIN32)
   #else
 
-  #if 1
+  #if 0
   // kill any children by killing group
   killpg(0, sig);
   #else
-  // pass on to any children
-  for(const auto &pid : pids_of_children)
-    kill(pid, sig);
+  // pass on to our child, if any
+  if(child_pid != 0)
+    kill(child_pid, sig);
   #endif
 
   exit(sig); // should contemplate something from sysexits.h

src/util/signal_catcher.h

@@ -14,4 +14,10 @@ void install_signal_catcher();
 void remove_signal_catcher();
 void signal_catcher(int sig);
 
+#ifndef _WIN32
+#include <csignal>
+void register_child(pid_t);
+void unregister_child();
+#endif
+
 #endif // CPROVER_UTIL_SIGNAL_CATCHER_H