[asterisk-dev] [Code Review] 4173: bridge: Protect bridge channel when changing state and make it smarter

[Joshua Colp]

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

Review request for Asterisk Developers.


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


Repository: Asterisk


Description
-------

This is a patch originally done by Richard. This is only applicable to Asterisk 11. It adds locking around bridge channel state changes and makes it smarter, so that an important state (such as hangup) can't be undone. Without this change under high load ConfBridge may keep channels around when it shouldn't.


Diffs
-----

  /branches/11/main/bridging.c 427520 

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


Testing
-------

Ran sipp against chan_sip with ConfBridge at heavy CPS and confirmed that channels I would expect to be terminated were.


Thanks,

Joshua Colp

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