diff options
Diffstat (limited to 'Src')
| -rw-r--r-- | Src/builtin.c | 8 | ||||
| -rw-r--r-- | Src/params.c | 19 |
2 files changed, 23 insertions, 4 deletions
diff --git a/Src/builtin.c b/Src/builtin.c index 37e3db7eb..48ad69324 100644 --- a/Src/builtin.c +++ b/Src/builtin.c @@ -2255,7 +2255,15 @@ bin_typeset(char *name, char **argv, Options ops, int func) return 1; } + /* + * pm->ename is only deleted when the struct is, so + * we need to free it here if it already exists. + */ + if (pm->ename) + zsfree(pm->ename); pm->ename = ztrdup(asg->name); + if (apm->ename) + zsfree(apm->ename); apm->ename = ztrdup(asg0.name); if (oldval) setsparam(asg0.name, oldval); diff --git a/Src/params.c b/Src/params.c index c77b23566..90389c40c 100644 --- a/Src/params.c +++ b/Src/params.c @@ -1599,6 +1599,7 @@ setstrvalue(Value v, char *val) } if (v->pm->flags & PM_HASHED) { zerr("%s: attempt to set slice of associative array", v->pm->nam, 0); + zsfree(val); return; } v->pm->flags &= ~PM_UNSET; @@ -2377,12 +2378,12 @@ strsetfn(Param pm, char *x) /* Function to get value of an array parameter */ +static char *nullarray = NULL; + /**/ char ** arrgetfn(Param pm) { - static char *nullarray = NULL; - return pm->u.arr ? pm->u.arr : &nullarray; } @@ -2558,7 +2559,9 @@ strvargetfn(Param pm) mod_export char ** arrvargetfn(Param pm) { - return *((char ***)pm->u.data); + char **arrptr = *((char ***)pm->u.data); + + return arrptr ? arrptr : &nullarray; } /* Function to set value of generic special array parameter. * @@ -2577,7 +2580,15 @@ arrvarsetfn(Param pm, char **x) freearray(*dptr); if (pm->flags & PM_UNIQUE) uniqarray(x); - *dptr = x ? x : mkarray(NULL); + /* + * Special tied arrays point to variables accessible in other + * ways which need to be set to NULL. We can't do this + * with user tied variables since we can leak memory. + */ + if ((pm->flags & PM_SPECIAL) & !x) + *dptr = mkarray(NULL); + else + *dptr = x; if (pm->ename && x) arrfixenv(pm->ename, x); } |