Fix deadlock in process exit handling

runtime/bin/process_macos.cc

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include <errno.h>
 #include <fcntl.h>
 #include <signal.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
 
 #include "bin/fdutils.h"
 #include "bin/process.h"
 #include "bin/set.h"
 
 
-class ActiveProcess {
+class ProcessInfo {
  public:
-  pid_t pid;
-  intptr_t fd;
+  ProcessInfo(pid_t pid, intptr_t fd) : pid_(pid), fd_(fd) { }
 
-  bool operator==(const ActiveProcess &other) const {
-    if (pid == other.pid) {
-      return true;
-    }
-    return false;
-  }
+  pid_t pid() { return pid_; }
+  intptr_t fd() { return fd_; }
+  ProcessInfo* next() { return next_; }
+  void set_next(ProcessInfo* next) { next_ = next; }
+
+ private:
+  pid_t pid_;  // Process pid.
+  intptr_t fd_;  // File descriptor for pipe to report exit code.
+  ProcessInfo* next_;
 };
 
 
-static Set<ActiveProcess> activeProcesses;
+ProcessInfo* active_processes = NULL;
+
+
+static void AddProcess(ProcessInfo* process) {
+  process->set_next(active_processes);
+  active_processes = process;
+}
+
+
+static ProcessInfo* LookupProcess(pid_t pid) {
+  ProcessInfo* current = active_processes;
+  while (current != NULL) {
+    if (current->pid() == pid) {
+      return current;
+    }
+    current = current->next();
+  }
+  return NULL;
+}
+
+
+static ProcessInfo* RemoveProcess(pid_t pid) {
+  ProcessInfo* prev = NULL;
+  ProcessInfo* current = active_processes;
+  while (current != NULL) {
+    if (current->pid() == pid) {
+      if (prev == NULL) {
+        active_processes = current->next();
+      } else {
+        prev->set_next(current->next());
+      }
+      return current;
+    }
+    prev = current;
+    current = current->next();
+  }
+  return NULL;
+}
 
 
 static char* SafeStrNCpy(char* dest, const char* src, size_t n) {
   strncpy(dest, src, n);
   dest[n - 1] = '\0';
   return dest;
 }
 
 
 static void SetChildOsErrorMessage(char* os_error_message,
                                    int os_error_message_len) {
   SafeStrNCpy(os_error_message, strerror(errno), os_error_message_len);
 }
 
 
-void ExitHandle(int processSignal, siginfo_t* siginfo, void* tmp) {
-  assert(processSignal == SIGCHLD);
+void ExitHandler(int process_signal, siginfo_t* siginfo, void* tmp) {
+  assert(process_signal == SIGCHLD);
   struct sigaction act;
   bzero(&act, sizeof(act));
   act.sa_handler = SIG_IGN;
   act.sa_flags = SA_NOCLDSTOP | SA_SIGINFO;
   if (sigaction(SIGCHLD, &act, 0) != 0) {
     perror("Process start: disabling signal handler failed");
   }
-  pid_t pid = siginfo->si_pid;
-  ActiveProcess element;
-  element.pid = pid;
-  ActiveProcess* current = activeProcesses.Remove(element);
-  if (current != NULL) {
-    intptr_t message = siginfo->si_status;
+  ProcessInfo* process = LookupProcess(siginfo->si_pid);
+  if (process != NULL) {
+    intptr_t message[2] = { siginfo->si_pid, siginfo->si_status };
     intptr_t result =
-        FDUtils::WriteToBlocking(current->fd, &message, sizeof(message));
+        FDUtils::WriteToBlocking(process->fd(), &message, sizeof(message));
     if (result != sizeof(message)) {
-      perror("ExitHandle notification failed");
+      perror("ExitHandler notification failed");
     }
-    close(current->fd);
-
-    delete current;
+    close(process->fd());
   }
   act.sa_handler = 0;
-  act.sa_sigaction = ExitHandle;
+  act.sa_sigaction = ExitHandler;
   if (sigaction(SIGCHLD, &act, 0) != 0) {
     perror("Process start: enabling signal handler failed");
   }
 }
 
 
 int Process::Start(const char* path,
                    char* arguments[],
                    intptr_t arguments_length,
                    intptr_t* in,
@@ skipping 69 matching lines @@
 
   char* program_arguments[arguments_length + 2];
   program_arguments[0] = const_cast<char *>(path);
   for (int i = 0; i < arguments_length; i++) {
     program_arguments[i + 1] = arguments[i];
   }
   program_arguments[arguments_length + 1] = NULL;
 
   struct sigaction act;
   bzero(&act, sizeof(act));
-  act.sa_sigaction = ExitHandle;
+  act.sa_sigaction = ExitHandler;
   act.sa_flags = SA_NOCLDSTOP | SA_SIGINFO;
   if (sigaction(SIGCHLD, &act, 0) != 0) {
     perror("Process start: setting signal handler failed");
   }
   pid = fork();
   if (pid < 0) {
     SetChildOsErrorMessage(os_error_message, os_error_message_len);
     close(read_in[0]);
     close(read_in[1]);
     close(read_err[0]);
@@ skipping 50 matching lines @@
     close(read_in[0]);
     close(read_in[1]);
     close(read_err[0]);
     close(read_err[1]);
     close(write_out[0]);
     close(write_out[1]);
     fprintf(stderr, "Error pipe creation failed: %s\n", os_error_message);
     return errno;
   }
 
-  ActiveProcess* activeProcess = new ActiveProcess();
-  activeProcess->pid = pid;
-  activeProcess->fd = event_fds[1];
-  activeProcesses.Add(*activeProcess);
+  ProcessInfo* process = new ProcessInfo(pid, event_fds[1]);
+  AddProcess(process);
   *exit_event = event_fds[0];
   FDUtils::SetNonBlocking(event_fds[0]);
 
   // Notify child process to start.
   char msg = '1';
   result = FDUtils::WriteToBlocking(read_in[1], &msg, sizeof(msg));
   if (result != sizeof(msg)) {
     perror("Failed sending notification message");
   }
 
@@ skipping 45 matching lines @@
 }
 
 
 bool Process::Kill(intptr_t id) {
   int result = kill(id, SIGKILL);
   if (result == -1) {
     return false;
   }
   return true;
 }
+
+
+void Process::Exit(intptr_t id) {
+  RemoveProcess(id);
+}