[asterisk-dev] [Code Review] 3567: [channels/chan_unistim.c]: Possible unlocked mutex

[Peter Whisker]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3567/
-----------------------------------------------------------

(Updated May 28, 2014, 7:59 a.m.)


Review request for Asterisk Developers and Igor Goncharovsky.


Changes
-------

Amended as suggested by wdoekes - now patched against branches/11 and simply moved the unlock up four lines.


Bugs: ASTERISK-23792
    https://issues.asterisk.org/jira/browse/ASTERISK-23792


Repository: Asterisk


Description
-------

There is a conditional exit from a mutexed area of code which does not unlock before the return.


Diffs (updated)
-----

  /branches/11/channels/chan_unistim.c 414674 

Diff: https://reviewboard.asterisk.org/r/3567/diff/


Testing
-------

Inspection and build


Thanks,

Peter Whisker

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140528/6459c4c8/attachment.html>