[test-results] [Bamboo] Asterisk Testing > Asterisk Trunk > #1168 has FAILED (2 tests failed, 1 failures were new). Change made by Matthew Jordan.

[Bamboo]

-----------------------------------------------------------------------
Asterisk Testing > Asterisk Trunk > #1168 failed.
-----------------------------------------------------------------------
Code has been updated by Matthew Jordan.
1/2 jobs failed, with 2 failing tests, 1 failure was new.

http://bamboo.asterisk.org/browse/TESTING-ASTERISKTRUNK-1168/


--------------
Failing Jobs
--------------
  - Asterisk CentOS 6 32-Bit (CentOS 6): 2 of 504 tests failed.



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

>Fix crash in chan_sip when a core initiated op occurs at the same time as a BYE
>
>When a BYE request is processed in chan_sip, the current SIP dialog is detached
>from its associated Asterisk channel structure. The tech_pvt pointer in the
>channel object is set to NULL, and the dialog persists for an RFC mandated
>period of time to handle re-transmits.
>
>While this process occurs, the channel is locked (which is good).
>Unfortunately, operations that are initiated externally have no way of knowing
>that the channel they've just obtained (which is still valid) and that they are
>attempting to lock is about to have its tech_pvt pointer removed. By the time
>they obtain the channel lock and call the channel technology callback, the
>tech_pvt is NULL.
>
>This patch adds a few checks to some channel callbacks that make sure the
>tech_pvt isn't NULL before using it. Prime offenders were the DTMF digit
>callbacks, which would crash if AMI initiated a DTMF on the channel at the
>same time as a BYE was received from the UA. This patch also adds checks on
>sip_transfer (as AMI can also cause a callback into this function), as well
>as sip_indicate (as lots of things can queue an indication onto a channel).
>
>Review: https://reviewboard.asterisk.org/r/2434/
>
>(closes issue ASTERISK-20225)
>Reported by: Jeff Hoppe
>........
>
>Merged revisions 385170 from http://svn.asterisk.org/svn/asterisk/branches/1.8
>........
>
>Merged revisions 385173 from http://svn.asterisk.org/svn/asterisk/branches/11
>



--------------
Tests
--------------
New Test Failures (1)
   - AsteriskTestSuite: S/channels/ s i p/sip custom presence/multiple state change
Existing Test Failures (1)
   - AsteriskTestSuite: S/apps/confbridge/confbridge marked
Fixed Tests (1)
   - AsteriskTestSuite: S/channels/ s i p/sip custom presence/nominal state change

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