Free the forms properly; they're xmlnodes, not strings.

Mon, 04 May 2009 02:21:23 +0000

author
Paul Aurich <darkrain42@pidgin.im>
date
Mon, 04 May 2009 02:21:23 +0000
changeset 27040
775b48f69568
parent 27038
e33051e2790c
child 27041
d8128690074b

Free the forms properly; they're xmlnodes, not strings.

libpurple/protocols/jabber/caps.c file | annotate | diff | comparison | revisions
--- a/libpurple/protocols/jabber/caps.c	Mon May 04 00:04:55 2009 +0000
+++ b/libpurple/protocols/jabber/caps.c	Mon May 04 02:21:23 2009 +0000
@@ -132,7 +132,11 @@
 	}
 
 	free_string_glist(info->features);
-	free_string_glist(info->forms);
+
+	while (info->forms) {
+		xmlnode_free(info->forms->data);
+		info->forms = g_list_delete_link(info->forms, info->forms);
+	}
 
 	jabber_caps_node_exts_unref(info->exts);
 

mercurial