Changeset 2173 for trunk/src

Timestamp:

Jul 3, 2005, 5:40:02 AM (20 years ago)

Author:

bird

Message:

Clear FD_CLOEXEC on the handle returned by dup, dup2 and fcntl(F_DUPFD_).

Location:

trunk/src/emx

Files:

• ChangeLog.LIBC (modified) (1 diff, 1 prop)
• include/emx/io.h (modified) (1 diff, 1 prop)
• src/lib/libc.def (modified) (1 diff, 1 prop)
• src/lib/sys/__dup.c (modified) (2 diffs, 1 prop)
• src/lib/sys/__dup2.c (modified) (2 diffs, 1 prop)
• src/lib/sys/__fcntl.c (modified) (1 diff, 1 prop)
• src/lib/sys/filehandles.c (modified) (1 diff, 1 prop)

trunk/src/emx/ChangeLog.LIBC

@@ -13 +13 @@
         o Fixed mixup in fts.c port.
         o Replaced the EMX ftw with the BSD one so we implement the SuS specs.
+
 
 2005-07-01: knut st. osmundsen <[email protected]>

trunk/src/emx/include/emx/io.h

@@ -496 +496 @@
 PLIBCFH __libc_FH(int fh);
 int     __libc_FHEx(int fh, __LIBC_PFH *ppFH);
+
 
 

trunk/src/emx/src/lib/libc.def

@@ -1891 +1891 @@
     "__obstack_newchunk" @1891
     "__std_nftw" @1892
+

trunk/src/emx/src/lib/sys/__dup.c

@@ -9 +9 @@
 #include <emx/io.h>
 #include <emx/syscalls.h>
+
 #include "syscalls.h"
 #include "b_fs.h"
@@ -61 +62 @@
                 pFHNew->Dev         = pFH->Dev;
                 pFHNew->pFsInfo     = __libc_back_fsInfoObjAddRef(pFH->pFsInfo);
+
+
+
+
+
+
+
+
+
+
             }
             else

trunk/src/emx/src/lib/sys/__dup2.c

@@ -8 +8 @@
 #include <emx/syscalls.h>
 #include <emx/io.h>
+
 #include "syscalls.h"
 #include "b_fs.h"
@@ -94 +95 @@
                 pFHNew->Dev         = pFH->Dev;
                 pFHNew->pFsInfo     = __libc_back_fsInfoObjAddRef(pFH->pFsInfo);
+
+
+
+
+
+
+
+
+
+
             }
             else

trunk/src/emx/src/lib/sys/__fcntl.c

@@ -193 +193 @@
 {
     LIBCLOG_ENTER("pFH=%p hFile=%d arg=%#x\n", (void *)pFH, hFile, arg);
-    int     rc;
-    ULONG   fulState;
-    FS_VAR();
-
-    FS_SAVE_LOAD();
-    rc = DosQueryFHState(hFile, &fulState);
+
+    /*
+     * Calc new flags.
+     */
+    unsigned fFlags = pFH->fFlags;
+    fFlags = (fFlags & ~__LIBC_FH_FDFLAGS_MASK) | (arg << __LIBC_FH_FDFLAGS_SHIFT);
+    if (arg & FD_CLOEXEC)
+        fFlags |= O_NOINHERIT;
+    else
+        fFlags &= ~O_NOINHERIT;
+
+    /*
+     * Update the flags.
+     */
+    int rc = __libc_FHSetFlags(pFH, hFile, fFlags);
     if (!rc)
-    {
-        ULONG fulNewState;
-        LIBC_ASSERTM(     ( (fulState & OPEN_FLAGS_NOINHERIT) != 0 )
-                      ==  (   (pFH->fFlags & (O_NOINHERIT | (FD_CLOEXEC << __LIBC_FH_FDFLAGS_SHIFT)))
-                           == (O_NOINHERIT | (FD_CLOEXEC << __LIBC_FH_FDFLAGS_SHIFT)) ),
-                     "Inherit flags are out of sync for file hFile %d (%#x)! fulState=%08lx fFlags=%08x\n",
-                     hFile, hFile, fulState, pFH->fFlags);
-        if (arg & FD_CLOEXEC)
-            fulNewState = fulState | OPEN_FLAGS_NOINHERIT;
-        else
-            fulNewState = fulState & ~OPEN_FLAGS_NOINHERIT;
-        if (fulNewState != fulState)
-        {
-            fulNewState &= 0x7f88; /* Mask the flags accepted the API. */
-            rc = DosSetFHState(hFile, fulNewState);
-        }
-    }
-    FS_RESTORE();
-
-    if (!rc)
-    {
-        unsigned fFlags = pFH->fFlags;
-        fFlags = (fFlags & ~__LIBC_FH_FDFLAGS_MASK) | (arg << __LIBC_FH_FDFLAGS_SHIFT);
-        if (arg & FD_CLOEXEC)
-            fFlags |= O_NOINHERIT;
-        else
-            fFlags &= ~O_NOINHERIT;
-        __atomic_xchg(&pFH->fFlags, fFlags);
         LIBCLOG_RETURN_INT(0);
-    }
-
-    /* error! */
-    _sys_set_errno(rc);
+    errno = -rc;
     LIBCLOG_RETURN_INT(-1);
 }

trunk/src/emx/src/lib/sys/filehandles.c

@@ -1143 +1143 @@
     return rc;
 }
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+