changeset 24475:113e95371d3c

I don't believe message should ever be null
author Mark Doliner <mark@kingant.net>
date Tue, 25 Nov 2008 02:08:22 +0000
parents 01e8334101df
children faff4870f99d
files libpurple/protocols/msn/soap.c
diffstat 1 files changed, 8 insertions(+), 8 deletions(-) [+]
line wrap: on
line diff
--- 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);
 }