# HG changeset patch
# User Elliott Sales de Andrade <qulogic@pidgin.im>
# Date 1281757143 0
# Node ID cd022bd83677916154d06f9c351308589ddca269
# Parent  f6017a90c1b89a9ac21962f41007129ce68dedec
I think this might be a leak. But probably only if the server changed
what it was sending.

diff -r f6017a90c1b8 -r cd022bd83677 libpurple/protocols/msn/contact.c
--- a/libpurple/protocols/msn/contact.c	Wed Aug 11 04:37:49 2010 +0000
+++ b/libpurple/protocols/msn/contact.c	Sat Aug 14 03:39:03 2010 +0000
@@ -684,20 +684,20 @@
 		xmlnode *annotation;
 		MsnUser *user;
 
+		g_free(passport);
+		g_free(Name);
+		g_free(uid);
+		g_free(type);
+		g_free(mobile_number);
+		g_free(alias);
+		passport = Name = uid = type = mobile_number = alias = NULL;
+		mobile = FALSE;
+
 		if (!(contactId = xmlnode_get_child(contactNode,"contactId"))
 				|| !(contactInfo = xmlnode_get_child(contactNode, "contactInfo"))
 				|| !(contactType = xmlnode_get_child(contactInfo, "contactType")))
 			continue;
 
-		g_free(passport);
-		g_free(Name);
-		g_free(alias);
-		g_free(uid);
-		g_free(type);
-		g_free(mobile_number);
-		passport = Name = uid = type = mobile_number = alias = NULL;
-		mobile = FALSE;
-
 		uid = xmlnode_get_data(contactId);
 		type = xmlnode_get_data(contactType);
 
@@ -836,6 +836,7 @@
 	g_free(uid);
 	g_free(type);
 	g_free(mobile_number);
+	g_free(alias);
 }
 
 static gboolean