On Fri, 2007-12-21 at 14:36 +0000, Peter Clifton wrote: > On Fri, 2007-12-21 at 14:13 +0000, Robert Fitzsimons wrote: > > The change to split out gschem TOPLEVEL variables, left some code > > which tries to copy the TOPLEVEL variables from the old data > > structure. This caused a segfalut to happen on my system. > > > > Signed-off-by: Robert Fitzsimons <[EMAIL PROTECTED]> > > --- > > gschem/src/x_image.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/gschem/src/x_image.c b/gschem/src/x_image.c > > index 260911c..aad3eb5 100644 > > --- a/gschem/src/x_image.c > > +++ b/gschem/src/x_image.c > > @@ -649,7 +649,7 @@ GdkPixbuf *x_image_get_pixbuf (GSCHEM_TOPLEVEL > > *w_current) > > /* Do a copy of the w_current struct and work with it */ > > memcpy(&new_w_current, w_current, sizeof(GSCHEM_TOPLEVEL)); > > /* Do a copy of the toplevel struct and work with it */ > > - memcpy(&toplevel, w_current, sizeof(TOPLEVEL)); > > + memcpy(&toplevel, w_current->toplevel, sizeof(TOPLEVEL)); > > > > new_w_current.toplevel = &toplevel; > > Good catch, thanks! I'll push this ASAP.
Pushed, thanks again! (I'm really glad to see some people are testing the development code!) -- Peter Clifton Electrical Engineering Division, Engineering Department, University of Cambridge, 9, JJ Thomson Avenue, Cambridge CB3 0FA Tel: +44 (0)7729 980173 - (No signal in the lab!) _______________________________________________ geda-dev mailing list [email protected] http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev
