[gaim-migrate @ 9413]

Wed, 14 Apr 2004 05:16:52 +0000

author
Nickolai Zeldovich
date
Wed, 14 Apr 2004 05:16:52 +0000
changeset 8661
9fdfb3108ab1
parent 8660
db1625f0f9d1
child 8662
73e15a735a88

[gaim-migrate @ 9413]
Patch by Nickolai Zeldovich to fix an infinite loop bug that sometimes
happens when MSN disconnects (closes patch #923500).

committer: Christian Hammond <chipx86@chipx86.com>

COPYRIGHT file | annotate | diff | comparison | revisions
ChangeLog file | annotate | diff | comparison | revisions
src/protocols/msn/session.c file | annotate | diff | comparison | revisions
--- a/COPYRIGHT	Wed Apr 14 04:10:26 2004 +0000
+++ b/COPYRIGHT	Wed Apr 14 05:16:52 2004 +0000
@@ -136,5 +136,6 @@
 Matt Wilson
 Ximian
 Jared Yanovich
+Nickolai Zeldovich
 Marco Ziech
 Jaroen Zwartepoorte
--- a/ChangeLog	Wed Apr 14 04:10:26 2004 +0000
+++ b/ChangeLog	Wed Apr 14 05:16:52 2004 +0000
@@ -17,6 +17,8 @@
 	* MSN error reporting works again (Stu Tomlinson)
 	* MSN e-mail notifications should no longer cause Gaim to crash
 	  (Felipe Contreras)
+	* Fixed an infinite loop bug that would sometimes cause MSN to lock
+	  up (Nickolai Zeldovich)
 	* All away messages should now show up in tooltips
 	* Removing zephyr buddies no longer crashes (Arun A. Tharuvai)
 
--- a/src/protocols/msn/session.c	Wed Apr 14 04:10:26 2004 +0000
+++ b/src/protocols/msn/session.c	Wed Apr 14 05:16:52 2004 +0000
@@ -65,7 +65,13 @@
 		msn_switchboard_destroy(session->switches->data);
 
 	while (session->lists.forward)
-		msn_user_destroy(session->lists.forward->data);
+	{
+		MsnUser *user = (MsnUser *)session->lists.forward->data;
+
+		msn_user_destroy(user);
+
+		session->lists.forward = g_slist_remove(session->lists.forward, user);
+	}
 
 	if (session->lists.allow != NULL)
 		g_slist_free(session->lists.allow);

mercurial