# HG changeset patch
# User Mark Doliner <mark@kingant.net>
# Date 1227578902 0
# Node ID 113e95371d3c104130a168d85ba2cdc3575409c7
# Parent  01e8334101dfd7b26fc8a097c8a651bb60407d5e
I don't believe message should ever be null

diff -r 01e8334101df -r 113e95371d3c libpurple/protocols/msn/soap.c
--- a/libpurple/protocols/msn/soap.c	Tue Nov 25 02:02:21 2008 +0000
+++ b/libpurple/protocols/msn/soap.c	Tue Nov 25 02:08:22 2008 +0000
@@ -84,14 +84,12 @@
 static void
 msn_soap_message_destroy(MsnSoapMessage *message)
 {
-	if (message) {
-		g_slist_foreach(message->headers, (GFunc)g_free, NULL);
-		g_slist_free(message->headers);
-		g_free(message->action);
-		if (message->xml)
-			xmlnode_free(message->xml);
-		g_free(message);
-	}
+	g_slist_foreach(message->headers, (GFunc)g_free, NULL);
+	g_slist_free(message->headers);
+	g_free(message->action);
+	if (message->xml)
+		xmlnode_free(message->xml);
+	g_free(message);
 }
 
 static void
@@ -265,6 +263,8 @@
 	const char *host, const char *path, gboolean secure,
 	MsnSoapCallback cb, gpointer cb_data)
 {
+	g_return_if_fail(message != NULL);
+
 	msn_soap_message_send_internal(session, message, host, path, secure,
 		cb, cb_data, FALSE);
 }