From 1a89fbabb4bd2cfe1d6f4850137ecb60f0040b39 Mon Sep 17 00:00:00 2001 From: Peter Stephenson Date: Mon, 12 Jan 2004 12:09:10 +0000 Subject: 19361: Memory leaks --- Src/params.c | 19 +++++++++++++++---- 1 file changed, 15 insertions(+), 4 deletions(-) (limited to 'Src/params.c') 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); } -- cgit v1.2.3