changeset 24334:d402644d0f98

slplink->swboard will only be null if slplink->session is null. This should never happen, so this should be a g_return_if_fail() and not just a simple if(bad)return;
author Mark Doliner <mark@kingant.net>
date Thu, 13 Nov 2008 06:28:03 +0000
parents c6c714b7841d
children d1211d64c825
files libpurple/protocols/msn/slplink.c
diffstat 1 files changed, 1 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/libpurple/protocols/msn/slplink.c	Thu Nov 13 06:18:53 2008 +0000
+++ b/libpurple/protocols/msn/slplink.c	Thu Nov 13 06:28:03 2008 +0000
@@ -234,8 +234,7 @@
 			slplink->swboard = msn_session_get_swboard(slplink->session,
 													   slplink->remote_user, MSN_SB_FLAG_FT);
 
-			if (slplink->swboard == NULL)
-				return;
+			g_return_if_fail(slplink->swboard != NULL);
 
 			/* If swboard is destroyed we will be too */
 			slplink->swboard->slplinks = g_list_prepend(slplink->swboard->slplinks, slplink);