Fixed two memory leaks when creating folderitems.

The Folderitem's "node" member is set to a new empty
GNode in folder_item_new(), but is later set to
another newly allocated GNode.

This commit removes the extra allocation in one case
(folder_new()), and gets rid of the original GNode
in another case, where avoiding the second allocation
is impossible without completely rewriting the recursive
folderitem appending in folder_get_from_xml().

diff --git a/src/folder.c b/src/folder.c
index b653bacd0601b365aed7b5fa3819f02fb1352973..902b990db42360689f4ef1160181ee10d2a046ca 100644 (file)
--- a/src/folder.c
+++ b/src/folder.c
@@ -192,7 +192,7 @@ Folder *folder_new(FolderClass *klass, const gchar *name, const gchar *path)
                return NULL;
        }
        item->folder = folder;
                return NULL;
        }
        item->folder = folder;

-       folder->node = item->node = g_node_new(item);
+       folder->node = item->node;

        folder->data = NULL;
 
        return folder;
        folder->data = NULL;
 
        return folder;


@@ -4046,6 +4046,7 @@ static gpointer xml_to_folder_item(gpointer nodedata, gpointer data)
        }
 
        item = folder_item_new(folder, "", "");
        }
 
        item = folder_item_new(folder, "", "");

+       g_node_destroy(item->node);

        if (folder->klass->item_set_xml != NULL)
                folder->klass->item_set_xml(folder, item, xmlnode->tag);
        else
        if (folder->klass->item_set_xml != NULL)
                folder->klass->item_set_xml(folder, item, xmlnode->tag);
        else