* path.cc (cwdstuff::set): Avoid double aquiring of cwd_lock.

	Open directory with inheriting enabled.

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index e9047b288550018cbaae73663cec3dc6c467e7de..f23f5e2b1e8eddef8ec3b7b9ab97432cf75ee9ac 100644 (file)
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,3 +1,8 @@
+2007-05-22  Corinna Vinschen  <corinna@vinschen.de>
+
+       * path.cc (cwdstuff::set): Avoid double aquiring of cwd_lock.
+       Open directory with inheriting enabled.
+
 2007-05-21  Christian Franke <franke@computer.org>
 
        * fhandler_floppy.cc (fhandler_dev_floppy::lseek): Don't invalidate

diff --git a/winsup/cygwin/path.cc b/winsup/cygwin/path.cc
index 3fadeb8c1ab2efc08cd48595fcfaae7beec69d59..1bd7cff2375f14e26e2a36559141dba2b75eef2e 100644 (file)
--- a/winsup/cygwin/path.cc
+++ b/winsup/cygwin/path.cc
@@ -4224,7 +4224,8 @@ cwdstuff::set (const char *win32_cwd, const char *posix_cwd, bool doit)
 
   if (win32_cwd)
     {
-      cwd_lock.acquire ();
+      if (!cwd_lock.acquired ())
+       cwd_lock.acquire ();
       if (doit)
        {
          /* We utilize the user parameter block.  The directory is
@@ -4252,7 +4253,7 @@ cwdstuff::set (const char *win32_cwd, const char *posix_cwd, bool doit)
              goto out;
            }
          h = CreateFile (win32_cwd, FILE_TRAVERSE, FILE_SHARE_VALID_FLAGS,
-                         NULL, OPEN_EXISTING, FILE_FLAG_BACKUP_SEMANTICS,
+                         &sec_none, OPEN_EXISTING, FILE_FLAG_BACKUP_SEMANTICS,
                          NULL);
          if (h == INVALID_HANDLE_VALUE)
            {