[test-results] [Bamboo] Asterisk Testing > Asterisk Trunk > #159 has FAILED. Change made by Matthew Jordan.

[Bamboo]

-----------------------------------------------------------------------
Asterisk Testing > Asterisk Trunk > #159 failed.
-----------------------------------------------------------------------
Code has been updated by Matthew Jordan.
No failed tests found, a possible compilation error.

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


--------------
Failing Jobs
--------------
  - Asterisk CentOS 6 64-Bit (CentOS 6): No tests found.


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

>Fix places in resources where a negative return value could impact execution
>
>This patch addresses a number of modules in resources that did not handle the
>negative return value from function calls adequately.  This includes:
>
>* res_agi.c: if the result of the read function is a negative number,
>indicating some failure, the result would instead be treated as the number
>of bytes read.  This patch now treats negative results in the same manner
>as an end of file condition, with the exception that it also logs the
>error code indicated by the return.
>
>* res_musiconhold.c: if spawn_mp3 fails to assign a file descriptor to srcfd,
>and instead assigns a negative value, that file descriptor could later be
>passed to functions that require a valid file descriptor.  If spawn_mp3 fails,
>we now immediately retry instead of continuing in the logic.
>
>* res_rtp_asterisk.c: if no codec can be matched between two RTP instances
>in a peer to peer bridge, we immediately return instead of attempting to
>use the codec payload type as an index to determine the appropriate negotiated
>codec.
>
>(issue ASTERISK-19655)
>Reported by: Matt Jordan
>
>Review: https://reviewboard.asterisk.org/r/1863/
>........
>
>Merged revisions 362362 from http://svn.asterisk.org/svn/asterisk/branches/1.8
>........
>
>Merged revisions 362364 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/20120417/21c7341b/attachment.htm>