[test-results] [Bamboo] Asterisk Testing > Asterisk 1.8 Branch > #598 was SUCCESSFUL (with 292 tests). Change made by Matthew Jordan.

[Bamboo]

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

http://bamboo.asterisk.org/browse/TESTING-ASTERISK18BRANCH-598/


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

>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.
> * Holding the privat elock 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
>
>



--------------
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/8ba482cd/attachment.htm>