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

Alec Davis asteriskteam at digium.com
Wed Nov 18 00:27:50 CST 2015


Hello Mark Michelson, Richard Mudgett, Anonymous Coward #1000019, Matt Jordan, Joshua Colp,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/1636

to look at the new patch set (#3).

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/36/1636/3
-- 
To view, visit https://gerrit.asterisk.org/1636
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie05f4e2d08912606178bf1fded57cc022c7a2e1a
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Alec Davis <sivad.a at paradise.net.nz>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list