aboutsummaryrefslogtreecommitdiffstats
path: root/res
diff options
context:
space:
mode:
authorqwell <qwell@f38db490-d61c-443f-a65b-d21fe96a405b>2007-10-02 18:59:39 +0000
committerqwell <qwell@f38db490-d61c-443f-a65b-d21fe96a405b>2007-10-02 18:59:39 +0000
commit5e271c91999fd27ab816d351a1c18d8b9ab6a895 (patch)
treefeb2155e9d99b07d439b539ae25c905538e7024a /res
parent9cd8f58a3d5aa66ef4005c2b2a8b1c0a01366723 (diff)
Merged revisions 84410 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4 (closes issue #10821) ........ r84410 | qwell | 2007-10-02 13:52:55 -0500 (Tue, 02 Oct 2007) | 4 lines Finish up on transferee channel before return on failure. Issue 10821, patch by Ivan ........ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@84432 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'res')
-rw-r--r--res/res_features.c13
1 files changed, 10 insertions, 3 deletions
diff --git a/res/res_features.c b/res/res_features.c
index 71c5addb1..e45d02184 100644
--- a/res/res_features.c
+++ b/res/res_features.c
@@ -956,8 +956,11 @@ static int builtin_atxfer(struct ast_channel *chan, struct ast_channel *peer, st
return FEATURE_RETURN_SUCCESS;
}
- if (check_compat(transferer, newchan))
+ if (check_compat(transferer, newchan)) {
+ /* we do mean transferee here, NOT transferer */
+ finishup(transferee);
return -1;
+ }
memset(&bconfig,0,sizeof(struct ast_bridge_config));
ast_set_flag(&(bconfig.features_caller), AST_FEATURE_DISCONNECT);
ast_set_flag(&(bconfig.features_callee), AST_FEATURE_DISCONNECT);
@@ -970,8 +973,10 @@ static int builtin_atxfer(struct ast_channel *chan, struct ast_channel *peer, st
transferer->_softhangup = 0;
return FEATURE_RETURN_SUCCESS;
}
- if (check_compat(transferee, newchan))
+ if (check_compat(transferee, newchan)) {
+ finishup(transferee);
return -1;
+ }
ast_indicate(transferee, AST_CONTROL_UNHOLD);
if ((ast_autoservice_stop(transferee) < 0)
@@ -1061,8 +1066,10 @@ static int builtin_atxfer(struct ast_channel *chan, struct ast_channel *peer, st
return -1;
/* newchan is up, we should prepare transferee and bridge them */
- if (check_compat(transferee, newchan))
+ if (check_compat(transferee, newchan)) {
+ finishup(transferee);
return -1;
+ }
ast_indicate(transferee, AST_CONTROL_UNHOLD);
if ((ast_waitfordigit(transferee, 100) < 0)