threads / discuss / 37696

PATCH] mingw: Fix a typo in the pipe() implementation.

Subject: PATCH] mingw: Fix a typo in the pipe() implementation.

## tl;dr

One message between Oct 9, 2014 and Oct 9, 2014.

replies: 0people: 1as markdown or json

Ralph Loader· Oct 9, 2014, 08:03 UTC · lore

We checked the first file descriptor for errors twice, instead of checking both file descriptors.

Signed-off-by: Ralph Loader <rcl@ihug.co.nz>
---
While this appears utterly trivial, I noticed this reading the code not
running it - I do not have the relevant OS to compile and test on.
 compat/mingw.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/compat/mingw.c b/compat/mingw.c
index c5c37e5..7c92783 100644
--- a/compat/mingw.c
+++ b/compat/mingw.c
@@ -681,7 +681,7 @@ int pipe(int filedes[2])
 		return -1;
 	}
 	filedes[1] = _open_osfhandle((int)h[1], O_NOINHERIT);
-	if (filedes[0] < 0) {
+	if (filedes[1] < 0) {
 		close(filedes[0]);
 		CloseHandle(h[1]);
 		return -1;
-- 
2.1.0

← back to recent threads