# HG changeset patch
# User Bruno Haible <bruno@clisp.org>
# Date 1251015654 -7200
# Node ID bd0609bf2760f062b53572c01f350d0e539ad791
# Parent  ed51af85012bda19d031a087ad3ef79412f65315
Fix test of fcntl's return value.

diff --git a/ChangeLog b/ChangeLog
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2009-08-23  Bruno Haible  <bruno@clisp.org>
+
+	* lib/pipe2.c (pipe2): Fix test of fcntl's return value.
+
 2009-08-23  Bruno Haible  <bruno@clisp.org>
 
 	* modules/fchdir (Depends-on): Use fcntl-h instead of fcntl.
diff --git a/lib/pipe2.c b/lib/pipe2.c
--- a/lib/pipe2.c
+++ b/lib/pipe2.c
@@ -70,9 +70,9 @@
       int fcntl_flags;
 
       if ((fcntl_flags = fcntl (fd[1], F_GETFL, 0)) < 0
-	  || fcntl (fd[1], F_SETFL, fcntl_flags | O_NONBLOCK) < 0
+	  || fcntl (fd[1], F_SETFL, fcntl_flags | O_NONBLOCK) == -1
 	  || (fcntl_flags = fcntl (fd[0], F_GETFL, 0)) < 0
-	  || fcntl (fd[0], F_SETFL, fcntl_flags | O_NONBLOCK) < 0)
+	  || fcntl (fd[0], F_SETFL, fcntl_flags | O_NONBLOCK) == -1)
 	goto fail;
     }
 
@@ -81,9 +81,9 @@
       int fcntl_flags;
 
       if ((fcntl_flags = fcntl (fd[1], F_GETFD, 0)) < 0
-	  || fcntl (fd[1], F_SETFD, fcntl_flags | FD_CLOEXEC) < 0
+	  || fcntl (fd[1], F_SETFD, fcntl_flags | FD_CLOEXEC) == -1
 	  || (fcntl_flags = fcntl (fd[0], F_GETFD, 0)) < 0
-	  || fcntl (fd[0], F_SETFD, fcntl_flags | FD_CLOEXEC) < 0)
+	  || fcntl (fd[0], F_SETFD, fcntl_flags | FD_CLOEXEC) == -1)
 	goto fail;
     }