selftests: net: af_unix: Test connect() with different netns.
authorKuniyuki Iwashima <kuniyu@amazon.com>
Sat, 2 Jul 2022 15:48:18 +0000 (08:48 -0700)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 5 Jul 2022 09:34:58 +0000 (11:34 +0200)
This patch add a test that checks connect()ivity between two sockets:

    unnamed socket -> bound socket
                      * SOCK_STREAM or SOCK_DGRAM
                      * pathname or abstract
                      * same or different netns

Signed-off-by: Kuniyuki Iwashima <kuniyu@amazon.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
tools/testing/selftests/net/.gitignore
tools/testing/selftests/net/af_unix/Makefile
tools/testing/selftests/net/af_unix/unix_connect.c [new file with mode: 0644]

index a29f796189347e67b4ad20a7fd65e63f5bbfa8a8..1257baa79286e9db9fb392d4782a68fe3aff8867 100644 (file)
@@ -37,3 +37,4 @@ gro
 ioam6_parser
 toeplitz
 cmsg_sender
+unix_connect
\ No newline at end of file
index df341648f8180ae8378cefb7be034ea42559cf16..969620ae992843bf2f21eb80189c92656122f1d6 100644 (file)
@@ -1,2 +1,3 @@
-TEST_GEN_PROGS := test_unix_oob
+TEST_GEN_PROGS := test_unix_oob unix_connect
+
 include ../../lib.mk
diff --git a/tools/testing/selftests/net/af_unix/unix_connect.c b/tools/testing/selftests/net/af_unix/unix_connect.c
new file mode 100644 (file)
index 0000000..157e44e
--- /dev/null
@@ -0,0 +1,149 @@
+// SPDX-License-Identifier: GPL-2.0
+
+#define _GNU_SOURCE
+#include <sched.h>
+
+#include <stdio.h>
+#include <unistd.h>
+
+#include <sys/socket.h>
+#include <sys/un.h>
+
+#include "../../kselftest_harness.h"
+
+FIXTURE(unix_connect)
+{
+       int server, client;
+       int family;
+};
+
+FIXTURE_VARIANT(unix_connect)
+{
+       int type;
+       char sun_path[8];
+       int len;
+       int flags;
+       int err;
+};
+
+FIXTURE_VARIANT_ADD(unix_connect, stream_pathname)
+{
+       .type = SOCK_STREAM,
+       .sun_path = "test",
+       .len = 4 + 1,
+       .flags = 0,
+       .err = 0,
+};
+
+FIXTURE_VARIANT_ADD(unix_connect, stream_abstract)
+{
+       .type = SOCK_STREAM,
+       .sun_path = "\0test",
+       .len = 5,
+       .flags = 0,
+       .err = 0,
+};
+
+FIXTURE_VARIANT_ADD(unix_connect, stream_pathname_netns)
+{
+       .type = SOCK_STREAM,
+       .sun_path = "test",
+       .len = 4 + 1,
+       .flags = CLONE_NEWNET,
+       .err = 0,
+};
+
+FIXTURE_VARIANT_ADD(unix_connect, stream_abstract_netns)
+{
+       .type = SOCK_STREAM,
+       .sun_path = "\0test",
+       .len = 5,
+       .flags = CLONE_NEWNET,
+       .err = ECONNREFUSED,
+};
+
+FIXTURE_VARIANT_ADD(unix_connect, dgram_pathname)
+{
+       .type = SOCK_DGRAM,
+       .sun_path = "test",
+       .len = 4 + 1,
+       .flags = 0,
+       .err = 0,
+};
+
+FIXTURE_VARIANT_ADD(unix_connect, dgram_abstract)
+{
+       .type = SOCK_DGRAM,
+       .sun_path = "\0test",
+       .len = 5,
+       .flags = 0,
+       .err = 0,
+};
+
+FIXTURE_VARIANT_ADD(unix_connect, dgram_pathname_netns)
+{
+       .type = SOCK_DGRAM,
+       .sun_path = "test",
+       .len = 4 + 1,
+       .flags = CLONE_NEWNET,
+       .err = 0,
+};
+
+FIXTURE_VARIANT_ADD(unix_connect, dgram_abstract_netns)
+{
+       .type = SOCK_DGRAM,
+       .sun_path = "\0test",
+       .len = 5,
+       .flags = CLONE_NEWNET,
+       .err = ECONNREFUSED,
+};
+
+FIXTURE_SETUP(unix_connect)
+{
+       self->family = AF_UNIX;
+}
+
+FIXTURE_TEARDOWN(unix_connect)
+{
+       close(self->server);
+       close(self->client);
+
+       if (variant->sun_path[0])
+               remove("test");
+}
+
+#define offsetof(type, member) ((size_t)&((type *)0)->(member))
+
+TEST_F(unix_connect, test)
+{
+       socklen_t addrlen;
+       struct sockaddr_un addr = {
+               .sun_family = self->family,
+       };
+       int err;
+
+       self->server = socket(self->family, variant->type, 0);
+       ASSERT_NE(-1, self->server);
+
+       addrlen = offsetof(struct sockaddr_un, sun_path) + variant->len;
+       memcpy(&addr.sun_path, variant->sun_path, variant->len);
+
+       err = bind(self->server, (struct sockaddr *)&addr, addrlen);
+       ASSERT_EQ(0, err);
+
+       if (variant->type == SOCK_STREAM) {
+               err = listen(self->server, 32);
+               ASSERT_EQ(0, err);
+       }
+
+       err = unshare(variant->flags);
+       ASSERT_EQ(0, err);
+
+       self->client = socket(self->family, variant->type, 0);
+       ASSERT_LT(0, self->client);
+
+       err = connect(self->client, (struct sockaddr *)&addr, addrlen);
+       ASSERT_EQ(variant->err, err == -1 ? errno : 0);
+}
+
+TEST_HARNESS_MAIN