]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
Merged revisions 339297 via svnmerge from
authorJonathan Rose <jrose@digium.com>
Tue, 4 Oct 2011 14:09:50 +0000 (14:09 +0000)
committerJonathan Rose <jrose@digium.com>
Tue, 4 Oct 2011 14:09:50 +0000 (14:09 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r339297 | jrose | 2011-10-04 09:01:05 -0500 (Tue, 04 Oct 2011) | 13 lines

  Reverting revision 333265 due to component connection problems it introduces.

  I'm going to attempt some generic res_jabber cleanup and come up with a new fix for this
  problem, but first it seems prudent to remove this rather broad attempt to fix it and
  instead approach this problem either from the same angle but looking only at canceling
  (or possibly rescheduling) the send when we absolutely know it will cause a segfault
  or, if that can't be easily accomplished, strictly from the devstate side of things.
  Also, I'm pretty sure a lot of the code in res_jabber isn't thread safe.

  (issue ASTERISK-18626)
  (issue ASTERISK-18078)
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/10@339298 65c4cc65-6c06-0410-ace0-fbb531ad65f3

res/res_jabber.c

index a4772b19c22562b608247dce0e0abf80bca29dd9..240f0e2d6eb86aceca728c1aae31fbde40196716 100644 (file)
@@ -1511,15 +1511,7 @@ static int aji_send_raw(struct aji_client *client, const char *xmlstr)
 #endif
        /* If needed, data will be sent unencrypted, and logHook will
           be called inside iks_send_raw */
-       if((client->timeout != 0 && client->state == AJI_CONNECTED) || (client->state == 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;
-       }
-
+       ret = iks_send_raw(client->p, xmlstr);
        if (ret != IKS_OK) {
                return ret;
        }