[asterisk-dev] [Code Review] 2962: res_stasis.c: Fix locking issues with the app_bridge_moh container.

rmudgett reviewboard at asterisk.org
Fri Nov 8 14:20:29 CST 2013


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

(Updated Nov. 8, 2013, 2:20 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 402593


Repository: Asterisk


Description
-------

* Fix unlinking from the app_bridges_moh container in remove_bridge_moh() without a lock under normal circumstances.

* Made check ast_bridge_set_after_callback() return value in bridge_moh_create() to handle failure.

* Fixed SCOPED_AO2LOCK() locking over too much scope in stasis_app_bridge_moh_channel() and stasis_app_bridge_moh_stop().

* Fixed unusual usage of ao2_unlink_flag() in control_unlink().

* Fixed orphaned bridge from off nominal path in stasis_app_bridge_create().

* Fixed strange construct in stasis_app_unsubscribe().  From a bad merge?

* Made load_module() cleanup on failure.


Diffs
-----

  /branches/12/res/res_stasis.c 402056 

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


Testing
-------

Compile and code inspection.


Thanks,

rmudgett

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


More information about the asterisk-dev mailing list