GNOME Bugzilla – Bug 166438
gconf-editor crash when closing one of multiple windows.
Last modified: 2005-02-10 11:20:10 UTC
Distribution: Fedora Core release 3 (Heidelberg) Package: gconf-editor Severity: normal Version: GNOME2.9.90 2.9.x Gnome-Distributor: GARNOME Synopsis: gconf-editor crash when closing one of multiple windows. Bugzilla-Product: gconf-editor Bugzilla-Component: general Bugzilla-Version: 2.9.x BugBuddy-GnomeVersion: 2.0 (2.9.1) Description: This backtrace is from a crash that is the result of changing a key outside of gconf-editor when it has multiple windows open. Multiple windows were open in gconf-editor at the time of the crash, in fact the order of operations was: 1. open gconf-editor 2. open a new window in gconf-editor (now 2 windows) 3. open a new window (now 3) 4. close a window (now 2 gconf-editor windows open) 5. change key = crash. I suspect that the close window internals are not properly cleaning up. Debugging Information: Backtrace was generated from '/opt/garnome/bin/gconf-editor' Using host libthread_db library "/lib/tls/libthread_db.so.1". `shared object read from target memory' has disappeared; keeping its symbols. [Thread debugging using libthread_db enabled] [New Thread -1216878912 (LWP 29989)] 0xffffe410 in __kernel_vsyscall ()
+ Trace 55365
Thread 1 (Thread -1216878912 (LWP 29989))
------- Bug moved to this database by unknown@bugzilla.gnome.org 2005-02-06 07:40 ------- Unknown platform unknown. Setting to default platform "Other". Unknown milestone "unknown" in product "gconf-editor". Setting to default milestone for this product, '---' Setting to default status "UNCONFIRMED". Setting qa contact to the default for this product. This bug either had no qa contact or an invalid one.
Cannot reproduce it here, but maybe it is a gconf problem... dunno
You need to do more with a window before closing it, here is how I can reliably crash every time: This example uses a key in /apps/metacity/general/ 1) open gnome-sound-properties and display the system bell tab. 2) enable visual feedback so that we can change the flash type. 3) open gconf-editor 4) make /apps/metacity/general the displayed set of keys (visual_bell_type resides here and should now be visible) 5) open a new window with File->New Settings Window 6) close the first gconf window (the one displaying our key) 7) In the sound preferences, change the visual feedback flash type, the crash usually happens immediately, but once I had to toggle between fullscreen and flash_frame a couple times.
Perfect instructions. Now I can reproduce, and yes, the problem seems to be that closing the window does not do the destroy correcly. Does attached patch fix the problem for you?
Created attachment 37067 [details] [review] Patch forcing gtk_object_destroy on delete window event.
Created attachment 37080 [details] [review] Attaching a patch that removes notify handler upon list model finalize. Sorry it still crashes with the patch from comment #4. I am attaching a patch that works for me. It removes the notify handler when the list model is finalized.
Thank you very much. I've committed your patch.