[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 03/11] test-util-sockets: Clean up SocketAddress construction
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH 03/11] test-util-sockets: Clean up SocketAddress construction |
Date: |
Fri, 30 Oct 2020 09:36:32 +0000 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
On Thu, Oct 29, 2020 at 02:38:25PM +0100, Markus Armbruster wrote:
> The thread functions build the SocketAddress from global variable
> @abstract_sock_name and the tight flag passed as pointer
> argument (either NULL or (gpointer)1). There is no need for such
> hackery; simply pass the SocketAddress instead.
>
> While there, dumb down g_rand_int_range() to g_random_int().
>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
> tests/test-util-sockets.c | 62 +++++++++++++++------------------------
> 1 file changed, 24 insertions(+), 38 deletions(-)
>
> diff --git a/tests/test-util-sockets.c b/tests/test-util-sockets.c
> index 9d317e73a6..b1b5628bd5 100644
> --- a/tests/test-util-sockets.c
> +++ b/tests/test-util-sockets.c
> @@ -230,25 +230,14 @@ static void test_socket_fd_pass_num_nocli(void)
> #endif
>
> #ifdef __linux__
> -static gchar *abstract_sock_name;
> -
> static gpointer unix_server_thread_func(gpointer user_data)
> {
> - SocketAddress addr;
Keep this but as a pointer, and initialize it to "user_data",
so that it is clear what data type this parameter is expected
to be.
> - Error *err = NULL;
> - int fd = -1;
> - int connfd = -1;
> + int fd;
> + int connfd;
> struct sockaddr_un un;
> socklen_t len = sizeof(un);
>
> - addr.type = SOCKET_ADDRESS_TYPE_UNIX;
> - addr.u.q_unix.path = abstract_sock_name;
> - addr.u.q_unix.has_tight = true;
> - addr.u.q_unix.tight = user_data != NULL;
> - addr.u.q_unix.has_abstract = true;
> - addr.u.q_unix.abstract = true;
> -
> - fd = socket_listen(&addr, 1, &err);
> + fd = socket_listen(user_data, 1, &error_abort);
Then keep this as passing "addr"
>
> static gpointer unix_client_thread_func(gpointer user_data)
> {
> - SocketAddress addr;
Same note here
> - Error *err = NULL;
> - int fd = -1;
> -
> - addr.type = SOCKET_ADDRESS_TYPE_UNIX;
> - addr.u.q_unix.path = abstract_sock_name;
> - addr.u.q_unix.has_tight = true;
> - addr.u.q_unix.tight = user_data != NULL;
> - addr.u.q_unix.has_abstract = true;
> - addr.u.q_unix.abstract = true;
> -
> - fd = socket_connect(&addr, &err);
> + int fd;
>
> + fd = socket_connect(user_data, &error_abort);
> g_assert_cmpint(fd, >=, 0);
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|