projects
/
musl
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
move and deduplicate declarations of __procfdname to make it checkable
[musl]
/
src
/
process
/
posix_spawn.c
diff --git
a/src/process/posix_spawn.c
b/src/process/posix_spawn.c
index
f675a13
..
93fb155
100644
(file)
--- a/
src/process/posix_spawn.c
+++ b/
src/process/posix_spawn.c
@@
-22,6
+22,15
@@
struct args {
void __get_handler_set(sigset_t *);
void __get_handler_set(sigset_t *);
+static int __sys_dup2(int old, int new)
+{
+#ifdef SYS_dup2
+ return __syscall(SYS_dup2, old, new);
+#else
+ return __syscall(SYS_dup3, old, new, 0);
+#endif
+}
+
static int child(void *args_vp)
{
int i, ret;
static int child(void *args_vp)
{
int i, ret;
@@
-59,13
+68,17
@@
static int child(void *args_vp)
__libc_sigaction(i, &sa, 0);
}
__libc_sigaction(i, &sa, 0);
}
+ if (attr->__flags & POSIX_SPAWN_SETSID)
+ if ((ret=__syscall(SYS_setsid)) < 0)
+ goto fail;
+
if (attr->__flags & POSIX_SPAWN_SETPGROUP)
if ((ret=__syscall(SYS_setpgid, 0, attr->__pgrp)))
goto fail;
if (attr->__flags & POSIX_SPAWN_SETPGROUP)
if ((ret=__syscall(SYS_setpgid, 0, attr->__pgrp)))
goto fail;
- /* Use syscalls directly because
pthread state because the
- *
library functions attempt to do a multi-threaded synchronize
d
- *
id-change, which would
trash the parent's state. */
+ /* Use syscalls directly because
the library functions attempt
+ *
to do a multi-threaded synchronized id-change, which woul
d
+ * trash the parent's state. */
if (attr->__flags & POSIX_SPAWN_RESETIDS)
if ((ret=__syscall(SYS_setgid, __syscall(SYS_getgid))) ||
(ret=__syscall(SYS_setuid, __syscall(SYS_getuid))) )
if (attr->__flags & POSIX_SPAWN_RESETIDS)
if ((ret=__syscall(SYS_setgid, __syscall(SYS_getgid))) ||
(ret=__syscall(SYS_setuid, __syscall(SYS_getuid))) )
@@
-88,19
+101,26
@@
static int child(void *args_vp)
}
switch(op->cmd) {
case FDOP_CLOSE:
}
switch(op->cmd) {
case FDOP_CLOSE:
- if ((ret=__syscall(SYS_close, op->fd)))
- goto fail;
+ __syscall(SYS_close, op->fd);
break;
case FDOP_DUP2:
break;
case FDOP_DUP2:
- if ((ret=__syscall(SYS_dup2, op->srcfd, op->fd))<0)
- goto fail;
+ fd = op->srcfd;
+ if (fd != op->fd) {
+ if ((ret=__sys_dup2(fd, op->fd))<0)
+ goto fail;
+ } else {
+ ret = __syscall(SYS_fcntl, fd, F_GETFD);
+ ret = __syscall(SYS_fcntl, fd, F_SETFD,
+ ret & ~FD_CLOEXEC);
+ if (ret<0)
+ goto fail;
+ }
break;
case FDOP_OPEN:
break;
case FDOP_OPEN:
- fd = __syscall(SYS_open, op->path,
- op->oflag | O_LARGEFILE, op->mode);
+ fd = __sys_open(op->path, op->oflag, op->mode);
if ((ret=fd) < 0) goto fail;
if (fd != op->fd) {
if ((ret=fd) < 0) goto fail;
if (fd != op->fd) {
- if ((ret=__sys
call(SYS_dup2,
fd, op->fd))<0)
+ if ((ret=__sys
_dup2(
fd, op->fd))<0)
goto fail;
__syscall(SYS_close, fd);
}
goto fail;
__syscall(SYS_close, fd);
}
@@
-124,7
+144,7
@@
static int child(void *args_vp)
fail:
/* Since sizeof errno < PIPE_BUF, the write is atomic. */
ret = -ret;
fail:
/* Since sizeof errno < PIPE_BUF, the write is atomic. */
ret = -ret;
- if (ret) while (
write(
p, &ret, sizeof ret) < 0);
+ if (ret) while (
__syscall(SYS_write,
p, &ret, sizeof ret) < 0);
_exit(127);
}
_exit(127);
}
@@
-136,7
+156,7
@@
int __posix_spawnx(pid_t *restrict res, const char *restrict path,
char *const argv[restrict], char *const envp[restrict])
{
pid_t pid;
char *const argv[restrict], char *const envp[restrict])
{
pid_t pid;
- char stack[1024];
+ char stack[1024
+PATH_MAX
];
int ec=0, cs;
struct args args;
int ec=0, cs;
struct args args;