From: Geliang Tang <[email protected]>

This patch uses network helper start_server_str() in skc_to_unix_sock
test in skc_to_unix_sock.c to simplify the code.

This is the first time to use start_server_str() for a AF_UNIX socket.
The first "@" of "sun_path" needs to be skipped by make_sockaddr(), so
path string "sock_path + 1" ("skc_to_unix_sock") is passed to the helper,
not "sock_path" ("@skc_to_unix_sock").

Signed-off-by: Geliang Tang <[email protected]>
---
 .../bpf/prog_tests/skc_to_unix_sock.c         | 22 +++++--------------
 1 file changed, 6 insertions(+), 16 deletions(-)

diff --git a/tools/testing/selftests/bpf/prog_tests/skc_to_unix_sock.c 
b/tools/testing/selftests/bpf/prog_tests/skc_to_unix_sock.c
index 3eefdfed1db9..4c23409a3072 100644
--- a/tools/testing/selftests/bpf/prog_tests/skc_to_unix_sock.c
+++ b/tools/testing/selftests/bpf/prog_tests/skc_to_unix_sock.c
@@ -4,13 +4,16 @@
 #include <test_progs.h>
 #include <sys/un.h>
 #include "test_skc_to_unix_sock.skel.h"
+#include "network_helpers.h"
 
 static const char *sock_path = "@skc_to_unix_sock";
 
 void test_skc_to_unix_sock(void)
 {
+       struct network_helper_opts opts = {
+               .backlog = 1,
+       };
        struct test_skc_to_unix_sock *skel;
-       struct sockaddr_un sockaddr;
        int err, sockfd = 0;
 
        skel = test_skc_to_unix_sock__open();
@@ -28,21 +31,8 @@ void test_skc_to_unix_sock(void)
                goto cleanup;
 
        /* trigger unix_listen */
-       sockfd = socket(AF_UNIX, SOCK_STREAM, 0);
-       if (!ASSERT_GT(sockfd, 0, "socket failed"))
-               goto cleanup;
-
-       memset(&sockaddr, 0, sizeof(sockaddr));
-       sockaddr.sun_family = AF_UNIX;
-       strncpy(sockaddr.sun_path, sock_path, strlen(sock_path));
-       sockaddr.sun_path[0] = '\0';
-
-       err = bind(sockfd, (struct sockaddr *)&sockaddr, sizeof(sockaddr));
-       if (!ASSERT_OK(err, "bind failed"))
-               goto cleanup;
-
-       err = listen(sockfd, 1);
-       if (!ASSERT_OK(err, "listen failed"))
+       sockfd = start_server_str(AF_UNIX, SOCK_STREAM, sock_path + 1, 0, 
&opts);
+       if (!ASSERT_OK_FD(sockfd, "start_server_str"))
                goto cleanup;
 
        ASSERT_EQ(strcmp(skel->bss->path, sock_path), 0, "bpf_skc_to_unix_sock 
failed");
-- 
2.43.0


Reply via email to