Make ast_do_masquerade() a void function.
authorRichard Mudgett <rmudgett@digium.com>
Sat, 9 Feb 2013 01:31:55 +0000 (01:31 +0000)
committerRichard Mudgett <rmudgett@digium.com>
Sat, 9 Feb 2013 01:31:55 +0000 (01:31 +0000)
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@381086 65c4cc65-6c06-0410-ace0-fbb531ad65f3

channels/chan_sip.c
include/asterisk/channel.h
main/channel.c

index 678eaa7..b7603b6 100644 (file)
@@ -24807,9 +24807,7 @@ static int handle_invite_replaces(struct sip_pvt *p, struct sip_request *req, st
        ast_channel_unlock(c);
        sip_pvt_unlock(p->refer->refer_call);
        sip_pvt_unlock(p);
-       if (ast_do_masquerade(replacecall)) {
-               ast_log(LOG_WARNING, "Failed to perform masquerade with INVITE replaces\n");
-       }
+       ast_do_masquerade(replacecall);
        ast_channel_lock(c);
        if (earlyreplace || oneleggedreplace ) {
                ast_channel_hangupcause_set(c, AST_CAUSE_SWITCH_CONGESTION);
index a3974ee..f7e4d2d 100644 (file)
@@ -2322,7 +2322,7 @@ int ast_transfer(struct ast_channel *chan, char *dest);
  *     while the features are nice, the cost is very high in terms of pure nastiness. XXX
  * \param chan Channel to masquerade
  */
-int ast_do_masquerade(struct ast_channel *chan);
+void ast_do_masquerade(struct ast_channel *chan);
 
 /*!
  * \brief Find bridged channel
index 30df143..9c8f32c 100644 (file)
@@ -6863,7 +6863,7 @@ static void masquerade_colp_transfer(struct ast_channel *transferee, struct xfer
  *       this function, it invalidates our channel container locking order.  All channels
  *       must be unlocked before it is permissible to lock the channels' ao2 container.
  */
-int ast_do_masquerade(struct ast_channel *original)
+void ast_do_masquerade(struct ast_channel *original)
 {
        int x;
        int origstate;
@@ -6940,7 +6940,7 @@ int ast_do_masquerade(struct ast_channel *original)
                 */
                ast_channel_unlock(original);
                ao2_unlock(channels);
-               return 0;
+               return;
        }
 
        /* Bump the refs to ensure that they won't dissapear on us. */
@@ -7345,8 +7345,6 @@ int ast_do_masquerade(struct ast_channel *original)
        /* Release our held safety references. */
        ast_channel_unref(original);
        ast_channel_unref(clonechan);
-
-       return 0;
 }
 
 void ast_set_callerid(struct ast_channel *chan, const char *cid_num, const char *cid_name, const char *cid_ani)