about summary refs log tree commit diff
diff options
context:
space:
mode:
authorRoland McGrath <roland@hack.frob.com>2013-04-16 12:11:26 -0700
committerRoland McGrath <roland@hack.frob.com>2013-04-16 12:11:26 -0700
commite913141d5f4d4eed4b65f55b0077aeb1c8234e25 (patch)
tree6fe870f3f1bdb6bc874fc51bf462e50f4a7f53bc
parent4856bcd2df33adfc36c762a48853ec613c0d2f68 (diff)
downloadglibc-e913141d5f4d4eed4b65f55b0077aeb1c8234e25.tar.gz
glibc-e913141d5f4d4eed4b65f55b0077aeb1c8234e25.tar.xz
glibc-e913141d5f4d4eed4b65f55b0077aeb1c8234e25.zip
Don't test O_RDONLY case in tst-aio7.
-rw-r--r--ChangeLog5
-rw-r--r--rt/tst-aio7.c17
2 files changed, 6 insertions, 16 deletions
diff --git a/ChangeLog b/ChangeLog
index ca232e7a68..90d6d47637 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 c156e86dd0..b25c33c153 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.  */