Block signals while forking so they aren't run in the child process.

base/process/launch_posix.cc

Index: base/process/launch_posix.cc
diff --git a/base/process/launch_posix.cc b/base/process/launch_posix.cc
index 3708af6bac417b49e4015fed93eb49966ed78363..d7cdd3ed2d266ec5fd86ca457412a14a8c2fb9a7 100644
--- a/base/process/launch_posix.cc
+++ b/base/process/launch_posix.cc
@@ -77,6 +77,22 @@ void SetEnvironment(char** env) {
 #endif
 }
 
+// Set the calling thread's signal mask to new_sigmask and return
+// the previous signal mask.
+sigset_t SetSignalMask(const sigset_t& new_sigmask) {
+  sigset_t old_sigmask;
+#if defined(OS_ANDROID)
+  // POSIX says pthread_sigmask() must be used in multi-threaded processes,
+  // but Android's pthread_sigmask() was broken until 4.1:
+  // https://code.google.com/p/android/issues/detail?id=15337
+  // http://stackoverflow.com/questions/13777109/pthread-sigmask-on-android-not-working
+  RAW_CHECK(sigprocmask(SIG_SETMASK, &new_sigmask, &old_sigmask) == 0);
+#else
+  RAW_CHECK(pthread_sigmask(SIG_SETMASK, &new_sigmask, &old_sigmask) == 0);
+#endif
+  return old_sigmask;
+}
+
 #if !defined(OS_LINUX) || \
     (!defined(__i386__) && !defined(__x86_64__) && !defined(__arm__))
 void ResetChildSignalHandlersToDefaults() {
@@ -394,9 +410,14 @@ bool LaunchProcess(const std::vector<std::string>& argv,
   if (options.environ)
     new_environ.reset(AlterEnvironment(*options.environ, GetEnvironment()));
 
+  sigset_t full_sigset;
+  sigfillset(&full_sigset);
+  const sigset_t orig_sigmask = SetSignalMask(full_sigset);
+
   pid_t pid;
 #if defined(OS_LINUX)
   if (options.clone_flags) {
+    RAW_CHECK(!(options.clone_flags & (CLONE_THREAD | CLONE_VM)));

[jln (very slow on Chromium), 2013/08/05 21:55:40]

Maybe add CLONE_SIGHAND as well here ?

Add a comment along the lines of "Signal handling in this function assumes the
creation of a new process, check that a thread is not being created by mistake
and that signal handling follows the process-creation rules".

[mdempsky_google, 2013/08/05 22:05:19]

Done.

     pid = syscall(__NR_clone, options.clone_flags, 0, 0, 0);
   } else
 #endif
@@ -404,6 +425,11 @@ bool LaunchProcess(const std::vector<std::string>& argv,
     pid = fork();
   }
 
+  // Always restore the original signal mask in the parent.
+  if (pid != 0) {
+    SetSignalMask(orig_sigmask);
+  }
+
   if (pid < 0) {
     DPLOG(ERROR) << "fork";
     return false;
@@ -469,6 +495,7 @@ bool LaunchProcess(const std::vector<std::string>& argv,
 #endif  // defined(OS_MACOSX)
 
     ResetChildSignalHandlersToDefaults();
+    SetSignalMask(orig_sigmask);
 
 #if 0
     // When debugging it can be helpful to check that we really aren't making