pidgin: 31ef531a: Free the forms properly; they're xmlnode...

darkrain42 at pidgin.im darkrain42 at pidgin.im
Sun May 3 23:00:29 EDT 2009


-----------------------------------------------------------------
Revision: 31ef531a4ba2ec1a28f7ea716791f9f9169b313b
Ancestor: 8a85735913e31b9becf0404220e7fe2335dec8a9
Author: darkrain42 at pidgin.im
Date: 2009-05-04T02:21:23
Branch: im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/31ef531a4ba2ec1a28f7ea716791f9f9169b313b

Modified files:
        libpurple/protocols/jabber/caps.c

ChangeLog: 

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

-------------- next part --------------
============================================================
--- libpurple/protocols/jabber/caps.c	3c5eea3fb2cdc32875e903beba953763c0b668de
+++ libpurple/protocols/jabber/caps.c	f110487c8aef3f256260f5a8661a33a1bb9e9276
@@ -132,8 +132,12 @@ jabber_caps_client_info_destroy(JabberCa
 	}
 
 	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);
 
 	g_free(info);


More information about the Commits mailing list