summaryrefslogtreecommitdiff
path: root/sh.c
diff options
context:
space:
mode:
authorEric Andersen2001-03-12 21:36:49 +0000
committerEric Andersen2001-03-12 21:36:49 +0000
commit4987bbf97924044c88007c45ee2674d97b93ad92 (patch)
treee6e1eb55e0a787df1c768713bfca467439ec0217 /sh.c
parent5ec5415402ee6e4ae21e4a50c04360908325ba81 (diff)
downloadbusybox-4987bbf97924044c88007c45ee2674d97b93ad92.zip
busybox-4987bbf97924044c88007c45ee2674d97b93ad92.tar.gz
A fix from Larry Doolittle for some nasty segfaults when sh.c tried to free a
corrupted cmd pointer, and where it would truncate a command line at the first double space. -Erik
Diffstat (limited to 'sh.c')
-rw-r--r--sh.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/sh.c b/sh.c
index d321c92..67d6e4f 100644
--- a/sh.c
+++ b/sh.c
@@ -970,10 +970,10 @@ static int expand_arguments(char *command)
{
int flags = GLOB_NOCHECK|GLOB_BRACE|GLOB_TILDE;
- char * tmpcmd;
+ char * tmpcmd, *cmd, *cmd_copy;
/* We need a clean copy, so strsep can mess up the copy while
* we write stuff into the original (in a minute) */
- char * cmd = strdup(command);
+ cmd = cmd_copy = strdup(command);
*command = '\0';
for (tmpcmd = cmd; (tmpcmd = strsep(&cmd, " \t")) != NULL;) {
if (*tmpcmd == '\0')
@@ -1006,7 +1006,7 @@ static int expand_arguments(char *command)
globfree (&expand_result);
}
}
- free(cmd);
+ free(cmd_copy);
trim(command);
}