From: Eric Blake <ebb9@byu.net>
Date: Thu, 13 Aug 2009 18:11:47 +0000 (-0600)
Subject: test-dup2: fix bad assumption
X-Git-Tag: v0.1~5662
X-Git-Url: http://erislabs.org.uk/gitweb/?a=commitdiff_plain;h=bb218f8d923ea9bac42c37bb7cf190b4dbd695f1;p=gnulib.git

test-dup2: fix bad assumption

* tests/test-dup2.c (main): Tolerate leaked fds from environment.
Reported by Peter Breitenlohner <peb@mppmu.mpg.de>.

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

diff --git a/ChangeLog b/ChangeLog
index 960f3cc42..d6c3670e6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 2009-08-13  Eric Blake  <ebb9@byu.net>
 
+	test-dup2: fix bad assumption
+	* tests/test-dup2.c (main): Tolerate leaked fds from environment.
+	Reported by Peter Breitenlohner <peb@mppmu.mpg.de>.
+
 	test-version-etc: fix CRLF portability issue
 	* tests/test-version-etc.sh: Use tr, not sed, as not all sed
 	recognize \r.
diff --git a/tests/test-dup2.c b/tests/test-dup2.c
index 67f49eff0..2009aaa26 100644
--- a/tests/test-dup2.c
+++ b/tests/test-dup2.c
@@ -67,8 +67,12 @@ main ()
   char buffer[1];
   int fd = open (file, O_CREAT | O_RDWR, 0600);
 
-  ASSERT (0 <= fd);
+  /* Assume std descriptors were provided by invoker.  */
+  ASSERT (STDERR_FILENO < fd);
   ASSERT (is_open (fd));
+  /* Ignore any other fd's leaked into this process.  */
+  close (fd + 1);
+  close (fd + 2);
   ASSERT (!is_open (fd + 1));
   ASSERT (!is_open (fd + 2));