]> Savannah Git Hosting - gnulib.git/commitdiff
dup2: work around cygwin bug
authorEric Blake <eblake@redhat.com>
Mon, 31 Dec 2012 18:15:53 +0000 (11:15 -0700)
committerEric Blake <eblake@redhat.com>
Mon, 31 Dec 2012 18:51:32 +0000 (11:51 -0700)
Detected by './gnulib-tool --test dup2 cloexec'.
Reported upstream: http://cygwin.com/ml/cygwin/2012-12/msg00377.html
and fixed already: http://cygwin.com/ml/cygwin-cvs/2012-q4/msg00202.html
but as we want to work with older cygwin, we'll have to carry this
in gnulib for a while.

* m4/dup2.m4 (gl_FUNC_DUP2): Flush out cygwin core dump.
* lib/dup2.c (rpl_dup2): Work around it.
* doc/posix-functions/dup2.texi (dup2): Document it.

Signed-off-by: Eric Blake <eblake@redhat.com>
ChangeLog
doc/posix-functions/dup2.texi
lib/dup2.c
m4/dup2.m4

index 9d1a69ba03ae0c9129f40c21bf12978b6c24afcf..af52d73a13861a9412752a34aba51a012f73d465 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2012-12-31  Eric Blake  <eblake@redhat.com>
+
+       dup2: work around cygwin bug
+       * m4/dup2.m4 (gl_FUNC_DUP2): Flush out cygwin core dump.
+       * lib/dup2.c (rpl_dup2): Work around it.
+       * doc/posix-functions/dup2.texi (dup2): Document it.
+
 2012-12-30  Paul Eggert  <eggert@cs.ucla.edu>
 
        regex: remove unnecessary dependency on localcharset.h
index 012bb7f0e091df6a611d99dd73a9c85e6fccd6a4..14e5236d4ff1b0ac5373e2d401566d0a4f5369db 100644 (file)
@@ -18,7 +18,7 @@ mingw, MSVC 9.
 
 @item
 This function crashes when invoked with invalid arguments on some platforms:
-MSVC 9.
+Cygwin 1.7.17, MSVC 9.
 
 @item
 This function resets the @code{FD_CLOEXEC} flag when duplicating an fd
index f6d0f1c73c5aa3c1fe4d7b66c933d553885df7f6..36b339906761c85264df4b69108cb3c67759d6e9 100644 (file)
@@ -95,7 +95,10 @@ rpl_dup2 (int fd, int desired_fd)
 # ifdef F_GETFL
   /* On Linux kernels 2.6.26-2.6.29, dup2 (fd, fd) returns -EBADF.
      On Cygwin 1.5.x, dup2 (1, 1) returns 0.
+     On Cygwin 1.7.17, dup2 (1, -1) dumps core.
      On Haiku, dup2 (fd, fd) mistakenly clears FD_CLOEXEC.  */
+  if (desired_fd < 0)
+    fd = desired_fd;
   if (fd == desired_fd)
     return fcntl (fd, F_GETFL) == -1 ? -1 : fd;
 # endif
index fc86e8085babcbd500bafb28599d77b80952a68e..fa7f6d5513d6f7f3f44b7ebf01b02028a4f0d4bb 100644 (file)
@@ -1,4 +1,4 @@
-#serial 18
+#serial 19
 dnl Copyright (C) 2002, 2005, 2007, 2009-2012 Free Software Foundation, Inc.
 dnl This file is free software; the Free Software Foundation
 dnl gives unlimited permission to copy and/or distribute it,
@@ -39,6 +39,9 @@ AC_DEFUN([gl_FUNC_DUP2],
             /* Many gnulib modules require POSIX conformance of EBADF.  */
             if (dup2 (2, 1000000) == -1 && errno != EBADF)
               result |= 16;
+            /* Flush out a cygwin core dump.  */
+            if (dup2 (2, -1) != -1 || errno != EBADF)
+              result |= 32;
             return result;
            ])
         ],