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

Alec Davis asteriskteam at digium.com
Mon Nov 16 14:38:58 CST 2015


Alec Davis has uploaded a new change for review.

  https://gerrit.asterisk.org/1641

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 #close
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/41/1641/1

diff --git a/apps/app_queue.c b/apps/app_queue.c
index 2dcb3b5..99a2b6c 100644
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -6472,7 +6472,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/1641
To unsubscribe, visit https://gerrit.asterisk.org/settings

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



More information about the asterisk-code-review mailing list