tests: use fixed socket address in net-y-unix.test

This allows net-y-unix invocation without arguments.

* tests/net-y-unix.c (TEST_SOCKET): New macro.
(void): Use it instead of av[1].
* tests/net-y-unix.test: Do not specify arguments for the test
executable.

diff --git a/tests/net-y-unix.c b/tests/net-y-unix.c
index 0b27c79c031a0596b978c065e080e496165ed8ca..98ea9563b3e727c2583f41ed194005ba5fd4d90b 100644 (file)
--- a/tests/net-y-unix.c
+++ b/tests/net-y-unix.c
@@ -1,7 +1,7 @@
 /*
  * This file is part of net-y-unix strace test.
  *
- * Copyright (c) 2013-2016 Dmitry V. Levin <ldv@altlinux.org>
+ * Copyright (c) 2013-2017 Dmitry V. Levin <ldv@altlinux.org>
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -38,19 +38,19 @@
 #include <sys/socket.h>
 #include <sys/un.h>
 
+#define TEST_SOCKET "net-y-unix.socket"
+
 int
-main(int ac, const char **av)
+main(void)
 {
-       assert(ac == 2);
-
-       struct sockaddr_un addr = { .sun_family = AF_UNIX };
-       unsigned int sun_path_len = strlen(av[1]);
-       assert(sun_path_len > 0 && sun_path_len <= sizeof(addr.sun_path));
-       strncpy(addr.sun_path, av[1], sizeof(addr.sun_path));
+       static const struct sockaddr_un addr = {
+               .sun_family = AF_UNIX,
+               .sun_path = TEST_SOCKET
+       };
        struct sockaddr * const listen_sa = tail_memdup(&addr, sizeof(addr));
 
        TAIL_ALLOC_OBJECT_CONST_PTR(socklen_t, len);
-       *len = offsetof(struct sockaddr_un, sun_path) + strlen(av[1]) + 1;
+       *len = offsetof(struct sockaddr_un, sun_path) + strlen(TEST_SOCKET) + 1;
        if (*len > sizeof(addr))
                *len = sizeof(addr);
 
@@ -61,11 +61,12 @@ main(int ac, const char **av)
        printf("socket(AF_UNIX, SOCK_STREAM, 0) = %d<socket:[%lu]>\n",
               listen_fd, listen_inode);
 
-       (void) unlink(av[1]);
+       (void) unlink(TEST_SOCKET);
        if (bind(listen_fd, listen_sa, *len))
                perror_msg_and_skip("bind");
        printf("bind(%d<socket:[%lu]>, {sa_family=AF_UNIX, sun_path=\"%s\"}"
-              ", %u) = 0\n", listen_fd, listen_inode, av[1], (unsigned) *len);
+              ", %u) = 0\n",
+              listen_fd, listen_inode, TEST_SOCKET, (unsigned) *len);
 
        if (listen(listen_fd, 1))
                perror_msg_and_skip("listen");
@@ -85,7 +86,7 @@ main(int ac, const char **av)
                perror_msg_and_fail("getsockname");
        printf("getsockname(%d<socket:[%lu]>, {sa_family=AF_UNIX"
               ", sun_path=\"%s\"}, [%d->%d]) = 0\n", listen_fd, listen_inode,
-              av[1], (int) sizeof(addr), (int) *len);
+              TEST_SOCKET, (int) sizeof(addr), (int) *len);
 
        int connect_fd = socket(AF_UNIX, SOCK_STREAM, 0);
        if (connect_fd < 0)
@@ -98,7 +99,7 @@ main(int ac, const char **av)
                perror_msg_and_fail("connect");
        printf("connect(%d<socket:[%lu]>, {sa_family=AF_UNIX"
               ", sun_path=\"%s\"}, %u) = 0\n",
-              connect_fd, connect_inode, av[1], (unsigned) *len);
+              connect_fd, connect_inode, TEST_SOCKET, (unsigned) *len);
 
        struct sockaddr * const accept_sa = tail_alloc(sizeof(addr));
        memset(accept_sa, 0, sizeof(addr));
@@ -119,7 +120,7 @@ main(int ac, const char **av)
                perror_msg_and_fail("getpeername");
        printf("getpeername(%d<socket:[%lu]>, {sa_family=AF_UNIX"
               ", sun_path=\"%s\"}, [%d->%d]) = 0\n", connect_fd, connect_inode,
-              av[1], (int) sizeof(addr), (int) *len);
+              TEST_SOCKET, (int) sizeof(addr), (int) *len);
 
        char text[] = "text";
        assert(sendto(connect_fd, text, sizeof(text) - 1, MSG_DONTWAIT, NULL, 0)
@@ -163,14 +164,14 @@ main(int ac, const char **av)
                perror_msg_and_fail("getsockname");
        printf("getsockname(%d<socket:[%lu]>, {sa_family=AF_UNIX"
               ", sun_path=\"%s\"}, [%d->%d]) = 0\n",
-              listen_fd, listen_inode, av[1],
+              listen_fd, listen_inode, TEST_SOCKET,
               (int) sizeof(addr), (int) *len);
 
        if (connect(connect_fd, listen_sa, *len))
                perror_msg_and_fail("connect");
        printf("connect(%d<socket:[%lu]>, {sa_family=AF_UNIX"
               ", sun_path=\"%s\"}, %u) = 0\n",
-              connect_fd, connect_inode, av[1], (unsigned) *len);
+              connect_fd, connect_inode, TEST_SOCKET, (unsigned) *len);
 
        memset(accept_sa, 0, sizeof(addr));
        *len = sizeof(addr);
@@ -192,7 +193,7 @@ main(int ac, const char **av)
                perror_msg_and_fail("getpeername");
        printf("getpeername(%d<socket:[%lu]>, {sa_family=AF_UNIX"
               ", sun_path=\"%s\"}, [%d->%d]) = 0\n",
-              connect_fd, connect_inode, av[1],
+              connect_fd, connect_inode, TEST_SOCKET,
               (int) sizeof(addr), (int) *len);
 
        memset(accept_sa, 0, sizeof(addr));
@@ -224,7 +225,7 @@ main(int ac, const char **av)
        assert(close(accept_fd) == 0);
        printf("close(%d<socket:[%lu]>) = 0\n", accept_fd, accept_inode);
 
-       assert(unlink(av[1]) == 0);
+       assert(unlink(TEST_SOCKET) == 0);
 
        assert(close(listen_fd) == 0);
        printf("close(%d<socket:[%lu]>) = 0\n",

diff --git a/tests/net-y-unix.test b/tests/net-y-unix.test
index bdc126e6cab9397c85465ae724631936ec16532c..c5df1f9da006505710e84d0a6a36c9742919a744 100755 (executable)
--- a/tests/net-y-unix.test
+++ b/tests/net-y-unix.test
@@ -2,7 +2,7 @@
 #
 # Check decoding of network syscalls in -y mode.
 #
-# Copyright (c) 2016 Dmitry V. Levin <ldv@altlinux.org>
+# Copyright (c) 2016-2017 Dmitry V. Levin <ldv@altlinux.org>
 # All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
@@ -35,8 +35,7 @@
 
 check_prog sed
 
-addr="$NAME-local-stream"
-run_prog "../$NAME" $addr > /dev/null
+run_prog "../$NAME" > /dev/null
 
 run_strace -a20 -y -eclose,network $args > "$EXP"
 # Filter out close() calls made by ld.so and libc.