luke
/
ngspice
mirror of https://git.code.sf.net/p/ngspice/ngspice
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Commit ad0f59657 ("fix a memory leak", 2018-08-13) made 

the special device parameter notation @m1[vt0] unusable.
This fix frees only the tokens that are already overridden
in function INPinsertNofree(). The reason for using this
function instead of INPinsert() is not obvious.
This commit is contained in:
Holger Vogt 2018-09-19 00:01:44 +02:00
parent 66e8e440ea
commit 2a3c88add6
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/frontend/outitf.c
View File

@ -449,6 +449,7 @@ addSpecialDesc(runDesc *run, char *name, char *devname, char *param, int depind)
{
dataDesc *data;
char *unique, *freeunique; /* unique char * from back-end */
int ret;
if (!run->numData)
run->data = TMALLOC(dataDesc, 1);
@ -463,11 +464,12 @@ addSpecialDesc(runDesc *run, char *name, char *devname, char *param, int depind)
freeunique = unique = copy(devname);
/* MW. My "special" routine here */
INPinsertNofree(&unique, ft_curckt->ci_symtab);
data->specName = copy(unique);
/* unique will be overridden, if it already exists */
ret = INPinsertNofree(&unique, ft_curckt->ci_symtab);
data->specName = unique;
tfree(freeunique);
if (ret == E_EXISTS)
tfree(freeunique);
data->specParamName = copy(param);
@ -1253,7 +1255,6 @@ freeRun(runDesc *run)
for (i = 0; i < run->numData; i++) {
tfree(run->data[i].name);
tfree(run->data[i].specParamName);
tfree(run->data[i].specName);
}
tfree(run->data);