Re: gEDA-user: CVS gschem crash in recent_files_add

[Ivan Stankovic <ivan.stankovic@xxxxxx>]

On Sun, Jun 03, 2007 at 10:46:19PM +0100, Peter Clifton wrote:
> On Sun, 2007-06-03 at 13:02 -0700, Ben Jackson wrote:
> > recent_files_add starts at global_window_current and walks back through
> > 'prev' as far as it can.  In my crash, global_window_current looks fine,
> > its 'prev' looks fine (like a suitable pointer) but the contents of THAT
> > are garbage, which causes a crash in a GTK sanity check.
> > 
> > This seems to be due to s_toplevel_init() creating exactly that:  A
> > toplevel with the only initialized fields being wid, next, prev.
> > 
> > I don't really know what was intended, but I see a few solutions:
> > 
> > 1)  Have recent_files_add ignore wid==-1 toplevels, or
> 
> That is the "correct" fix for the current code.

It probably is (I can't get gschem to crash...). But I'm wondering 
why there are toplevels with wid = -1. If I see it on the screen, then
surely it must have all the associated gtk widgets realized and
attached... right?

-- 
Ivan Stankovic, ivan.stankovic@xxxxxx

"Protect your digital freedom and privacy, eliminate DRM, 
learn more at http://www.defectivebydesign.org/what_is_drm";


_______________________________________________
geda-user mailing list
geda-user@xxxxxxxxxxxxxx
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user