After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 300744 - New view -> close original window -> crash
New view -> close original window -> crash
Status: RESOLVED FIXED
Product: dia
Classification: Other
Component: win32
0.94
Other All
: High critical
: 0.95
Assigned To: Steffen Macke
Dia maintainers
: 301954 421524 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2005-04-15 14:16 UTC by Lode Leroy
Modified: 2007-03-22 21:34 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Lode Leroy 2005-04-15 14:16:11 UTC
Steps to reproduce:
1. start dia
2. select new view
3. close original window


Stack trace:
Assertion failed
diagram.c line 85
dia->displays == NULL


Other information:
Comment 1 Alan Horkan 2005-04-15 15:52:14 UTC
There are many features that we would like to add but we do not currently have
the time or resources to do so.  We very much appreciate your interest, and if
you can provide patches to help implement any of these features it would be
extremely helpful.  

Otherwise it would be very helpful if you can provide detailed descriptions of
the functionality and how you expect it to work including but not limited to
text descriptions, screenshots or glade mockups and information on how other
programs have implemented similar functionality.  It is worth noting that some
of your requests could be extremely difficult and time consuming to implement.  

Thanks again for your interet in Dia.  

I will try and respond to each of your requests individually at some point, but
if you can double check to ensure there are no duplicates it would be helpful.  

Sincerely

Alan Horkan
Comment 2 Lars Clausen 2005-04-15 16:39:29 UTC
Can't reproduce on Linux with 0.94 or current CVS.
Comment 3 Christian Kirbach 2005-04-16 09:01:31 UTC
Me neither on Linux.Must be a Win32 issue.
Comment 4 Hans Breuer 2005-04-20 19:52:23 UTC
Nope. Just being irreproducable does not mean it can't happen on that platform.
IIRC there was a race between diagram and display destruction. Should be fixed
by adding the right refernce counting. A side effect of :

2004-12-11  Hans Breuer  <hans@breuer.org>

        * lib/diagramdata.[ch] app/app_procs.c app/diagram.c
          lib/libdia.def : get rid of new_diagram_data()
        Diagram _is_ DiagramData (and does not have one any longer,
        just a compatibility pointer)
 
Comment 5 Hans Breuer 2005-04-23 15:50:07 UTC
Just another nope ;) Actually I can reproduce this under
Linux *and* Windows with Dia 0.94 *and* current cvs.

diagram.c:94: diagram_finalize: Assertion 'dia->displays==((void *)0)' failed.
app/run_dia:sh: line 22: 3195 Aborted

[Obviously one has to check with a debug enabled build cause this
is just an assert() ...]

Anyway, fixed in cvs.

2005-04-23  Hans Breuer  <hans@breuer.org>

	* app/display.c : stop special casing the first display of
	a diagram. Every display now has it's own refernce. The initial
	reference gets dropped when there is no display anymore. See
	diagram_remove_ddisplay(). Fixes bug #300744.

Comment 6 Hans Breuer 2005-04-30 11:29:06 UTC
*** Bug 301954 has been marked as a duplicate of this bug. ***
Comment 7 Hans Breuer 2007-03-22 21:34:06 UTC
*** Bug 421524 has been marked as a duplicate of this bug. ***