tests: another fix

- test80: resolve race condition resulting from unintended send

Change-Id: Id029d679a3903de0f712a15d4756952dbc36070c

diff --git a/minix/tests/common-socket.c b/minix/tests/common-socket.c
index 9300fa52840ca853ee3b6225e6e4c191221fc7f6..7de4f39ff662eca35948b19569fc73298ea7b3b2 100644 (file)
--- a/minix/tests/common-socket.c
+++ b/minix/tests/common-socket.c
@@ -1637,12 +1637,12 @@ test_nonblock(const struct socket_test_info *info)
                test_fail("recv() should have yielded EAGAIN");
 
        /* This may be an implementation aspect, or even plain wrong (?). */
-       if (send(client_sd, buf, sizeof(buf), 0) != -1) {
-               if (!info->ignore_send_waiting) {
+       if (!info->ignore_send_waiting) {
+               if (send(client_sd, buf, sizeof(buf), 0) != -1) {
                        test_fail("send() should have failed");
+               } else if (errno != EAGAIN) {
+                       test_fail("send() should have yielded EAGAIN");
                }
-       } else if (errno != EAGAIN) {
-               test_fail("send() should have yielded EAGAIN");
        }
 
        switch (fork()) {