[test-results] [Bamboo] Asterisk Testing > Asterisk Trunk > #121 has FAILED. Change made by Mark Michelson.

[Bamboo]

-----------------------------------------------------------------------
Asterisk Testing > Asterisk Trunk > #121 failed.
-----------------------------------------------------------------------
Code has been updated by Mark Michelson.
No failed tests found, a possible compilation error.

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


--------------
Failing Jobs
--------------
  - Asterisk CentOS 6 64-Bit (CentOS 6): 92 tests passed.


--------------
Code Changes
--------------
Mark Michelson (360886):

>Fix potential race condition during call pickup.
>
>Prior to this patch, a connected line update was queued during
>call pickup and then an answer frame was queued. The original
>caller would presumably then have his connected line updated
>and then the call would be answered.
>
>In actuality, the answer frame was not how the call ended up
>being answered. Rather, an odd section in app_dial that checks
>if the called channel's state is up.
>
>The result is that the order of the connected line update and
>the answer were variable. In most cases, this wasn't actually
>a bad thing. However, if the 'I' option was passed to dial, the
>connected line update would be inhibited.
>
>The fix is to queued the connected line after the answer frame is
>queued. This way the race in app_dial is between two
>conditions resulting in an answer. This way the connected line
>update occurs after the answer every time.
>
>(closes issue ASTERISK-19183)
>Reported by: Thomas Arimont
>Tested by: Thomas Arimont
>    Mark Michelson
>Patches:
>    ASTERISK-19183.patch uploaded by Mark Michelson (license 5049)
>........
>
>Merged revisions 360884 from http://svn.asterisk.org/svn/asterisk/branches/1.8
>........
>
>Merged revisions 360885 from http://svn.asterisk.org/svn/asterisk/branches/10
>


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