Mercurial > pidgin
changeset 24530: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); }