2007-06-11 Murray Cumming <murrayc@murrayc.com>

* src/modest-account-mgr.c: (modest_account_mgr_finalize):
        Free the changed_conf_keys string items. They are already freed in the
        idle handler if that runs first.

pmo-trunk-r2145

diff --git a/ChangeLog2 b/ChangeLog2
index d403a86..564fc2b 100644 (file)
--- a/ChangeLog2
+++ b/ChangeLog2
@@ -1,5 +1,11 @@
 2007-06-11  Murray Cumming  <murrayc@murrayc.com>
 
+       * src/modest-account-mgr.c: (modest_account_mgr_finalize):
+       Free the changed_conf_keys string items. They are already freed in the 
+       idle handler if that runs first.
+
+2007-06-11  Murray Cumming  <murrayc@murrayc.com>
+
        * src/modest-ui-actions.c:
        (modest_ui_actions_on_password_requested): Check that the server name 
        is not NULL, because I just saw a password dialog with (null) as the 

diff --git a/src/modest-account-mgr.c b/src/modest-account-mgr.c
index 3dc0bc7..f5bde23 100644 (file)
--- a/src/modest-account-mgr.c
+++ b/src/modest-account-mgr.c
@@ -220,8 +220,10 @@ modest_account_mgr_finalize (GObject * obj)
        if (priv->timeout)
                g_source_remove (priv->timeout);
                
-       if (priv->changed_conf_keys)
+       if (priv->changed_conf_keys) {
+               g_slist_foreach (priv->changed_conf_keys, (GFunc) g_free, NULL);
                g_slist_free (priv->changed_conf_keys);
+       }
 
        G_OBJECT_CLASS(parent_class)->finalize (obj);
 }