[Asterisk-code-review] bridge.c: Hangup attended transfer target after it has been ... (asterisk[certified/13.1])

[Richard Mudgett]

Richard Mudgett has uploaded a new change for review.

  https://gerrit.asterisk.org/2666

Change subject: bridge.c: Hangup attended transfer target after it has been swapped out
......................................................................

bridge.c: Hangup attended transfer target after it has been swapped out

After completing an attended transfer the transfer target channel (the one that
gets swapped out) was not being hung up after leaving the bridge. This resulted
in a channel possibly being left around. Added an explicit softhangup for the
channel in question after the transfer is successfully completed in order to
make sure the channel is hung up.

ASTERISK-24782 #close
Reported by: John Bigelow
Review: https://reviewboard.asterisk.org/r/4575/

This patch is a remedial cherry-pick from v13.

Change-Id: I26cc0c207acf74ade93e6567febf7b9776452058
---
M main/bridge.c
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/66/2666/1

diff --git a/main/bridge.c b/main/bridge.c
index c1729e5..f621942 100644
--- a/main/bridge.c
+++ b/main/bridge.c
@@ -4540,6 +4540,7 @@
 		ast_bridge_unlock(to_transferee_bridge);
 		ast_bridge_unlock(to_target_bridge);
 
+		ast_softhangup(to_transfer_target, AST_SOFTHANGUP_DEV);
 		goto end;
 	}
 

-- 
To view, visit https://gerrit.asterisk.org/2666
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I26cc0c207acf74ade93e6567febf7b9776452058
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: certified/13.1
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>