From owner-freebsd-standards@FreeBSD.ORG Mon Mar 24 15:40:04 2008 Return-Path: Delivered-To: freebsd-standards@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 2AB3F1065673 for ; Mon, 24 Mar 2008 15:40:04 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id F12078FC35 for ; Mon, 24 Mar 2008 15:40:03 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) by freefall.freebsd.org (8.14.2/8.14.2) with ESMTP id m2OFe30Q016612 for ; Mon, 24 Mar 2008 15:40:03 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.2/8.14.1/Submit) id m2OFe3i2016611; Mon, 24 Mar 2008 15:40:03 GMT (envelope-from gnats) Resent-Date: Mon, 24 Mar 2008 15:40:03 GMT Resent-Message-Id: <200803241540.m2OFe3i2016611@freefall.freebsd.org> Resent-From: FreeBSD-gnats-submit@FreeBSD.org (GNATS Filer) Resent-To: freebsd-standards@FreeBSD.org Resent-Reply-To: FreeBSD-gnats-submit@FreeBSD.org, Ed Schouten Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id D879B106566B for ; Mon, 24 Mar 2008 15:38:26 +0000 (UTC) (envelope-from ed@hoeg.nl) Received: from palm.hoeg.nl (mx0.hoeg.nl [IPv6:2001:610:652::211]) by mx1.freebsd.org (Postfix) with ESMTP id CD1EB8FC24 for ; Mon, 24 Mar 2008 15:38:24 +0000 (UTC) (envelope-from ed@hoeg.nl) Received: by palm.hoeg.nl (Postfix, from userid 1000) id E02241CC50; Mon, 24 Mar 2008 16:38:23 +0100 (CET) Message-Id: <20080324153823.E02241CC50@palm.hoeg.nl> Date: Mon, 24 Mar 2008 16:38:23 +0100 (CET) From: Ed Schouten To: FreeBSD-gnats-submit@FreeBSD.org X-Send-Pr-Version: 3.113 Cc: Subject: standards/122051: Add posix_spawn(3) X-BeenThere: freebsd-standards@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Ed Schouten List-Id: Standards compliance List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 24 Mar 2008 15:40:04 -0000 >Number: 122051 >Category: standards >Synopsis: Add posix_spawn(3) >Confidential: no >Severity: non-critical >Priority: medium >Responsible: freebsd-standards >State: open >Quarter: >Keywords: >Date-Required: >Class: sw-bug >Submitter-Id: current-users >Arrival-Date: Mon Mar 24 15:40:03 UTC 2008 >Closed-Date: >Last-Modified: >Originator: Ed Schouten >Release: FreeBSD 8.0-CURRENT amd64 >Organization: >Environment: System: FreeBSD flippo.80386.nl 8.0-CURRENT FreeBSD 8.0-CURRENT #0: Sun Mar 23 13:07:36 CET 2008 ed@flippo.80386.nl:/usr/obj/datadump/home/ed/p4/mpsafetty/sys/FLIPPO amd64 >Description: POSIX describes routines called posix_spawn() and posix_spawnp(), which can be used as replacements for exec/fork in a lot of cases. It doesn't really matter for FreeBSD, but there are some operating systems that want to implement POSIX routines, but don't use the MMU, which means forking is really expensive. FreeBSD doesn't have the routines in libc, which is bad, because we want to be as POSIX as possible. ;-) >How-To-Repeat: >Fix: The following patch adds posix_spawn() and posix_spawnp() to libc. I had to add yet another exec routine, namely execvpe(). We need to be able to override the environment, yet make it possible to search through $PATH. It isn't possible to make the structures opague, which is a pity with respect to binary compatibility. That's why I've decided to take the easy path by inlining most of the get/set routines. This patch still misses a manual page. --- include/Makefile +++ include/Makefile @@ -19,7 +19,7 @@ printf.h proc_service.h pthread.h \ pthread_np.h pwd.h ranlib.h readpassphrase.h regex.h regexp.h \ res_update.h resolv.h runetype.h search.h setjmp.h sgtty.h \ - signal.h stab.h \ + signal.h spawn.h stab.h \ stdbool.h stddef.h stdio.h stdlib.h string.h stringlist.h \ strings.h sysexits.h tar.h tgmath.h \ time.h timeconv.h timers.h ttyent.h \ --- include/spawn.h +++ include/spawn.h @@ -0,0 +1,206 @@ +/*- + * Copyright (c) 2008 Ed Schouten + * All rights reserved. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions + * are met: + * 1. Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * 2. Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in the + * documentation and/or other materials provided with the distribution. + * + * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND + * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE + * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE + * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT + * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY + * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF + * SUCH DAMAGE. + * + * $FreeBSD$ + */ + +#ifndef _SPAWN_H_ +#define _SPAWN_H_ + +#include +#include +#include +#include +#include + +#ifndef _MODE_T_DECLARED +typedef __mode_t mode_t; +#define _MODE_T_DECLARED +#endif + +#ifndef _PID_T_DECLARED +typedef __pid_t pid_t; +#define _PID_T_DECLARED +#endif + +#ifndef _SIGSET_T_DECLARED +#define _SIGSET_T_DECLARED +typedef __sigset_t sigset_t; +#endif + +struct sched_param; + +typedef struct { + short sa_flags; + pid_t sa_pgroup; + struct sched_param sa_schedparam; + int sa_schedpolicy; + sigset_t sa_sigdefault; + sigset_t sa_sigmask; +} posix_spawnattr_t; + +typedef struct { + STAILQ_HEAD(, __posix_spawn_file_actions_entry_t) fa_list; +} posix_spawn_file_actions_t; + +#define POSIX_SPAWN_RESETIDS 0x01 +#define POSIX_SPAWN_SETPGROUP 0x02 +#define POSIX_SPAWN_SETSCHEDPARAM 0x04 +#define POSIX_SPAWN_SETSCHEDULER 0x08 +#define POSIX_SPAWN_SETSIGDEF 0x10 +#define POSIX_SPAWN_SETSIGMASK 0x20 + +/* + * Spawn routines + */ +int posix_spawn(pid_t * __restrict, const char * __restrict, + const posix_spawn_file_actions_t *, const posix_spawnattr_t * __restrict, + char * const [__restrict], char * const [__restrict]); +int posix_spawnp(pid_t * __restrict, const char * __restrict, + const posix_spawn_file_actions_t *, const posix_spawnattr_t * __restrict, + char * const [__restrict], char * const [__restrict]); + +/* + * File descriptor actions + */ +int posix_spawn_file_actions_init(posix_spawn_file_actions_t *); +int posix_spawn_file_actions_destroy(posix_spawn_file_actions_t *); + +int posix_spawn_file_actions_addopen(posix_spawn_file_actions_t * __restrict, + int, const char * __restrict, int, mode_t); +int posix_spawn_file_actions_adddup2(posix_spawn_file_actions_t *, int, int); +int posix_spawn_file_actions_addclose(posix_spawn_file_actions_t *, int); + +/* + * Spawn attributes + */ + +static __inline int +posix_spawnattr_init(posix_spawnattr_t *sa) +{ + sa->sa_flags = 0; + return (0); +} + +static __inline int +posix_spawnattr_destroy(posix_spawnattr_t *sa) +{ + return (0); +} + +static __inline int +posix_spawnattr_getflags(const posix_spawnattr_t * __restrict sa, + short * __restrict flags) +{ + *flags = sa->sa_flags; + return (0); +} + +static __inline int +posix_spawnattr_getpgroup(const posix_spawnattr_t * __restrict sa, + pid_t * __restrict pgroup) +{ + *pgroup = sa->sa_pgroup; + return (0); +} + +static __inline int +posix_spawnattr_getschedparam(const posix_spawnattr_t * __restrict sa, + struct sched_param * __restrict schedparam) +{ + *schedparam = sa->sa_schedparam; + return (0); +} + +static __inline int +posix_spawnattr_getschedpolicy(const posix_spawnattr_t * __restrict sa, + int * __restrict schedpolicy) +{ + *schedpolicy = sa->sa_schedpolicy; + return (0); +} + +static __inline int +posix_spawnattr_getsigdefault(const posix_spawnattr_t * __restrict sa, + sigset_t * __restrict sigdefault) +{ + *sigdefault = sa->sa_sigdefault; + return (0); +} + +static __inline int +posix_spawnattr_getsigmask(const posix_spawnattr_t * __restrict sa, + sigset_t * __restrict sigmask) +{ + *sigmask = sa->sa_sigmask; + return (0); +} + +static __inline int +posix_spawnattr_setflags(posix_spawnattr_t *sa, short flags) +{ + sa->sa_flags = flags; + return (0); +} + +static __inline int +posix_spawnattr_setpgroup(posix_spawnattr_t *sa, pid_t pgroup) +{ + sa->sa_pgroup = pgroup; + return (0); +} + +static __inline int +posix_spawnattr_setschedparam(posix_spawnattr_t *sa __restrict, + const struct sched_param * __restrict schedparam) +{ + sa->sa_schedparam = *schedparam; + return (0); +} + +static __inline int +posix_spawnattr_setschedpolicy(posix_spawnattr_t *sa, int schedpolicy) +{ + sa->sa_schedpolicy = schedpolicy; + return (0); +} + +static __inline int +posix_spawnattr_setsigdefault(posix_spawnattr_t * __restrict sa, + const sigset_t * __restrict sigdefault) +{ + sa->sa_sigdefault = *sigdefault; + return (0); +} + +static __inline int +posix_spawnattr_setsigmask(posix_spawnattr_t * __restrict sa, + const sigset_t * __restrict sigmask) +{ + sa->sa_sigmask = *sigmask; + return (0); +} + +#endif /* !_SPAWN_H_ */ --- include/unistd.h +++ include/unistd.h @@ -335,6 +335,7 @@ int execv(const char *, char * const *); int execve(const char *, char * const *, char * const *); int execvp(const char *, char * const *); +int execvpe(const char *, char * const *, char * const *); pid_t fork(void); long fpathconf(int, int); char *getcwd(char *, size_t); --- lib/libc/gen/Makefile.inc +++ lib/libc/gen/Makefile.inc @@ -21,7 +21,7 @@ initgroups.c isatty.c isinf.c isnan.c jrand48.c lcong48.c \ lockf.c lrand48.c mrand48.c nftw.c nice.c \ nlist.c nrand48.c opendir.c \ - pause.c pmadvise.c popen.c pselect.c \ + pause.c pmadvise.c popen.c posix_spawn.c pselect.c \ psignal.c pw_scan.c pwcache.c \ raise.c readdir.c readpassphrase.c rewinddir.c \ scandir.c seed48.c seekdir.c sem.c semctl.c \ @@ -80,7 +80,7 @@ err.3 verr.3 err.3 verrc.3 err.3 verrx.3 err.3 vwarn.3 err.3 vwarnc.3 \ err.3 vwarnx.3 err.3 warnc.3 err.3 warn.3 err.3 warnx.3 MLINKS+=exec.3 execl.3 exec.3 execle.3 exec.3 execlp.3 exec.3 exect.3 \ - exec.3 execv.3 exec.3 execvp.3 exec.3 execvP.3 + exec.3 execv.3 exec.3 execvp.3 exec.3 execvpe.3 exec.3 execvP.3 MLINKS+=fpclassify.3 finite.3 fpclassify.3 finitef.3 \ fpclassify.3 isfinite.3 fpclassify.3 isinf.3 fpclassify.3 isnan.3 \ fpclassify.3 isnormal.3 --- lib/libc/gen/Symbol.map +++ lib/libc/gen/Symbol.map @@ -118,6 +118,7 @@ execlp; execv; execvp; + execvpe; execvP; fmtcheck; fmtmsg; @@ -220,9 +221,16 @@ nrand48; opendir; pause; - posix_madvise; - popen; pclose; + popen; + posix_madvise; + posix_spawn; + posix_spawn_file_actions_addclose; + posix_spawn_file_actions_adddup2; + posix_spawn_file_actions_addopen; + posix_spawn_file_actions_destroy; + posix_spawn_file_actions_init; + posix_spawnp; shm_open; shm_unlink; pselect; --- lib/libc/gen/exec.3 +++ lib/libc/gen/exec.3 @@ -38,6 +38,7 @@ .Nm exect , .Nm execv , .Nm execvp , +.Nm execvpe , .Nm execvP .Nd execute a file .Sh LIBRARY @@ -64,6 +65,8 @@ .Ft int .Fn execvp "const char *file" "char *const argv[]" .Ft int +.Fn execvpe "const char *file" "char *const argv[]" "char *const envp[]" +.Ft int .Fn execvP "const char *file" "const char *search_path" "char *const argv[]" .Sh DESCRIPTION The @@ -118,9 +121,10 @@ pointer. .Pp The -.Fn execle -and +.Fn execle , .Fn exect +and +.Fn execvpe functions also specify the environment of the executed process by following the .Dv NULL @@ -142,6 +146,7 @@ The functions .Fn execlp , .Fn execvp , +.Fn execvpe , and .Fn execvP will duplicate the actions of the shell in searching for an executable file @@ -152,6 +157,7 @@ .Fn execlp and .Fn execvp , +.Fn execvpe , search path is the path specified in the environment by .Dq Ev PATH variable. @@ -277,7 +283,8 @@ .Fn execl , .Fn execle , .Fn execlp , -.Fn execvp +.Fn execvp , +.Fn execvpe and .Fn execvP functions @@ -319,3 +326,7 @@ .Fn execvP function first appeared in .Fx 5.2 . +The +.Fn execvpe +function first appeared in +.Fx 8.0 . --- lib/libc/gen/exec.c +++ lib/libc/gen/exec.c @@ -140,20 +140,15 @@ int execvp(const char *name, char * const *argv) { - const char *path; - - /* Get the path we're searching. */ - if ((path = getenv("PATH")) == NULL) - path = _PATH_DEFPATH; - - return (execvP(name, path, argv)); + return (execvpe(name, argv, environ)); } -int -execvP(name, path, argv) +static int +execvPe(name, path, argv, envp) const char *name; const char *path; char * const *argv; + char * const *envp; { char **memp; int cnt, lp, ln; @@ -269,3 +264,21 @@ done: return (-1); } + +int +execvP(const char *name, const char *path, char * const argv[]) +{ + return execvPe(name, path, argv, environ); +} + +int +execvpe(const char *name, char * const argv[], char * const envp[]) +{ + const char *path; + + /* Get the path we're searching. */ + if ((path = getenv("PATH")) == NULL) + path = _PATH_DEFPATH; + + return (execvPe(name, path, argv, envp)); +} --- lib/libc/gen/posix_spawn.c +++ lib/libc/gen/posix_spawn.c @@ -0,0 +1,301 @@ +/*- + * Copyright (c) 2008 Ed Schouten + * All rights reserved. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions + * are met: + * 1. Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * 2. Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in the + * documentation and/or other materials provided with the distribution. + * + * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND + * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE + * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE + * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT + * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY + * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF + * SUCH DAMAGE. + */ + +#include +__FBSDID("$FreeBSD$"); + +#include "namespace.h" + +#include +#include +#include +#include +#include +#include +#include "un-namespace.h" + +extern char **environ; + +typedef struct __posix_spawn_file_actions_entry_t { + STAILQ_ENTRY(__posix_spawn_file_actions_entry_t) fae_list; + enum { FAE_OPEN, FAE_DUP2, FAE_CLOSE } fae_action; + + int fae_fildes; + union { + struct { + char *path; +#define fae_path fae_data.open.path + int oflag; +#define fae_oflag fae_data.open.oflag + mode_t mode; +#define fae_mode fae_data.open.mode + } open; + struct { + int newfildes; +#define fae_newfildes fae_data.dup2.newfildes + } dup2; + } fae_data; +} posix_spawn_file_actions_entry_t; + +/* + * Spawn routines + */ + +static void +process_file_entry(posix_spawn_file_actions_entry_t *fae) +{ + int fd; + + switch (fae->fae_action) { + case FAE_OPEN: + /* Perform an open(), make it use the right fd */ + fd = _open(fae->fae_path, fae->fae_oflag, fae->fae_mode); + if (fd < 0) + _exit(127); + if (fd != fae->fae_fildes) { + if (_dup2(fd, fae->fae_fildes) == -1) + _exit(127); + if (_close(fd) != 0) + _exit(127); + } + if (_fcntl(fae->fae_fildes, F_SETFD, 0) == -1) + _exit(127); + break; + case FAE_DUP2: + /* Perform a dup2() */ + if (_dup2(fae->fae_fildes, fae->fae_newfildes) == -1) + _exit(127); + if (_fcntl(fae->fae_newfildes, F_SETFD, 0) == -1) + _exit(127); + break; + case FAE_CLOSE: + /* Perform a close() */ + if (_close(fae->fae_fildes) != 0) + _exit(127); + break; + } +} + +static pid_t +do_posix_spawn(const posix_spawn_file_actions_t *fa, + const posix_spawnattr_t *sa) +{ + posix_spawn_file_actions_entry_t *fae; + struct sigaction sigact = { .sa_flags = 0, .sa_handler = SIG_DFL }; + pid_t p; + int i; + + /* Off we go */ + p = fork(); + if (p != 0) + return (p); + + /* + * POSIX doesn't really describe in which order everything + * should be set. We'll just set them in the order in which they + * are mentioned. + */ + + /* Set signal masks/defaults */ + if (sa->sa_flags & POSIX_SPAWN_SETSIGMASK) { + _sigprocmask(SIG_SETMASK, &sa->sa_sigmask, NULL); + } + + if (sa->sa_flags & POSIX_SPAWN_SETSIGDEF) { + for (i = 1; i < NSIG; i++) { + if (sigismember(&sa->sa_sigdefault, i)) + if (_sigaction(i, &sigact, NULL) != 0) + _exit(127); + } + } + + /* Reset user ID's */ + if (sa->sa_flags & POSIX_SPAWN_RESETIDS) { + if (setegid(getgid()) != 0) + _exit(127); + if (seteuid(getuid()) != 0) + _exit(127); + } + + /* Set process group */ + if (sa->sa_flags & POSIX_SPAWN_SETPGROUP) { + if (setpgid(0, sa->sa_pgroup) != 0) + _exit(127); + } + + /* Set scheduler policy */ + if (sa->sa_flags & POSIX_SPAWN_SETSCHEDULER) { + if (sched_setscheduler(0, sa->sa_schedpolicy, + &sa->sa_schedparam) != 0) + _exit(127); + } else if (sa->sa_flags & POSIX_SPAWN_SETSCHEDPARAM) { + if (sched_setparam(0, &sa->sa_schedparam) != 0) + _exit(127); + } + + /* Replay all file descriptor modifications */ + if (fa != NULL) { + STAILQ_FOREACH(fae, &fa->fa_list, fae_list) + process_file_entry(fae); + } + + return (p); +} + +int posix_spawn(pid_t * __restrict pid, const char * __restrict path, + const posix_spawn_file_actions_t *fa, + const posix_spawnattr_t * __restrict sa, + char * const argv[__restrict], char * const envp[__restrict]) +{ + pid_t p; + + p = do_posix_spawn(fa, sa); + + switch (p) { + case -1: + return (-1); + case 0: + _execve(path, argv, envp != NULL ? envp : environ); + _exit(127); + default: + *pid = p; + return (0); + } +} + +int posix_spawnp(pid_t * __restrict pid, const char * __restrict path, + const posix_spawn_file_actions_t *fa, + const posix_spawnattr_t * __restrict sa, + char * const argv[__restrict], char * const envp[__restrict]) +{ + pid_t p; + + p = do_posix_spawn(fa, sa); + + switch (p) { + case -1: + return (-1); + case 0: + execvpe(path, argv, envp != NULL ? envp : environ); + _exit(127); + default: + *pid = p; + return (0); + } +} + +/* + * File descriptor actions + */ + +int +posix_spawn_file_actions_init(posix_spawn_file_actions_t *fa) +{ + STAILQ_INIT(&fa->fa_list); + return (0); +} + +int +posix_spawn_file_actions_destroy(posix_spawn_file_actions_t *fa) +{ + posix_spawn_file_actions_entry_t *fae; + + while ((fae = STAILQ_FIRST(&fa->fa_list)) != NULL) { + /* Remove file action entry from the queue */ + STAILQ_REMOVE_HEAD(&fa->fa_list, fae_list); + + /* Deallocate file action entry */ + if (fae->fae_action == FAE_OPEN) + free(fae->fae_path); + free(fae); + } + + return (0); +} + +int +posix_spawn_file_actions_addopen(posix_spawn_file_actions_t * __restrict fa, + int fildes, const char * __restrict path, int oflag, mode_t mode) +{ + posix_spawn_file_actions_entry_t *fae; + + /* Allocate object */ + fae = malloc(sizeof(posix_spawn_file_actions_entry_t)); + if (fae == NULL) + return (-1); + + /* Set values and store in queue */ + fae->fae_action = FAE_OPEN; + fae->fae_path = strdup(path); + if (fae->fae_path == NULL) { + free(fae); + return (-1); + } + fae->fae_fildes = fildes; + fae->fae_oflag = oflag; + fae->fae_mode = mode; + + STAILQ_INSERT_TAIL(&fa->fa_list, fae, fae_list); + return (0); +} + +int +posix_spawn_file_actions_adddup2(posix_spawn_file_actions_t *fa, + int fildes, int newfildes) +{ + posix_spawn_file_actions_entry_t *fae; + + /* Allocate object */ + fae = malloc(sizeof(posix_spawn_file_actions_entry_t)); + if (fae == NULL) + return (-1); + + /* Set values and store in queue */ + fae->fae_action = FAE_DUP2; + fae->fae_fildes = fildes; + fae->fae_newfildes = newfildes; + + STAILQ_INSERT_TAIL(&fa->fa_list, fae, fae_list); + return (0); +} + +int posix_spawn_file_actions_addclose(posix_spawn_file_actions_t *fa, + int fildes) +{ + posix_spawn_file_actions_entry_t *fae; + + /* Allocate object */ + fae = malloc(sizeof(posix_spawn_file_actions_entry_t)); + if (fae == NULL) + return (-1); + + /* Set values and store in queue */ + fae->fae_action = FAE_CLOSE; + fae->fae_fildes = fildes; + + STAILQ_INSERT_TAIL(&fa->fa_list, fae, fae_list); + return (0); +} >Release-Note: >Audit-Trail: >Unformatted: