[Asterisk-code-review] app queue: (try calling): mutex 'qe->chan' freed more times ... (asterisk[master])

[Alec Davis]

Alec Davis has uploaded a new change for review.

  https://gerrit.asterisk.org/1636

Change subject: app_queue: (try_calling): mutex 'qe->chan' freed more times than we've locked!
......................................................................

app_queue: (try_calling): mutex 'qe->chan' freed more times than we've locked!

commit aae45acbd (Mark Michelson 2015-04-15 10:38:02 -0500 6525)
refer ASTERISK-24958

above commit removed ast_channel_lock(qe->chan);
but failed to remove corresponding ast_channel_unlock(qe->chan);

ASTERISK 25561
Reported Alec Davis

Change-Id: Ie05f4e2d08912606178bf1fded57cc022c7a2e1a
---
M apps/app_queue.c
1 file changed, 0 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/36/1636/1

diff --git a/apps/app_queue.c b/apps/app_queue.c
index 862988b..234543f 100644
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -6523,7 +6523,6 @@
 		ast_test_flag(&(bridge_config.features_caller), AST_FEATURE_DISCONNECT),
 		forwardsallowed, ringing);
 
-	ast_channel_unlock(qe->chan);
 	ao2_lock(qe->parent);
 	if (qe->parent->strategy == QUEUE_STRATEGY_RRMEMORY || qe->parent->strategy == QUEUE_STRATEGY_RRORDERED) {
 		store_next_rr(qe, outgoing);

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie05f4e2d08912606178bf1fded57cc022c7a2e1a
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Alec Davis <sivad.a at paradise.net.nz>