On 20/12/2021 21.02, Philippe Mathieu-Daudé wrote:
On 12/20/21 20:26, Richard Henderson wrote:
On 12/20/21 2:27 AM, Thomas Huth wrote:
const gchar *tmpdir = g_get_tmp_dir();
gchar *tmpfile = g_strdup_printf("%s/failover_test_migrate-%u-%u",
- tmpdir, getpid(),
g_test_rand_int());
+ tmpdir, getpid(), g_random_int());
Random numbers plus pid are irrelevant, because you still don't have
guaranteed uniqueness -- think stale files in /tmp.
Use g_file_open_tmp.
Another use in test_socket_unix_abstract(),
tests/unit/test-util-sockets.c.
Using g_file_open_tmp is certainly better ... but the tests are currently
written in a way where they require the file name of the temporary file - so
switching to g_file_open_tmp() (which only provides a file handle) certainly
would need some rewrite here... Thus I'd suggest to go first with this patch
to silence the Assert messages, and then to clean this up properly later.
Thomas
