diff --git a/debianutils/run_parts.c b/debianutils/run_parts.c index 98fd58887..53d33cec5 100644 --- a/debianutils/run_parts.c +++ b/debianutils/run_parts.c @@ -61,6 +61,8 @@ static const struct option runparts_long_options[] = { { 0, 0, 0, 0 } }; +extern char **environ; + /* run_parts_main */ /* Process options */ int run_parts_main(int argc, char **argv) @@ -108,5 +110,5 @@ int run_parts_main(int argc, char **argv) args[0] = argv[optind]; args[argcount] = 0; - return(run_parts(args, test_mode)); + return(run_parts(args, test_mode, environ)); } diff --git a/include/libbb.h b/include/libbb.h index 6b75b8a89..ddc93c183 100644 --- a/include/libbb.h +++ b/include/libbb.h @@ -405,7 +405,7 @@ extern void run_shell ( const char *shell, int loginshell, const char *command, , security_id_t sid #endif ); -extern int run_parts(char **args, const unsigned char test_mode); +extern int run_parts(char **args, const unsigned char test_mode, char **env); extern int restricted_shell ( const char *shell ); extern void setup_environment ( const char *shell, int loginshell, int changeenv, const struct passwd *pw ); extern int correct_password ( const struct passwd *pw ); diff --git a/libbb/run_parts.c b/libbb/run_parts.c index 58645660b..171d93793 100644 --- a/libbb/run_parts.c +++ b/libbb/run_parts.c @@ -47,7 +47,7 @@ static int valid_name(const struct dirent *d) * test_mode = 2 means to fail siliently on missing directories */ -extern int run_parts(char **args, const unsigned char test_mode) +extern int run_parts(char **args, const unsigned char test_mode, char **env) { struct dirent **namelist = 0; struct stat st; @@ -92,7 +92,7 @@ extern int run_parts(char **args, const unsigned char test_mode) bb_perror_msg_and_die("failed to fork"); } else if (!pid) { args[0] = filename; - execv(filename, args); + execve(filename, args, env); exec_errno = errno; _exit(1); } diff --git a/networking/ifupdown.c b/networking/ifupdown.c index 818bec2d6..f91eddae8 100644 --- a/networking/ifupdown.c +++ b/networking/ifupdown.c @@ -1019,7 +1019,7 @@ static int doit(char *str) static int execute_all(struct interface_defn_t *ifd, execfn *exec, const char *opt) { int i; - char *buf; + char *buf[2]; for (i = 0; i < ifd->n_options; i++) { if (strcmp(ifd->option[i].name, opt) == 0) { @@ -1029,10 +1029,11 @@ static int execute_all(struct interface_defn_t *ifd, execfn *exec, const char *o } } - buf = xmalloc(bb_strlen(opt) + 19); - sprintf(buf, "/etc/network/if-%s.d", opt); - run_parts(&buf, 2); - free(buf); + bb_xasprintf(&buf[0], "/etc/network/if-%s.d", opt); + buf[1] = NULL; + + run_parts(&buf, 2, environ); + free(buf[0]); return (1); }