strace: terminate itself if interrupted by a signal

[strace] / tests / netlink_unix_diag.c

diff --git a/tests/netlink_unix_diag.c b/tests/netlink_unix_diag.c
index 817bf01ee0de57f3c6a755e00e17af9d97a5450b..62e3ef1f418ffd29ab602bd2e4dba891fe74f894 100644 (file)
--- a/tests/netlink_unix_diag.c
+++ b/tests/netlink_unix_diag.c
@@ -1,20 +1,28 @@
+/*
+ * This file is part of net-yy-unix strace test.
+ *
+ * Copyright (c) 2014-2016 Dmitry V. Levin <ldv@altlinux.org>
+ * Copyright (c) 2014-2018 The strace developers.
+ * All rights reserved.
+ *
+ * SPDX-License-Identifier: GPL-2.0-or-later
+ */
+
+#include "tests.h"
 #include <assert.h>
+#include <errno.h>
 #include <stddef.h>
 #include <stdint.h>
 #include <string.h>
 #include <unistd.h>
 #include <sys/socket.h>
 #include <sys/un.h>
-#include <linux/netlink.h>
+#include "netlink.h"
 #include <linux/sock_diag.h>
 #include <linux/unix_diag.h>
 
-#if !defined NETLINK_SOCK_DIAG && defined NETLINK_INET_DIAG
-# define NETLINK_SOCK_DIAG NETLINK_INET_DIAG
-#endif
-
-static int
-send_query(const int fd, const int family, const int proto)
+static void
+send_query(const int fd)
 {
        struct sockaddr_nl nladdr = {
                .nl_family = AF_NETLINK
@@ -26,11 +34,10 @@ send_query(const int fd, const int family, const int proto)
                .nlh = {
                        .nlmsg_len = sizeof(req),
                        .nlmsg_type = SOCK_DIAG_BY_FAMILY,
-                       .nlmsg_flags = NLM_F_DUMP | NLM_F_REQUEST
+                       .nlmsg_flags = NLM_F_REQUEST | NLM_F_DUMP
                },
                .udr = {
-                       .sdiag_family = family,
-                       .sdiag_protocol = proto,
+                       .sdiag_family = AF_UNIX,
                        .udiag_states = -1,
                        .udiag_show = UDIAG_SHOW_NAME | UDIAG_SHOW_PEER
                }
@@ -40,28 +47,33 @@ send_query(const int fd, const int family, const int proto)
                .iov_len = sizeof(req)
        };
        struct msghdr msg = {
-               .msg_name = (void*)&nladdr,
+               .msg_name = (void *) &nladdr,
                .msg_namelen = sizeof(nladdr),
                .msg_iov = &iov,
                .msg_iovlen = 1
        };
 
-       return sendmsg(fd, &msg, 0) > 0;
+       if (sendmsg(fd, &msg, 0) <= 0)
+               perror_msg_and_skip("sendmsg");
 }
 
-static int
+static void
 check_responses(const int fd)
 {
-       static char buf[8192];
+       static union {
+               struct nlmsghdr hdr;
+               long buf[8192 / sizeof(long)];
+       } hdr_buf;
+
        struct sockaddr_nl nladdr = {
                .nl_family = AF_NETLINK
        };
        struct iovec iov = {
-               .iov_base = buf,
-               .iov_len = sizeof(buf)
+               .iov_base = hdr_buf.buf,
+               .iov_len = sizeof(hdr_buf.buf)
        };
        struct msghdr msg = {
-               .msg_name = (void*)&nladdr,
+               .msg_name = (void *) &nladdr,
                .msg_namelen = sizeof(nladdr),
                .msg_iov = &iov,
                .msg_iovlen = 1
@@ -69,12 +81,25 @@ check_responses(const int fd)
 
        ssize_t ret = recvmsg(fd, &msg, 0);
        if (ret <= 0)
-               return 0;
+               perror_msg_and_skip("recvmsg");
+
+       struct nlmsghdr *h = &hdr_buf.hdr;
+       if (!is_nlmsg_ok(h, ret))
+               error_msg_and_skip("!is_nlmsg_ok");
+       if (h->nlmsg_type == NLMSG_ERROR) {
+               const struct nlmsgerr *err = NLMSG_DATA(h);
+               if (h->nlmsg_len < NLMSG_LENGTH(sizeof(*err)))
+                       error_msg_and_skip("NLMSG_ERROR");
+               errno = -err->error;
+               perror_msg_and_skip("NLMSG_ERROR");
+       }
+       if (h->nlmsg_type != SOCK_DIAG_BY_FAMILY)
+               error_msg_and_skip("unexpected nlmsg_type %u",
+                                  (unsigned) h->nlmsg_type);
 
-       struct nlmsghdr *h = (struct nlmsghdr*)buf;
-       return (NLMSG_OK(h, ret) &&
-               h->nlmsg_type != NLMSG_ERROR &&
-               h->nlmsg_type != NLMSG_DONE) ? 1 : 0;
+       const struct unix_diag_msg *diag = NLMSG_DATA(h);
+       if (h->nlmsg_len < NLMSG_LENGTH(sizeof(*diag)))
+               error_msg_and_skip("short response");
 }
 
 #define SUN_PATH "netlink_unix_diag_socket"
@@ -90,16 +115,19 @@ int main(void)
        close(1);
 
        (void) unlink(SUN_PATH);
-       if (socket(PF_LOCAL, SOCK_STREAM, 0) ||
-           bind(0, (struct sockaddr *) &addr, len) ||
-           listen(0, 5))
-               return 77;
+       if (socket(AF_UNIX, SOCK_STREAM, 0))
+               perror_msg_and_skip("socket AF_UNIX");
+       if (bind(0, (struct sockaddr *) &addr, len))
+               perror_msg_and_skip("bind");
+       if (listen(0, 5))
+               perror_msg_and_skip("listen");
 
        assert(unlink(SUN_PATH) == 0);
 
        if (socket(AF_NETLINK, SOCK_RAW, NETLINK_SOCK_DIAG) != 1)
-               return 77;
+               perror_msg_and_skip("socket AF_NETLINK");
 
-       return (send_query(1, AF_UNIX, 0) &&
-               check_responses(1)) ? 0 : 77;
+       send_query(1);
+       check_responses(1);
+       return 0;
 }