Re: [PATCH] tests/unit/test-util-sockets: Use g_file_open_tmp() to create temp file

[Richard Henderson]

On 12/24/21 3:45 PM, Philippe Mathieu-Daudé wrote:

Similarly to commit e63ed64c6d1 ("tests/qtest/virtio-net-failover:
Use g_file_open_tmp() to create temporary file"), avoid calling
g_test_rand_int() before g_test_init(): use g_file_open_tmp().

Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com>
---
  tests/unit/test-util-sockets.c | 6 ++++--
  1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/tests/unit/test-util-sockets.c b/tests/unit/test-util-sockets.c
index 72b92465298..896247e3ed3 100644
--- a/tests/unit/test-util-sockets.c
+++ b/tests/unit/test-util-sockets.c
@@ -305,9 +305,11 @@ static void test_socket_unix_abstract(void)
      };
      int i;

+ i = g_file_open_tmp("unix-XXXXXX", &addr.u.q_unix.path, NULL);

+    g_assert_true(i >= 0);

Just g_assert.  Otherwise,
Reviewed-by: Richard Henderson <richard.hender...@linaro.org>


r~

+    close(i);
+
      addr.type = SOCKET_ADDRESS_TYPE_UNIX;
-    addr.u.q_unix.path = g_strdup_printf("unix-%d-%u",
-                                         getpid(), g_random_int());
      addr.u.q_unix.has_abstract = true;
      addr.u.q_unix.abstract = true;
      addr.u.q_unix.has_tight = false;