kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-09-03 11:25:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

res_xmpp: Don't crash when trying to send a message without a connection

Browse Source 
If we never establish a connection to our Jabber server, iksemel never sets up
its internal transport pointer, so attempting to send a message dereferences a
NULL pointer and causes a crash.

ASTERISK-21855 #close
Reported by: Jeremy Kister

Change-Id: I204a568894e4a53ab929783ecc594a000f04d79c
This commit is contained in:
Sean Bright
2017-03-22 18:32:37 -04:00
parent c1ab8ca74c
commit 7657c279b5
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
res/res_xmpp.c
View File

@@ -2562,10 +2562,16 @@ static void xmpp_log_hook(void *data, const char *xmpp, size_t size, int incomin
static int xmpp_client_send_raw_message(struct ast_xmpp_client *client, const char *message)
{
int ret;
#ifdef HAVE_OPENSSL
int len = strlen(message);
if (client->state == XMPP_STATE_DISCONNECTED) {
/* iks_send_raw will crash without a connection */
return IKS_NET_NOCONN;
}
#ifdef HAVE_OPENSSL
if (xmpp_is_secure(client)) {
int len = strlen(message);
ret = SSL_write(client->ssl_session, message, len);
if (ret) {
/* Log the message here, because iksemel's logHook is