Hi,

"conformance/interfaces/aio_error/3-1" failed with following message:
------------
conformance/interfaces/aio_error/3-1: execution: FAILED: Output: 
aio_error/3-1.c bad aio_read return value()
------------

This testset seems to be the error root test for "aio_error()".
Therefore, "aio_error()" must be called after failure of "aio_write()", 
I think.
But, "exit()" is called when expected failure of "aio_write()" occurred:
------------(3-1.c)
        if (aio_write(&aiocb) != 0)
 {
                printf(TNAME " bad aio_read return value()\n");
                exit(PTS_FAIL);
 }
------------

I think that "exit()" must be called when "aio_write()" succeeded.

And the message of unexpected movement of "aio_write()" is 
"aio_read()"'s one.
And the indent of this program "3-1.c" seems to include both space and tab.
I want to revise them, too.


Signed-off-by: Tomonori Mitani <[email protected]>
============
---
a/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c 2010
-09-22 22:31:24.000000000 +0900
+++
b/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c 2010
-09-30 09:51:42.000000000 +0900
@@ -36,42 +36,41 @@
 
 int main()
 {
-
-        char tmpfname[256];
+       char tmpfname[256];
 #define BUF_SIZE 512
-        char buf[BUF_SIZE];
-        int fd;
-        struct aiocb aiocb;
+       char buf[BUF_SIZE];
+       int fd;
+       struct aiocb aiocb;
        int ret=0;
 
        if (sysconf(_SC_ASYNCHRONOUS_IO) != 200112L)
                return PTS_UNSUPPORTED;
 
-        snprintf(tmpfname, sizeof(tmpfname), "/tmp/pts_aio_error_3_1_%d",
-                  getpid());
-        unlink(tmpfname);
-        fd = open(tmpfname, O_CREAT | O_RDWR | O_EXCL,
-                  S_IRUSR | S_IWUSR);
-        if (fd == -1)
-        {
-                printf(TNAME " Error at open(): %s\n",
-                       strerror(errno));
-                exit(PTS_UNRESOLVED);
-        }
+       snprintf(tmpfname, sizeof(tmpfname), "/tmp/pts_aio_error_3_1_%d",
+                        getpid());
+       unlink(tmpfname);
+       fd = open(tmpfname, O_CREAT | O_RDWR | O_EXCL,
+                         S_IRUSR | S_IWUSR);
+       if (fd == -1)
+       {
+               printf(TNAME " Error at open(): %s\n",
+                          strerror(errno));
+               exit(PTS_UNRESOLVED);
+       }
 
-        unlink(tmpfname);
+       unlink(tmpfname);
 
        memset (&aiocb, 0, sizeof (struct aiocb));
 
-        aiocb.aio_fildes = fd;
-        aiocb.aio_buf = buf;
-        aiocb.aio_reqprio = -1;
-        aiocb.aio_nbytes = BUF_SIZE;
+       aiocb.aio_fildes = fd;
+       aiocb.aio_buf = buf;
+       aiocb.aio_reqprio = -1;
+       aiocb.aio_nbytes = BUF_SIZE;
 
-        if (aio_write(&aiocb) != 0)
+       if (aio_write(&aiocb) == 0)
        {
-                printf(TNAME " bad aio_read return value()\n");
-                exit(PTS_FAIL);
+               printf(TNAME " bad aio_write return value()\n");
+               exit(PTS_FAIL);
        }
 
        while (aio_error (&aiocb) == EINPROGRESS);
============


Regards--

-Tomonori Mitani

Attachment: 3-1.patch
Description: Binary data

------------------------------------------------------------------------------
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to