# HG changeset patch
# User Mark Doliner <mark@kingant.net>
# Date 1226557683 0
# Node ID d402644d0f9858f6b9933cccec0335b4cc287923
# Parent  c6c714b7841db84b655175556b5a14b7fa301474
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;

diff -r c6c714b7841d -r d402644d0f98 libpurple/protocols/msn/slplink.c
--- 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);