From: Roland McGrath <roland@hack.frob.com>
Date: Tue, 16 Apr 2013 19:11:26 +0000 (-0700)
Subject: Don't test O_RDONLY case in tst-aio7.
X-Git-Tag: glibc-2.18~398
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e913141d5f4d4eed4b65f55b0077aeb1c8234e25;p=thirdparty%2Fglibc.git

Don't test O_RDONLY case in tst-aio7.
---

diff --git a/ChangeLog b/ChangeLog
index ca232e7a68e..90d6d476377 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2013-04-16  Roland McGrath  <roland@hack.frob.com>
+
+	* rt/tst-aio7.c (do_test): Don't test O_RDONLY fd case, which is now
+	considered kosher.
+
 2013-04-16  Siddhesh Poyarekar  <siddhesh@redhat.com>
 
 	* benchtests/Makefile: Include cppflags-iterator.mk to add
diff --git a/rt/tst-aio7.c b/rt/tst-aio7.c
index c156e86dd00..b25c33c1539 100644
--- a/rt/tst-aio7.c
+++ b/rt/tst-aio7.c
@@ -78,7 +78,7 @@ do_test (void)
       puts ("aio_cancel( -1, {-2..} ) did not return -1 or errno != EBADF");
   }
 
-  /* Test for aio_fsync() detecting bad fd, and fd not open for writing.  */
+  /* Test for aio_fsync() detecting bad fd.  */
   {
     struct aiocb cb;
     int fd = -1;
@@ -98,21 +98,6 @@ do_test (void)
 	puts ("aio_fsync( op, {-1..} ) did not return -1 or errno != EBADF");
 	++result;
       }
-
-    if ((fd = open ("/dev/null", O_RDONLY)) < 0)
-      error (1, errno, "opening /dev/null");
-
-    cb.aio_fildes = fd;
-    errno = 0;
-
-    /* Case two: valid fd but open for read only.  */
-    if (aio_fsync (O_SYNC, &cb) != -1 || errno != EBADF)
-      {
-	puts ("aio_fsync( op, {RO..} ) did not return -1 or errno != EBADF");
-	++result;
-      }
-
-    close (fd);
   }
 
   /* Test for aio_suspend() suspending even if completed elements in list.  */