stdlib-safer: preserve cloexec flag for mkostemp[s]

mkostemp_safer(templ,O_CLOEXEC) did not always guarantee cloexec.

* lib/mkstemp-safer.c (mkostemp_safer, mkostemps_safer): Use new
fd_safer_flag.

Signed-off-by: Eric Blake <ebb9@byu.net>

diff --git a/ChangeLog b/ChangeLog
index 20535a05c5ea896968ed544b88849cdfe1820910..21c86be91ac4d853b4f3fbb599a28f851f27184a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 2009-12-05  Eric Blake  <ebb9@byu.net>
 
+       stdlib-safer: preserve cloexec flag for mkostemp[s]
+       * lib/mkstemp-safer.c (mkostemp_safer, mkostemps_safer): Use new
+       fd_safer_flag.
+
        unistd-safer: allow preservation of cloexec status via flag
        * lib/unistd-safer.h (dup_safer_flag, fd_safer_flag): New
        prototypes.

diff --git a/lib/mkstemp-safer.c b/lib/mkstemp-safer.c
index 95d315b030bdacc62a3543c5e6f09c31289450a6..ee242f3f104dee628c47df46674446f4c9fdca28 100644 (file)
--- a/lib/mkstemp-safer.c
+++ b/lib/mkstemp-safer.c
@@ -39,7 +39,7 @@ mkstemp_safer (char *templ)
 int
 mkostemp_safer (char *templ, int flags)
 {
-  return fd_safer (mkostemp (templ, flags));
+  return fd_safer_flag (mkostemp (templ, flags), flags);
 }
 #endif
 
@@ -49,7 +49,7 @@ mkostemp_safer (char *templ, int flags)
 int
 mkostemps_safer (char *templ, int suffixlen, int flags)
 {
-  return fd_safer (mkostemps (templ, suffixlen, flags));
+  return fd_safer_flag (mkostemps (templ, suffixlen, flags), flags);
 }
 #endif