comparison libpurple/protocols/msn/soap.c @ 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
comparison
equal deleted inserted replaced
24474:01e8334101df 24475:113e95371d3c
82 } 82 }
83 83
84 static void 84 static void
85 msn_soap_message_destroy(MsnSoapMessage *message) 85 msn_soap_message_destroy(MsnSoapMessage *message)
86 { 86 {
87 if (message) { 87 g_slist_foreach(message->headers, (GFunc)g_free, NULL);
88 g_slist_foreach(message->headers, (GFunc)g_free, NULL); 88 g_slist_free(message->headers);
89 g_slist_free(message->headers); 89 g_free(message->action);
90 g_free(message->action); 90 if (message->xml)
91 if (message->xml) 91 xmlnode_free(message->xml);
92 xmlnode_free(message->xml); 92 g_free(message);
93 g_free(message);
94 }
95 } 93 }
96 94
97 static void 95 static void
98 msn_soap_request_destroy(MsnSoapRequest *req, gboolean keep_message) 96 msn_soap_request_destroy(MsnSoapRequest *req, gboolean keep_message)
99 { 97 {
263 void 261 void
264 msn_soap_message_send(MsnSession *session, MsnSoapMessage *message, 262 msn_soap_message_send(MsnSession *session, MsnSoapMessage *message,
265 const char *host, const char *path, gboolean secure, 263 const char *host, const char *path, gboolean secure,
266 MsnSoapCallback cb, gpointer cb_data) 264 MsnSoapCallback cb, gpointer cb_data)
267 { 265 {
266 g_return_if_fail(message != NULL);
267
268 msn_soap_message_send_internal(session, message, host, path, secure, 268 msn_soap_message_send_internal(session, message, host, path, secure,
269 cb, cb_data, FALSE); 269 cb, cb_data, FALSE);
270 } 270 }
271 271
272 static gboolean 272 static gboolean