diff options
author | Eric Andersen <andersen@codepoet.org> | 2001-03-16 04:14:04 +0300 |
---|---|---|
committer | Eric Andersen <andersen@codepoet.org> | 2001-03-16 04:14:04 +0300 |
commit | 7f1978570fb8f27c05d7b0cf2a964ded5db41ee7 (patch) | |
tree | 7995a486b9aacb1afd7cc86004c5cd1f94881955 /init | |
parent | 4aaefc2a50600f2b2247ec2d607c19896e9cc0df (diff) |
A patch from Andreas Neuhaus to be especially careful to not dup
any env variables when spawning child processes.
Diffstat (limited to 'init')
-rw-r--r-- | init/init.c | 27 |
1 files changed, 17 insertions, 10 deletions
diff --git a/init/init.c b/init/init.c index 64c7768d6..145452ff3 100644 --- a/init/init.c +++ b/init/init.c @@ -396,11 +396,11 @@ static void console_init() static pid_t run(char *command, char *terminal, int get_enter) { - int i=0, j=0; + int i, j; int fd; pid_t pid; - char *tmpCmd; - char *cmd[255], *cmdpath; + char *tmpCmd, *s; + char *cmd[255], *cmdpath; char buf[255]; static const char press_enter[] = @@ -410,18 +410,25 @@ static pid_t run(char *command, char *terminal, int get_enter) "\nPlease press Enter to activate this console. "; char *environment[MAXENV+1] = { + termType, "HOME=/", "PATH=/usr/bin:/bin:/usr/sbin:/sbin", "SHELL=/bin/sh", - termType, - "USER=root" + "USER=root", + NULL }; - while (environment[i]) i++; - while ((environ[j]) && (i < MAXENV)) { - if (strncmp(environ[j], "TERM=", 5)) - environment[i++] = environ[j]; - j++; + /* inherit environment to the child, merging our values -andy */ + for (i=0; environ[i]; i++) { + for (j=0; environment[j]; j++) { + s = strchr(environment[j], '='); + if (!strncmp(environ[i], environment[j], s - environment[j])) + break; + } + if (!environment[j]) { + environment[j++] = environ[i]; + environment[j] = NULL; + } } if ((pid = fork()) == 0) { |