[asterisk-commits] jrose: branch 10 r333266 - in /branches/10: ./ res/res_jabber.c

[SVN commits to the Asterisk project]

Author: jrose
Date: Thu Aug 25 14:00:05 2011
New Revision: 333266

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=333266
Log:
Merged revisions 333265 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r333265 | jrose | 2011-08-25 13:47:42 -0500 (Thu, 25 Aug 2011) | 14 lines
  
  Segfault when publishing device states via XMPP and not connected
  
  When using publishing device state with res_jabber, Asterisk will attempt
  to send a device state using the unconnected client using iks_send_raw
  and crash. This patch checks the validity of the connection before 
  attempting to send the device state.
  
  (closes issue ASTERISK-18078)
  Reported by: Michael L. Young
  Patches:
        res_jabber-segfault-pubsub-not-connected2.patch (license #5026) patch uploaded by Michael L. Young
  Tested by: Jonathan Rose
........

Modified:
    branches/10/   (props changed)
    branches/10/res/res_jabber.c

Propchange: branches/10/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: branches/10/res/res_jabber.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/res/res_jabber.c?view=diff&rev=333266&r1=333265&r2=333266
==============================================================================
--- branches/10/res/res_jabber.c (original)
+++ branches/10/res/res_jabber.c Thu Aug 25 14:00:05 2011
@@ -1511,7 +1511,15 @@
 #endif
 	/* If needed, data will be sent unencrypted, and logHook will
 	   be called inside iks_send_raw */
-	ret = iks_send_raw(client->p, xmlstr);
+	if((client->timeout != 0 && client->status == AJI_CONNECTED) || (client->status == AJI_CONNECTING))
+	{
+	    ret = iks_send_raw(client->p, xmlstr);
+	}
+	else {
+		ast_log(LOG_WARNING, "JABBER: Unable to send message to %s, we are not connected", client->name);
+		return -1;
+	}
+
 	if (ret != IKS_OK) {
 		return ret;
 	}