[test-results] [Bamboo] Asterisk - 10 > Ubuntu Lucid (10.04) > #417 has FAILED (1 tests failed). Change made by rmudgett.

[Bamboo]

-----------------------------------------------------------------------
Asterisk - 10 > Ubuntu Lucid (10.04) > #417 failed.
-----------------------------------------------------------------------
Code has been updated by rmudgett.
1/189 tests failed.

http://bamboo.asterisk.org/browse/AST10-LUCID-417/


--------------
Failing Jobs
--------------
  - amd64 (Default Stage): 1 of 189 tests failed.


--------------
Code Changes
--------------
rmudgett (348952):

>Fix extension state callback references in chan_sip.
>
>Chan_sip gives a dialog reference to the extension state callback and
>assumes that when ast_extension_state_del() returns, the callback cannot
>happen anymore.  Chan_sip then reduces the dialog reference count
>associated with the callback.  Recent changes (ASTERISK-17760) have
>resulted in the potential for the callback to happen after
>ast_extension_state_del() has returned.  For chan_sip, this could be very
>bad because the dialog pointer could have already been destroyed.
>
>* Added ast_extension_state_add_destroy() so chan_sip can account for the
>sip_pvt reference given to the extension state callback when the extension
>state callback is deleted.
>
>* Fix pbx.c awkward statecbs handling in ast_extension_state_add_destroy()
>and handle_statechange() now that the struct ast_state_cb has a destructor
>to call.
>
>* Ensure that ast_extension_state_add_destroy() will never return -1 or 0
>for a successful registration.
>
>* Fixed pbx.c statecbs_cmp() to compare the correct information.  The
>passed in value to compare is a change_cb function pointer not an object
>pointer.
>
>* Make pbx.c ast_merge_contexts_and_delete() not perform callbacks with
>AST_EXTENSION_REMOVED with locks held.  Chan_sip is notorious for
>deadlocking when those locks are held during the callback.
>
>* Removed unused lock declaration for the pbx.c store_hints list.
>
>(closes issue ASTERISK-18844)
>Reported by: rmudgett
>
>Review: https://reviewboard.asterisk.org/r/1635/
>........
>
>Merged revisions 348940 from http://svn.asterisk.org/svn/asterisk/branches/1.8
>


--------------
Tests
--------------
New Test Failures (1)
   - AsteriskTestSuite: S/apps/confbridge/confbridge nominal

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