X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;f=src%2Fruntime%2Frun-program.c;h=57df79a5969e1eb0e84a14c99f37cfd518cc6693;hb=b83353d9f998e5c0e34604b5593df70c66d2c510;hp=64d9bdf355b582ccc6b647a2e38617272f0358ce;hpb=eb3a715584cf010842c63e78a5a90377f9aee7e7;p=sbcl.git diff --git a/src/runtime/run-program.c b/src/runtime/run-program.c index 64d9bdf..57df79a 100644 --- a/src/runtime/run-program.c +++ b/src/runtime/run-program.c @@ -25,9 +25,10 @@ #include #include #include - +#include #include #include +#include #ifdef LISP_FEATURE_OPENBSD /* FIXME: there has to be a better way to avoid ./util.h here */ @@ -99,14 +100,61 @@ set_pty(char *pty_name) extern char **environ; int spawn(char *program, char *argv[], int sin, int sout, int serr, - int search, char *envp[], char *pty_name, int wait) + int search, char *envp[], char *pty_name, int wait, char *pwd) { - int pid = fork(); + pid_t pid; int fd; + int channel[2]; sigset_t sset; + int failure_code = 2; + + channel[0] = -1; + channel[1] = -1; + if (!pipe(channel)) { + if (-1==fcntl(channel[1], F_SETFD, FD_CLOEXEC)) { + close(channel[1]); + channel[1] = -1; + } + } - if (pid != 0) + pid = fork(); + if (pid) { + if ((-1 != pid) && (-1 != channel[1])) { + int child_errno = 0; + int bytes = sizeof(int); + int n; + char *p = (char*)&child_errno; + close(channel[1]); + /* Try to read child errno from channel. */ + while ((bytes > 0) && + (n = read(channel[0], p, bytes))) { + if (-1 == n) { + if (EINTR == errno) { + continue; + } else { + break; + } + } else { + bytes -= n; + p += n; + } + } + close(channel[0]); + if (child_errno) { + int status; + waitpid(pid, &status, 0); + /* Our convention to tell Lisp that it was the exec or + chdir that failed, not the fork. */ + /* FIXME: there are other values waitpid(2) can return. */ + if (WIFEXITED(status)) { + pid = -WEXITSTATUS(status); + } + errno = child_errno; + } + } return pid; + } + close (channel[0]); /* Put us in our own process group, but only if we need not * share stdin with our parent. In the latter case we claim @@ -144,20 +192,47 @@ int spawn(char *program, char *argv[], int sin, int sout, int serr, /* Close all other fds. */ #ifdef SVR4 for (fd = sysconf(_SC_OPEN_MAX)-1; fd >= 3; fd--) - close(fd); + if (fd != channel[1]) close(fd); #else for (fd = getdtablesize()-1; fd >= 3; fd--) - close(fd); + if (fd != channel[1]) close(fd); #endif - environ = envp; - /* Exec the program. */ - if (search) - execvp(program, argv); - else - execv(program, argv); + if (pwd && chdir(pwd) < 0) { + failure_code = 3; + } else { + if (envp) { + environ = envp; + } + /* Exec the program. */ + if (search) + execvp(program, argv); + else + execv(program, argv); + } - exit (1); + /* When exec or chdir fails and channel is available, send the errno value. */ + if (-1 != channel[1]) { + int our_errno = errno; + int bytes = sizeof(int); + int n; + char *p = (char*)&our_errno; + while ((bytes > 0) && + (n = write(channel[1], p, bytes))) { + if (-1 == n) { + if (EINTR == errno) { + continue; + } else { + break; + } + } else { + bytes -= n; + p += n; + } + } + close(channel[1]); + } + _exit(failure_code); } #else /* !LISP_FEATURE_WIN32 */ @@ -188,7 +263,8 @@ HANDLE spawn ( int search, char *envp, char *ptyname, - int wait + int wait, + char *pwd ) { int stdout_backup, stdin_backup, stderr_backup, wait_mode; @@ -225,11 +301,18 @@ HANDLE spawn ( wait_mode = P_WAIT; } + /* Change working directory if supplied. */ + if (pwd) { + if (chdir(pwd) < 0) { + goto error_exit; + } + } + /* Spawn process given on the command line*/ if (search) - hProcess = (HANDLE) spawnvp ( wait_mode, program, argv ); + hProcess = (HANDLE) spawnvp ( wait_mode, program, (char* const* )argv ); else - hProcess = (HANDLE) spawnv ( wait_mode, program, argv ); + hProcess = (HANDLE) spawnv ( wait_mode, program, (char* const* )argv ); /* Now that the process is launched, replace the original * in/out/err handles and close the backups. */