[test-results] [Bamboo] Asterisk Testing > Asterisk 11 Branch > #401 was SUCCESSFUL (with 446 tests). Change made by Matthew Jordan.

Bamboo bamboo at asterisk.org
Tue Mar 26 15:11:56 CDT 2013


-----------------------------------------------------------------------
Asterisk Testing > Asterisk 11 Branch > #401 was successful.
-----------------------------------------------------------------------
Code has been updated by Matthew Jordan.
All 2 jobs passed with 446 tests in total.

http://bamboo.asterisk.org/browse/TESTING-AST11BRANCH-401/


--------------
Code Changes
--------------
Matthew Jordan (383878):

>Resolve deadlock between SIP registration and channel based functions
>
>In r373424, several reentrancy problems in chan_sip were addressed. As a
>result, the SIP channel driver is now properly locking the channel driver
>private information in certain operations that it wasn't previously. This
>exposed two latent problems either in register_verify or by functions called
>by register_verify. This includes:
> * Holding the private lock while calling sip_send_mwi_to_peer. This can create
>   a new sip_pvt via sip_alloc, which will obtain the channel container lock.
>   This is a locking inversion, as any channel related lock must be obtained
>   prior to obtaining the SIP channel technology private lock.
>
>   Note that this issue was already fixed in Asterisk 11.
>
> * Holding the private lock while calling sip_poke_peer. In the same vein as
>   sip_send_mwi_to_peer, sip_poke_peer can create a new SIP private, causing
>   the same locking inversion.
>
>Note that this locking inversion typically occured when CLI commands were run
>while a SIP REGISTER request was being processed, as many CLI commands (such
>as 'sip show channels', 'core show channels', etc.) have to obtain the channel
>container lock.
>
>(issue ASTERISK-21068)
>Reported by: Nicolas Bouliane
>
>(issue ASTERISK-20550)
>Reported by: David Brillert
>
>(issue ASTERISK-21314)
>Reported by: Badalian Vyacheslav
>
>(issue ASTERISK-21296)
>Reported by: Gabriel Birke
>........
>
>Merged revisions 383863 from http://svn.asterisk.org/svn/asterisk/branches/1.8
>



--------------
Tests
--------------
Fixed Tests (1)
   - AsteriskTestSuite: S/apps/queues/reload queue members

--
This message is automatically generated by Atlassian Bamboo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/test-results/attachments/20130326/00b6a46e/attachment-0001.htm>


More information about the Test-results mailing list