[Gnucash-changes] r12875 - gnucash/trunk - The GtkObject destroy function can be called multiple times. Protect

[David Hampton]

Author: hampton
Date: 2006-01-18 23:02:01 -0500 (Wed, 18 Jan 2006)
New Revision: 12875
Trac: http://svn.gnucash.org/trac/changeset/12875

Modified:
   gnucash/trunk/ChangeLog
   gnucash/trunk/src/gnome-utils/gnc-query-list.c
Log:
The GtkObject destroy function can be called multiple times. Protect
from unregistering the component multiple times.


Modified: gnucash/trunk/ChangeLog
===================================================================
--- gnucash/trunk/ChangeLog	2006-01-19 02:34:23 UTC (rev 12874)
+++ gnucash/trunk/ChangeLog	2006-01-19 04:02:01 UTC (rev 12875)
@@ -1,3 +1,9 @@
+2006-01-18  David Hampton  <hampton at employees.org>
+
+	* src/gnome-utils/gnc-query-list.c: The GtkObject destroy function
+	can be called multiple times. Protect from unregistering the
+	component multiple times.
+
 2006-01-18  Joshua Sled  <jsled at asynchronous.org>
 
 	* src/app-utils/gnc-account-merge.[ch]: C-style comment

Modified: gnucash/trunk/src/gnome-utils/gnc-query-list.c
===================================================================
--- gnucash/trunk/src/gnome-utils/gnc-query-list.c	2006-01-19 02:34:23 UTC (rev 12874)
+++ gnucash/trunk/src/gnome-utils/gnc-query-list.c	2006-01-19 04:02:01 UTC (rev 12875)
@@ -457,8 +457,10 @@
   GNCQueryListPriv *priv;
 
   priv = GNC_QUERY_LIST_GET_PRIVATE(list);
-  if (priv->component_id >= 0)
+  if (priv->component_id > 0) {
     gnc_unregister_gui_component (priv->component_id);
+    priv->component_id = 0;
+  }
   if (list->query)
   {
     xaccFreeQuery(list->query);