* fork.cc (fork): Lock the process before forking to prevent things like new

fds from being opened, etc.
* sync.h (lock_process::dont_bother): New function.

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index a292b95..cf8ac0a 100644 (file)
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,5 +1,11 @@
 2006-07-14  Christopher Faylor  <cgf@timesys.com>
 
+       * fork.cc (fork): Lock the process before forking to prevent things
+       like new fds from being opened, etc.
+       * sync.h (lock_process::dont_bother): New function.
+
+2006-07-14  Christopher Faylor  <cgf@timesys.com>
+
        * include/cygwin/types.h: Update copyright.
 
 2006-07-14  Christopher Faylor  <cgf@timesys.com>

diff --git a/winsup/cygwin/fork.cc b/winsup/cygwin/fork.cc
index 2043629..ed85e74 100644 (file)
--- a/winsup/cygwin/fork.cc
+++ b/winsup/cygwin/fork.cc
@@ -533,6 +533,7 @@ fork ()
       return -1;
     }
 
+  lock_process now;
   if (sig_send (NULL, __SIGHOLD))
     {
       if (exit_state)
@@ -547,7 +548,10 @@ fork ()
   __asm__ volatile ("movl %%esp,%0": "=r" (esp));
 
   if (ischild)
-    res = grouped.child (esp);
+    {
+      res = grouped.child (esp);
+      now.dont_bother ();
+    }
   else
     {
       res = grouped.parent (esp);

diff --git a/winsup/cygwin/sync.h b/winsup/cygwin/sync.h
index 8fd01df..f7ce7e1 100644 (file)
--- a/winsup/cygwin/sync.h
+++ b/winsup/cygwin/sync.h
@@ -50,6 +50,7 @@ class lock_process
   static muto locker;
 public:
   static void init () {locker.init ("lock_process");}
+  void dont_bother () {skip_unlock = true;}
   lock_process (bool exiting = false)
   {
     locker.acquire ();