Mercurial > pidgin3 / changeset

• changeset
• raw

changeset

We shouldn't free user->extinfo here because it can also contain phone

Wed, 02 Jun 2010 23:29:21 +0000

author

Elliott Sales de Andrade <qulogic@pidgin.im>

date

Wed, 02 Jun 2010 23:29:21 +0000

changeset 30405

14a5070a9407

parent 30404

dea46dcf2393

child 30406

e6828577476b

We shouldn't free user->extinfo here because it can also contain phone
numbers. Mark says freeing this struct at this point is not so much of a
priority in the grand scheme of saving memory.

--- a/libpurple/protocols/msn/notification.c	Wed Jun 02 21:41:16 2010 +0000
+++ b/libpurple/protocols/msn/notification.c	Wed Jun 02 23:29:21 2010 +0000
@@ -1610,11 +1610,6 @@
 		msn_user_set_statusline(user, NULL);
 	}
 
-	if (user->extinfo && user->extinfo->media_type == CURRENT_MEDIA_UNKNOWN) {
-		g_free(user->extinfo);
-		user->extinfo = NULL;
-	}
-
 	msn_user_update(user);
 }