This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] posix: New function posix_spawn_file_actions_addchdir_np [BZ #17405]
- From: Adhemerval Zanella <adhemerval dot zanella at linaro dot org>
- To: Florian Weimer <fweimer at redhat dot com>
- Cc: libc-alpha at sourceware dot org
- Date: Thu, 1 Nov 2018 14:10:15 -0300
- Subject: Re: [PATCH] posix: New function posix_spawn_file_actions_addchdir_np [BZ #17405]
- References: <87zhv0g5t3.fsf@oldenburg.str.redhat.com> <6d2e3b16-12ca-0e37-de6b-886ab9ee49b2@linaro.org> <87pnvspi7z.fsf@oldenburg.str.redhat.com> <87a7mvn1f5.fsf@oldenburg.str.redhat.com>
On 30/10/2018 08:01, Florian Weimer wrote:
> * Florian Weimer:
>
>> Hmm, I have to think about it. Would you consider mere creation of a
>> file in the expected directory consider sufficient evidence that the
>> directory changes happen as expected?
>
> Here's an updated patch with a more elaborate test.
LGTM.
>
> Thanks,
> Florian
>
> 2018-10-30 Florian Weimer <fweimer@redhat.com>
>
> [BZ #17405]
> * posix/Makefile (routines): Add spawn_faction_addchdir.
> (tests): Add tst-spawn-chdir.
> * posix/Versions (GLIBC_2.29): Add
> posix_spawn_file_actions_addchdir_np.
> * posix/spawn_faction_addchdir.c: New file.
> * posix/spawn_faction_destroy.c
> (__posix_spawn_file_actions_destroy): Handle spawn_do_chdir.
> * posix/spawn.h (posix_spawn_file_actions_addchdir_np): Declare.
> * posix/spawn_int.h (struct __spawn_action): Add spawn_do_chdir,
> chdir_action.
> * posix/tst-spawn-chdir.c: New file.
> * sysdeps/posix/spawni.c (__spawni_child): Handle spawn_do_chdir.
> * sysdeps/unix/sysv/linux/spawni.c (__spawni_child): Likewise.
> * sysdeps/mach/hurd/i386/libc.abilist (GLIBC_2.29): Add
> posix_spawn_file_actions_addchdir_np.
> * sysdeps/unix/sysv/linux/aarch64/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/alpha/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/arm/libc.abilist (GLIBC_2.29): Likewise.
> * sysdeps/unix/sysv/linux/hppa/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/i386/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/ia64/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/m68k/coldfire/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/m68k/m680x0/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/microblaze/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/mips/mips32/fpu/libc.abilist
> (GLIBC_2.29): Likewise.
> * sysdeps/unix/sysv/linux/mips/mips32/nofpu/libc.abilist
> (GLIBC_2.29): Likewise.
> * sysdeps/unix/sysv/linux/mips/mips64/n32/libc.abilist
> (GLIBC_2.29): Likewise.
> * sysdeps/unix/sysv/linux/mips/mips64/n64/libc.abilist
> (GLIBC_2.29): Likewise.
> * sysdeps/unix/sysv/linux/nios2/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/powerpc/powerpc32/fpu/libc.abilist
> (GLIBC_2.29): Likewise.
> * sysdeps/unix/sysv/linux/powerpc/powerpc32/nofpu/libc.abilist
> (GLIBC_2.29): Likewise.
> * sysdeps/unix/sysv/linux/powerpc/powerpc64/libc-le.abilist
> (GLIBC_2.29): Likewise.
> * sysdeps/unix/sysv/linux/powerpc/powerpc64/libc.abilist
> (GLIBC_2.29): Likewise.
> * sysdeps/unix/sysv/linux/riscv/rv64/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/s390/s390-32/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/s390/s390-64/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/sh/libc.abilist (GLIBC_2.29): Likewise.
> * sysdeps/unix/sysv/linux/sparc/sparc32/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/sparc/sparc64/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/x86_64/64/libc.abilist (GLIBC_2.29):
> Likewise.
> * sysdeps/unix/sysv/linux/x86_64/x32/libc.abilist (GLIBC_2.29):
> Likewise.
>
> diff --git a/NEWS b/NEWS
> index 3d13b321a7..f488821af1 100644
> --- a/NEWS
> +++ b/NEWS
> @@ -30,6 +30,11 @@ Major new features:
> HTM state is saved and restore lazily (the state being saved even when the
> process actually does not use HTM).
>
> +* The posix_spawn_file_actions_addchdir_np function has been added,
> + enabling posix_spawn and posix_spawnp to run the new process in a
> + different directory. This is a GNU extension and similar to the
> + Solaris function of the same name.
> +
> Deprecated and removed features, and other changes affecting compatibility:
>
> * The glibc.tune tunable namespace has been renamed to glibc.cpu and the
> diff --git a/posix/Makefile b/posix/Makefile
> index 83162123f9..d67f68d0db 100644
> --- a/posix/Makefile
> +++ b/posix/Makefile
> @@ -55,6 +55,7 @@ routines := \
> pread pwrite pread64 pwrite64 \
> spawn_faction_init spawn_faction_destroy spawn_faction_addclose \
> spawn_faction_addopen spawn_faction_adddup2 spawn_valid_fd \
> + spawn_faction_addchdir \
> spawnattr_init spawnattr_destroy \
> spawnattr_getdefault spawnattr_setdefault \
> spawnattr_getflags spawnattr_setflags \
> @@ -96,7 +97,7 @@ tests := test-errno tstgetopt testfnm runtests runptests \
> tst-posix_fadvise tst-posix_fadvise64 \
> tst-sysconf-empty-chroot tst-glob_symlinks tst-fexecve \
> tst-glob-tilde test-ssize-max tst-spawn4 bug-regex37 \
> - bug-regex38 tst-regcomp-truncated
> + bug-regex38 tst-regcomp-truncated tst-spawn-chdir
> tests-internal := bug-regex5 bug-regex20 bug-regex33 \
> tst-rfc3484 tst-rfc3484-2 tst-rfc3484-3 \
> tst-glob_lstat_compat tst-spawn4-compat
> diff --git a/posix/Versions b/posix/Versions
> index cad4c23e8c..56ab921562 100644
> --- a/posix/Versions
> +++ b/posix/Versions
> @@ -137,6 +137,9 @@ libc {
> GLIBC_2.27 {
> glob; glob64;
> }
> + GLIBC_2.29 {
> + posix_spawn_file_actions_addchdir_np;
> + }
> GLIBC_PRIVATE {
> __libc_fork; __libc_pread; __libc_pwrite;
> __nanosleep_nocancel; __pause_nocancel;
> diff --git a/posix/spawn.h b/posix/spawn.h
> index aafb27611c..c84ee4bf72 100644
> --- a/posix/spawn.h
> +++ b/posix/spawn.h
> @@ -185,6 +185,13 @@ extern int posix_spawn_file_actions_adddup2 (posix_spawn_file_actions_t *
> __file_actions,
> int __fd, int __newfd) __THROW;
>
> +#ifdef __USE_GNU
> +/* Add an action changing the directory to PATH during spawn. This
> + affects the subsequent file actions. */
> +extern int posix_spawn_file_actions_addchdir_np (posix_spawn_file_actions_t *,
> + const char *__path) __THROW;
> +#endif
> +
> __END_DECLS
>
> #endif /* spawn.h */
> diff --git a/posix/spawn_faction_addchdir.c b/posix/spawn_faction_addchdir.c
> new file mode 100644
> index 0000000000..0e7a45a5f3
> --- /dev/null
> +++ b/posix/spawn_faction_addchdir.c
> @@ -0,0 +1,53 @@
> +/* Add a directory change to a file action list for posix_spawn.
> + Copyright (C) 2000-2018 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <http://www.gnu.org/licenses/>. */
> +
> +#include <errno.h>
> +#include <spawn.h>
> +#include <string.h>
> +
> +#include "spawn_int.h"
> +
> +int
> +posix_spawn_file_actions_addchdir_np (posix_spawn_file_actions_t *file_actions,
> + const char *path)
> +{
> + struct __spawn_action *rec;
> +
> + char *path_copy = __strdup (path);
> + if (path_copy == NULL)
> + return ENOMEM;
> +
> + /* Allocate more memory if needed. */
> + if (file_actions->__used == file_actions->__allocated
> + && __posix_spawn_file_actions_realloc (file_actions) != 0)
> + {
> + /* This can only mean we ran out of memory. */
> + free (path_copy);
> + return ENOMEM;
> + }
> +
> + /* Add the new value. */
> + rec = &file_actions->__actions[file_actions->__used];
> + rec->tag = spawn_do_chdir;
> + rec->action.chdir_action.path = path_copy;
> +
> + /* Account for the new entry. */
> + ++file_actions->__used;
> +
> + return 0;
> +}
> diff --git a/posix/spawn_faction_destroy.c b/posix/spawn_faction_destroy.c
> index 2a2de4e41d..05ca9dc13b 100644
> --- a/posix/spawn_faction_destroy.c
> +++ b/posix/spawn_faction_destroy.c
> @@ -33,6 +33,9 @@ __posix_spawn_file_actions_destroy (posix_spawn_file_actions_t *file_actions)
> case spawn_do_open:
> free (sa->action.open_action.path);
> break;
> + case spawn_do_chdir:
> + free (sa->action.chdir_action.path);
> + break;
> case spawn_do_close:
> case spawn_do_dup2:
> /* No cleanup required. */
> diff --git a/posix/spawn_int.h b/posix/spawn_int.h
> index 171f67c649..9db35553c2 100644
> --- a/posix/spawn_int.h
> +++ b/posix/spawn_int.h
> @@ -29,7 +29,8 @@ struct __spawn_action
> {
> spawn_do_close,
> spawn_do_dup2,
> - spawn_do_open
> + spawn_do_open,
> + spawn_do_chdir,
> } tag;
>
> union
> @@ -50,6 +51,10 @@ struct __spawn_action
> int oflag;
> mode_t mode;
> } open_action;
> + struct
> + {
> + char *path;
> + } chdir_action;
> } action;
> };
>
> diff --git a/posix/tst-spawn-chdir.c b/posix/tst-spawn-chdir.c
> new file mode 100644
> index 0000000000..dc14f2059b
> --- /dev/null
> +++ b/posix/tst-spawn-chdir.c
> @@ -0,0 +1,192 @@
> +/* Test the posix_spawn_file_actions_addchdir_np function.
> + Copyright (C) 2018 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <http://www.gnu.org/licenses/>. */
> +
> +#include <array_length.h>
> +#include <errno.h>
> +#include <fcntl.h>
> +#include <spawn.h>
> +#include <stdlib.h>
> +#include <string.h>
> +#include <support/check.h>
> +#include <support/support.h>
> +#include <support/temp_file.h>
> +#include <support/xstdio.h>
> +#include <support/xunistd.h>
> +#include <unistd.h>
> +
> +/* Reads the file at PATH, which must consist of exactly one line.
> + Removes the line terminator at the end of the file. */
> +static char *
> +read_one_line (const char *path)
> +{
> + FILE *fp = xfopen (path, "r");
> + char *buffer = NULL;
> + size_t length = 0;
> + ssize_t ret = getline (&buffer, &length, fp);
> + if (ferror (fp))
> + FAIL_EXIT1 ("getline: %m");
> + if (ret < 1)
> + FAIL_EXIT1 ("getline returned %zd", ret);
> + if (fgetc (fp) != EOF)
> + FAIL_EXIT1 ("trailing bytes in %s", path);
> + if (ferror (fp))
> + FAIL_EXIT1 ("fgetc: %m");
> + xfclose (fp);
> + if (buffer[ret - 1] != '\n')
> + FAIL_EXIT1 ("missing line terminator in %s", path);
> + buffer[ret - 1] = 0;
> + return buffer;
> +}
> +
> +/* Return the path to the "pwd" program. */
> +const char *
> +get_pwd_program (void)
> +{
> + const char *const paths[] = { "/bin/pwd", "/usr/bin/pwd" };
> + for (size_t i = 0; i < array_length (paths); ++i)
> + if (access (paths[i], X_OK) == 0)
> + return paths[i];
> + FAIL_EXIT1 ("cannot find pwd program");
> +}
> +
> +static int
> +do_test (void)
> +{
> + /* Directory for temporary file data. Each subtest uses a numeric
> + subdirectory. */
> + char *directory = support_create_temp_directory ("tst-spawn-chdir-");
> + {
> + /* Avoid symbolic links, to get more consistent behavior from the
> + pwd command. */
> + char *tmp = realpath (directory, NULL);
> + if (tmp == NULL)
> + FAIL_EXIT1 ("realpath: %m");
> + free (directory);
> + directory = tmp;
> + }
> +
> + char *original_cwd = get_current_dir_name ();
> + if (original_cwd == NULL)
> + FAIL_EXIT1 ("get_current_dir_name: %m");
> +
> + int iteration = 0;
> + for (int do_spawnp = 0; do_spawnp < 2; ++do_spawnp)
> + for (int do_overwrite = 0; do_overwrite < 2; ++do_overwrite)
> + {
> + ++iteration;
> + printf ("info: iteration=%d do_spawnp=%d do_overwrite=%d\n",
> + iteration, do_spawnp, do_overwrite);
> +
> + /* The "pwd" program runs in this directory. */
> + char *iteration_directory = xasprintf ("%s/%d", directory, iteration);
> + add_temp_file (iteration_directory);
> + xmkdir (iteration_directory, 0777);
> +
> + /* This file receives output from "pwd". */
> + char *output_file_path
> + = xasprintf ("%s/output-file", iteration_directory);
> + add_temp_file (output_file_path);
> +
> + /* This subdirectory is used for chdir ordering checks. */
> + char *subdir_path = xasprintf ("%s/subdir", iteration_directory);
> + add_temp_file (subdir_path);
> + xmkdir (subdir_path, 0777);
> +
> + /* Also used for checking the order of actions. */
> + char *probe_file_path
> + = xasprintf ("%s/subdir/probe-file", iteration_directory);
> + add_temp_file (probe_file_path);
> + TEST_COMPARE (access (probe_file_path, F_OK), -1);
> + TEST_COMPARE (errno, ENOENT);
> +
> + /* This symbolic link is used in a relative path with
> + posix_spawn. */
> + char *pwd_symlink_path
> + = xasprintf ("%s/subdir/pwd-symlink", iteration_directory);
> + xsymlink (get_pwd_program (), pwd_symlink_path);
> + add_temp_file (pwd_symlink_path);
> +
> + posix_spawn_file_actions_t actions;
> + TEST_COMPARE (posix_spawn_file_actions_init (&actions), 0);
> + TEST_COMPARE (posix_spawn_file_actions_addchdir_np
> + (&actions, subdir_path), 0);
> + TEST_COMPARE (posix_spawn_file_actions_addopen
> + (&actions, 3, /* Arbitrary unused descriptor. */
> + "probe-file",
> + O_WRONLY | O_CREAT | O_EXCL, 0777), 0);
> + TEST_COMPARE (posix_spawn_file_actions_addclose (&actions, 3), 0);
> + /* Run the actual in iteration_directory. */
> + TEST_COMPARE (posix_spawn_file_actions_addchdir_np (&actions, ".."), 0);
> + TEST_COMPARE (posix_spawn_file_actions_addopen
> + (&actions, STDOUT_FILENO, "output-file",
> + O_WRONLY | O_CREAT | O_EXCL, 0777), 0);
> +
> + /* Check that posix_spawn_file_actions_addchdir_np made a copy
> + of the path. */
> + if (do_overwrite)
> + subdir_path[0] = '\0';
> +
> + char *const argv[] = { (char *) "pwd", NULL };
> + char *const envp[] = { NULL } ;
> + pid_t pid;
> + if (do_spawnp)
> + TEST_COMPARE (posix_spawnp (&pid, "pwd", &actions,
> + NULL, argv, envp), 0);
> + else
> + TEST_COMPARE (posix_spawn (&pid, "subdir/pwd-symlink", &actions,
> + NULL, argv, envp), 0);
> + TEST_VERIFY (pid > 0);
> + int status;
> + xwaitpid (pid, &status, 0);
> + TEST_COMPARE (status, 0);
> +
> + /* Check that the current directory did not change. */
> + {
> + char *cwd = get_current_dir_name ();
> + if (cwd == NULL)
> + FAIL_EXIT1 ("get_current_dir_name: %m");
> + TEST_COMPARE_BLOB (original_cwd, strlen (original_cwd),
> + cwd, strlen (cwd));
> + free (cwd);
> + }
> +
> +
> + /* Check the output from "pwd". */
> + {
> + char *pwd = read_one_line (output_file_path);
> + TEST_COMPARE_BLOB (iteration_directory, strlen (iteration_directory),
> + pwd, strlen (pwd));
> + free (pwd);
> + }
> +
> + /* This file must now exist. */
> + TEST_COMPARE (access (probe_file_path, F_OK), 0);
> +
> + TEST_COMPARE (posix_spawn_file_actions_destroy (&actions), 0);
> + free (pwd_symlink_path);
> + free (probe_file_path);
> + free (subdir_path);
> + free (output_file_path);
> + }
> +
> + free (directory);
> +
> + return 0;
> +}
> +
> +#include <support/test-driver.c>
> diff --git a/sysdeps/mach/hurd/i386/libc.abilist b/sysdeps/mach/hurd/i386/libc.abilist
> index e3fc05137b..62674e9ba7 100644
> --- a/sysdeps/mach/hurd/i386/libc.abilist
> +++ b/sysdeps/mach/hurd/i386/libc.abilist
> @@ -2038,6 +2038,7 @@ GLIBC_2.27 wcstof64x_l F
> GLIBC_2.28 fcntl64 F
> GLIBC_2.28 renameat2 F
> GLIBC_2.28 statx F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/posix/spawni.c b/sysdeps/posix/spawni.c
> index b138ab4393..c42fbd590e 100644
> --- a/sysdeps/posix/spawni.c
> +++ b/sysdeps/posix/spawni.c
> @@ -209,6 +209,11 @@ __spawni_child (void *arguments)
> != action->action.dup2_action.newfd)
> goto fail;
> break;
> +
> + case spawn_do_chdir:
> + if (__chdir (action->action.chdir_action.path) != 0)
> + goto fail;
> + break;
> }
> }
> }
> diff --git a/sysdeps/unix/sysv/linux/aarch64/libc.abilist b/sysdeps/unix/sysv/linux/aarch64/libc.abilist
> index 15f966f841..e66c741d04 100644
> --- a/sysdeps/unix/sysv/linux/aarch64/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/aarch64/libc.abilist
> @@ -2138,3 +2138,4 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> diff --git a/sysdeps/unix/sysv/linux/alpha/libc.abilist b/sysdeps/unix/sysv/linux/alpha/libc.abilist
> index e22b91655d..8df162fe99 100644
> --- a/sysdeps/unix/sysv/linux/alpha/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/alpha/libc.abilist
> @@ -2033,6 +2033,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/arm/libc.abilist b/sysdeps/unix/sysv/linux/arm/libc.abilist
> index a231812eea..43c804f9dc 100644
> --- a/sysdeps/unix/sysv/linux/arm/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/arm/libc.abilist
> @@ -123,6 +123,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.4 _Exit F
> GLIBC_2.4 _IO_2_1_stderr_ D 0xa0
> GLIBC_2.4 _IO_2_1_stdin_ D 0xa0
> diff --git a/sysdeps/unix/sysv/linux/hppa/libc.abilist b/sysdeps/unix/sysv/linux/hppa/libc.abilist
> index 24b11b15c0..88b01c2e75 100644
> --- a/sysdeps/unix/sysv/linux/hppa/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/hppa/libc.abilist
> @@ -1880,6 +1880,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/i386/libc.abilist b/sysdeps/unix/sysv/linux/i386/libc.abilist
> index 9762c81365..6d02f31612 100644
> --- a/sysdeps/unix/sysv/linux/i386/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/i386/libc.abilist
> @@ -2045,6 +2045,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/ia64/libc.abilist b/sysdeps/unix/sysv/linux/ia64/libc.abilist
> index 50c94adff9..4249712611 100644
> --- a/sysdeps/unix/sysv/linux/ia64/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/ia64/libc.abilist
> @@ -1914,6 +1914,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/m68k/coldfire/libc.abilist b/sysdeps/unix/sysv/linux/m68k/coldfire/libc.abilist
> index f57be98e49..d47b808862 100644
> --- a/sysdeps/unix/sysv/linux/m68k/coldfire/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/m68k/coldfire/libc.abilist
> @@ -124,6 +124,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.4 _Exit F
> GLIBC_2.4 _IO_2_1_stderr_ D 0x98
> GLIBC_2.4 _IO_2_1_stdin_ D 0x98
> diff --git a/sysdeps/unix/sysv/linux/m68k/m680x0/libc.abilist b/sysdeps/unix/sysv/linux/m68k/m680x0/libc.abilist
> index ddc7ebca67..d5e38308be 100644
> --- a/sysdeps/unix/sysv/linux/m68k/m680x0/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/m68k/m680x0/libc.abilist
> @@ -1989,6 +1989,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/microblaze/libc.abilist b/sysdeps/unix/sysv/linux/microblaze/libc.abilist
> index daed511f00..8596b84399 100644
> --- a/sysdeps/unix/sysv/linux/microblaze/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/microblaze/libc.abilist
> @@ -2130,3 +2130,4 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> diff --git a/sysdeps/unix/sysv/linux/mips/mips32/fpu/libc.abilist b/sysdeps/unix/sysv/linux/mips/mips32/fpu/libc.abilist
> index dda9797e0e..88e0f896d5 100644
> --- a/sysdeps/unix/sysv/linux/mips/mips32/fpu/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/mips/mips32/fpu/libc.abilist
> @@ -1967,6 +1967,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/mips/mips32/nofpu/libc.abilist b/sysdeps/unix/sysv/linux/mips/mips32/nofpu/libc.abilist
> index 42e930d565..aff7462c34 100644
> --- a/sysdeps/unix/sysv/linux/mips/mips32/nofpu/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/mips/mips32/nofpu/libc.abilist
> @@ -1965,6 +1965,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/mips/mips64/n32/libc.abilist b/sysdeps/unix/sysv/linux/mips/mips64/n32/libc.abilist
> index f89b44f3a1..71d82444aa 100644
> --- a/sysdeps/unix/sysv/linux/mips/mips64/n32/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/mips/mips64/n32/libc.abilist
> @@ -1973,6 +1973,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/mips/mips64/n64/libc.abilist b/sysdeps/unix/sysv/linux/mips/mips64/n64/libc.abilist
> index 1486af11d7..de6c53d293 100644
> --- a/sysdeps/unix/sysv/linux/mips/mips64/n64/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/mips/mips64/n64/libc.abilist
> @@ -1968,6 +1968,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/nios2/libc.abilist b/sysdeps/unix/sysv/linux/nios2/libc.abilist
> index aab1c44e46..e724bab9fb 100644
> --- a/sysdeps/unix/sysv/linux/nios2/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/nios2/libc.abilist
> @@ -2171,3 +2171,4 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> diff --git a/sysdeps/unix/sysv/linux/powerpc/powerpc32/fpu/libc.abilist b/sysdeps/unix/sysv/linux/powerpc/powerpc32/fpu/libc.abilist
> index 45839ed25b..e9ecbccb71 100644
> --- a/sysdeps/unix/sysv/linux/powerpc/powerpc32/fpu/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/powerpc/powerpc32/fpu/libc.abilist
> @@ -1993,6 +1993,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/powerpc/powerpc32/nofpu/libc.abilist b/sysdeps/unix/sysv/linux/powerpc/powerpc32/nofpu/libc.abilist
> index 344e5ae87c..da83ea6028 100644
> --- a/sysdeps/unix/sysv/linux/powerpc/powerpc32/nofpu/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/powerpc/powerpc32/nofpu/libc.abilist
> @@ -1997,6 +1997,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/powerpc/powerpc64/libc-le.abilist b/sysdeps/unix/sysv/linux/powerpc/powerpc64/libc-le.abilist
> index e26930e728..4535b40d15 100644
> --- a/sysdeps/unix/sysv/linux/powerpc/powerpc64/libc-le.abilist
> +++ b/sysdeps/unix/sysv/linux/powerpc/powerpc64/libc-le.abilist
> @@ -2228,3 +2228,4 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> diff --git a/sysdeps/unix/sysv/linux/powerpc/powerpc64/libc.abilist b/sysdeps/unix/sysv/linux/powerpc/powerpc64/libc.abilist
> index 8c1781adf4..65725de4f0 100644
> --- a/sysdeps/unix/sysv/linux/powerpc/powerpc64/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/powerpc/powerpc64/libc.abilist
> @@ -123,6 +123,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 _Exit F
> GLIBC_2.3 _IO_2_1_stderr_ D 0xe0
> GLIBC_2.3 _IO_2_1_stdin_ D 0xe0
> diff --git a/sysdeps/unix/sysv/linux/riscv/rv64/libc.abilist b/sysdeps/unix/sysv/linux/riscv/rv64/libc.abilist
> index daacceebd6..bbb3c4a8e7 100644
> --- a/sysdeps/unix/sysv/linux/riscv/rv64/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/riscv/rv64/libc.abilist
> @@ -2100,3 +2100,4 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> diff --git a/sysdeps/unix/sysv/linux/s390/s390-32/libc.abilist b/sysdeps/unix/sysv/linux/s390/s390-32/libc.abilist
> index 3a5ec2a1e4..e85ac2a178 100644
> --- a/sysdeps/unix/sysv/linux/s390/s390-32/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/s390/s390-32/libc.abilist
> @@ -2002,6 +2002,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/s390/s390-64/libc.abilist b/sysdeps/unix/sysv/linux/s390/s390-64/libc.abilist
> index 04a539ff02..d56931022c 100644
> --- a/sysdeps/unix/sysv/linux/s390/s390-64/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/s390/s390-64/libc.abilist
> @@ -1908,6 +1908,7 @@ GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> GLIBC_2.29 __fentry__ F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/sh/libc.abilist b/sysdeps/unix/sysv/linux/sh/libc.abilist
> index 1f4e648baa..ff939a15c4 100644
> --- a/sysdeps/unix/sysv/linux/sh/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/sh/libc.abilist
> @@ -1884,6 +1884,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/sparc/sparc32/libc.abilist b/sysdeps/unix/sysv/linux/sparc/sparc32/libc.abilist
> index d308ac8b05..64fa9e10a5 100644
> --- a/sysdeps/unix/sysv/linux/sparc/sparc32/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/sparc/sparc32/libc.abilist
> @@ -1996,6 +1996,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/sparc/sparc64/libc.abilist b/sysdeps/unix/sysv/linux/sparc/sparc64/libc.abilist
> index 6731ebfc28..db909d1506 100644
> --- a/sysdeps/unix/sysv/linux/sparc/sparc64/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/sparc/sparc64/libc.abilist
> @@ -1937,6 +1937,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/spawni.c b/sysdeps/unix/sysv/linux/spawni.c
> index 85239cedbf..9f3a137b5c 100644
> --- a/sysdeps/unix/sysv/linux/spawni.c
> +++ b/sysdeps/unix/sysv/linux/spawni.c
> @@ -258,6 +258,11 @@ __spawni_child (void *arguments)
> != action->action.dup2_action.newfd)
> goto fail;
> break;
> +
> + case spawn_do_chdir:
> + if (__chdir (action->action.chdir_action.path) != 0)
> + goto fail;
> + break;
> }
> }
> }
> diff --git a/sysdeps/unix/sysv/linux/x86_64/64/libc.abilist b/sysdeps/unix/sysv/linux/x86_64/64/libc.abilist
> index 816e4a7426..3b175f104b 100644
> --- a/sysdeps/unix/sysv/linux/x86_64/64/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/x86_64/64/libc.abilist
> @@ -1895,6 +1895,7 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
> GLIBC_2.3 __ctype_b_loc F
> GLIBC_2.3 __ctype_tolower_loc F
> GLIBC_2.3 __ctype_toupper_loc F
> diff --git a/sysdeps/unix/sysv/linux/x86_64/x32/libc.abilist b/sysdeps/unix/sysv/linux/x86_64/x32/libc.abilist
> index 6fee16a850..1b57710477 100644
> --- a/sysdeps/unix/sysv/linux/x86_64/x32/libc.abilist
> +++ b/sysdeps/unix/sysv/linux/x86_64/x32/libc.abilist
> @@ -2146,3 +2146,4 @@ GLIBC_2.28 thrd_current F
> GLIBC_2.28 thrd_equal F
> GLIBC_2.28 thrd_sleep F
> GLIBC_2.28 thrd_yield F
> +GLIBC_2.29 posix_spawn_file_actions_addchdir_np F
>