strdup the input to putenv to avoid in one case passing a string that is later

freed and in the other const strings.

looks sane to millert, ok ray
This commit is contained in:
Nicholas Marriott
2009-06-05 07:18:37 +00:00
parent 1156467726
commit a3c32841e6
2 changed files with 2 additions and 2 deletions

View File

@ -63,7 +63,7 @@ cmd_string_parse(const char *s, struct cmd_list **cmdlist, char **cause)
if ((t = strchr(s, ' ')) == NULL && (t = strchr(s, '\t')) == NULL)
t = strchr(s, '\0');
if ((u = strchr(s, '=')) != NULL && u < t) {
if (putenv((char *) s) != 0) {
if (putenv(xstrdup(s)) != 0) {
xasprintf(cause, "assignment failed: %s", s);
return (-1);
}