# HG changeset patch
# User Mark Doliner <mark@kingant.net>
# Date 1226567957 0
# Node ID e71c1fad229f234918483fe6b22284ff70f76b19
# Parent  f18f37a62971b13b48bf33eb125e33fda28f685f
cmdproc->data should never be null.  Log a warning message if it
is (this is the same type of message logged by g_return_if_fail,
right?)

diff -r f18f37a62971 -r e71c1fad229f libpurple/protocols/msn/slp.c
--- a/libpurple/protocols/msn/slp.c	Thu Nov 13 09:00:45 2008 +0000
+++ b/libpurple/protocols/msn/slp.c	Thu Nov 13 09:19:17 2008 +0000
@@ -758,17 +758,20 @@
 
 	if (slplink->swboard == NULL)
 	{
-		/* We will need this in order to change its flags. */
-		slplink->swboard = (MsnSwitchBoard *)cmdproc->data;
-		/* If swboard is NULL, something has probably gone wrong earlier on
-		 * I didn't want to do this, but MSN 7 is somehow causing us to crash
-		 * here, I couldn't reproduce it to debug more, and people are
-		 * reporting bugs. Hopefully this doesn't cause more crashes. Stu.
-		 */
-		if (slplink->swboard != NULL)
+		if (cmdproc->data == NULL)
+			g_warning("msn_p2p_msg cmdproc->data was NULL\n");
+		else {
+			/*
+			 * We will swboard in order to change its flags.  If its
+			 * NULL, something has probably gone wrong earlier on.  I
+			 * didn't want to do this, but MSN 7 is somehow causing us
+			 * to crash here, I couldn't reproduce it to debug more,
+			 * and people are reporting bugs. Hopefully this doesn't
+			 * cause more crashes. Stu.
+			 */
+			slplink->swboard = (MsnSwitchBoard *)cmdproc->data;
 			slplink->swboard->slplinks = g_list_prepend(slplink->swboard->slplinks, slplink);
-		else
-			purple_debug_error("msn", "msn_p2p_msg, swboard is NULL, ouch!\n");
+		}
 	}
 
 	msn_slplink_process_msg(slplink, msg);