[PATCH] posix: Use unique PID file in tst-spawn3 and tst-spawn3-pidfd

[Sam James]

Florian Weimer <fweimer@redhat.com> writes:

> The test sources are built twice and the two tests may run in parallel.
> They cannot use the same PID file.

Thanks.

Reviewed-by: Sam James <sam@gentoo.org>

>
> ---
>  posix/tst-spawn3.c | 11 ++++++++---
>  1 file changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/posix/tst-spawn3.c b/posix/tst-spawn3.c
> index 70a5a8ba49..8b946cbb4d 100644
> --- a/posix/tst-spawn3.c
> +++ b/posix/tst-spawn3.c
> @@ -30,6 +30,7 @@
>  #include <intprops.h>
>  
>  #include <support/check.h>
> +#include <support/support.h>
>  #include <support/temp_file.h>
>  #include <support/xunistd.h>
>  #include <tst-spawn.h>
> @@ -84,8 +85,12 @@ do_test (void)
>    posix_spawn_file_actions_t a;
>    TEST_COMPARE (posix_spawn_file_actions_init (&a), 0);
>  
> -  /* Executes a /bin/sh echo $$ 2>&1 > ${objpfx}tst-spawn3.pid .  */
> -  const char pidfile[] = OBJPFX "tst-spawn3.pid";
> +  /* Executes a /bin/sh echo $$ 2>&1 > ${objpfx}tst-spawn3-$$.pid .
> +     Embed this PID into the path because the test is built multiple
> +     times and can run in parallel.  */
> +  char *pidfile = xasprintf ("%s/tst-spawn3-%d.pid",
> +			     OBJPFX, (int) getpid ());
> +  add_temp_file (pidfile);
>    TEST_COMPARE (posix_spawn_file_actions_addopen (&a, STDOUT_FILENO, pidfile,
>  						  O_WRONLY| O_CREAT | O_TRUNC,
>  						  0644),
> @@ -145,7 +150,7 @@ do_test (void)
>    ssize_t n = read (pidfd, buf, sizeof (buf));
>    TEST_VERIFY (n < sizeof buf && n >= 0);
>  
> -  xunlink (pidfile);
> +  free (pidfile);
>  
>    /* We only expect to read the PID.  */
>    char *endp;
>
> base-commit: 20b13fa08b2831ee67fcc92edf2d89288bcc2c93
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 418 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/libc-alpha/attachments/20260709/657a58ae/attachment.sig>