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

Don't add copied scale vector as new vector, but as new scale vector, 

thus preventing crash after pushing X11 'Hardcopy' twice or
pushing 'quit' after 'Hardcopy'.
Scale vector was deleted two times, then crashimg ngsoice.
This commit is contained in:
Holger Vogt 2020-07-30 16:46:16 +02:00
parent c02da9dcc2
commit 137d5b6d56
1 changed files with 6 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/frontend/plotting/graphdb.c
View File

@ -157,11 +157,7 @@ GRAPH *CopyGraph(GRAPH *graph)
new_plotdata = newlink; /* put in front */
struct dvec * const new_scale = vec_copy(old_scale);
new_scale->v_flags |= VF_PERMANENT;
newlink = TMALLOC(struct dveclist, 1);
newlink->next = new_plotdata;
newlink->f_own_vector = TRUE;
newlink->vector = new_scale;
newlink->next = new_plotdata;
newlink->vector->v_scale = new_scale;
}
}
else {
@ -259,12 +255,13 @@ int DestroyGraph(int id)
nextd = d->next;
if (d->f_own_vector) {
/* list responsible for freeing this vector */
if (d->vector->v_scale)
dvec_free(d->vector->v_scale);
if (d->vector->v_scale) {
dvec_free(d->vector->v_scale);
}
dvec_free(d->vector);
}
txfree(d);
d = nextd;
txfree(d);
d = nextd;
}
}