diff options
author | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-06-12 16:07:32 +0000 |
---|---|---|
committer | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-06-12 16:07:32 +0000 |
commit | fa53f105f67f9a4b4f8884469ec23c074ba8018b (patch) | |
tree | 3b1144aeae1407069b788996512f38fa3c4d3412 /channels/chan_local.c | |
parent | c3c5710689246ab800983462b18ae888a9c41e9b (diff) |
Merged revisions 33638 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2
........
r33638 | kpfleming | 2006-06-12 11:03:29 -0500 (Mon, 12 Jun 2006) | 2 lines
only allow chan_local to masquerade the outbound channel onto its owner, instead of the other way around (this will ensure that group variables on the outbound channel as preserved)
........
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@33643 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'channels/chan_local.c')
-rw-r--r-- | channels/chan_local.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/channels/chan_local.c b/channels/chan_local.c index 10c96192b..68dbd7d89 100644 --- a/channels/chan_local.c +++ b/channels/chan_local.c @@ -211,7 +211,13 @@ static void check_bridge(struct local_pvt *p, int isoutbound) return; if (!p->chan || !p->owner) return; - if (isoutbound&& p->chan->_bridge /* Not ast_bridged_channel! Only go one step! */ && !p->owner->readq) { + + /* only do the masquerade if we are being called on the outbound channel, + if it has been bridged to another channel and if there are no pending + frames on the owner channel (because they would be transferred to the + outbound channel during the masquerade) + */ + if (isoutbound && p->chan->_bridge /* Not ast_bridged_channel! Only go one step! */ && !p->owner->readq) { /* Masquerade bridged channel into owner */ /* Lock everything we need, one by one, and give up if we can't get everything. Remember, we'll get another @@ -228,6 +234,11 @@ static void check_bridge(struct local_pvt *p, int isoutbound) ast_mutex_unlock(&(p->chan->_bridge)->lock); } } + /* We only allow masquerading in one 'direction'... it's important to preserve the state + (group variables, etc.) that live on p->chan->_bridge (and were put there by the dialplan) + when the local channels go away. + */ +#if 0 } else if (!isoutbound && p->owner && p->owner->_bridge && p->chan && !p->chan->readq) { /* Masquerade bridged channel into chan */ if (!ast_mutex_trylock(&(p->owner->_bridge)->lock)) { @@ -242,6 +253,7 @@ static void check_bridge(struct local_pvt *p, int isoutbound) } ast_mutex_unlock(&(p->owner->_bridge)->lock); } +#endif } } |