woj/nsjail
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add child only after successful clone()

Browse Source
This commit is contained in:
Robert Swiecki 2016-02-29 16:09:08 +01:00
parent 872a561b4c
commit 62eda3c738
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
subproc.c
View File

@ -270,8 +270,6 @@ void subprocRunChild(struct nsjconf_t *nsjconf, int fd_in, int fd_out, int fd_er
subprocNewProc(nsjconf, fd_in, fd_out, fd_err, sv[0]); subprocNewProc(nsjconf, fd_in, fd_out, fd_err, sv[0]);
} }
close(sv[0]); close(sv[0]);
subprocAdd(nsjconf, pid, fd_in);
if (pid == -1) { if (pid == -1) {
PLOG_E("clone(flags=%#x) failed. You probably need root privileges if your system " PLOG_E("clone(flags=%#x) failed. You probably need root privileges if your system "
"doesn't support CLONE_NEWUSER. Alternatively, you might want to recompile your " "doesn't support CLONE_NEWUSER. Alternatively, you might want to recompile your "
@ -280,6 +278,7 @@ void subprocRunChild(struct nsjconf_t *nsjconf, int fd_in, int fd_out, int fd_er
close(sv[1]); close(sv[1]);
return; return;
} }
subprocAdd(nsjconf, pid, fd_in);
if (subprocInitParent(nsjconf, pid, sv[1]) == false) { if (subprocInitParent(nsjconf, pid, sv[1]) == false) {
close(sv[1]); close(sv[1]);