From: Terry Wilson <twilson@digium.com>
Date: Mon, 7 Feb 2011 22:40:20 +0000 (+0000)
Subject: Merged revisions 306672 via svnmerge from
X-Git-Tag: 1.6.2.18-rc1~18
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=4cefaad7a0dd9e5aed869990ca36d888e1d5bd1e;p=thirdparty%2Fasterisk.git

Merged revisions 306672 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r306672 | twilson | 2011-02-07 14:35:20 -0800 (Mon, 07 Feb 2011) | 10 lines

  Don't try to pickup a call in the middle of a masquerade

  If A calls B which doesn't answer and C & D both try to do a call pickup, it is
  possible for ast_pickup_call to answer the call, then fail to masquerade one of
  the calls because the other one is already in the process of masquerading. This
  patch checks to see if the channel is in the process of masquerading before
  call before selecting it for a pickup.

  Review: https://reviewboard.asterisk.org/r/1094/
........


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

diff --git a/main/features.c b/main/features.c
index 10a4bdf5d6..f89d5fc5d2 100644
--- a/main/features.c
+++ b/main/features.c
@@ -4882,7 +4882,8 @@ static int find_channel_by_group(struct ast_channel *c, void *data) {
 		   change while we're here, but that isn't a problem. */
 		(c != chan) &&
 		(chan->pickupgroup & c->callgroup) &&
-		((c->_state == AST_STATE_RINGING) || (c->_state == AST_STATE_RING));
+		((c->_state == AST_STATE_RINGING) || (c->_state == AST_STATE_RING)) &&
+		!c->masq;
 }
 
 /*!