[asterisk-dev] [svn-commits] elguero: branch 1.8 r394640 - /branches/1.8/res/res_agi.c

[Paul Belanger]

On 13-07-18 08:51 AM, SVN commits to the Digium repositories wrote:
> Author: elguero
> Date: Thu Jul 18 07:51:22 2013
> New Revision: 394640
>
> URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=394640
> Log:
> Properly indicate failure to open an audio stream in res_agi
>
> If there is an error streaming an audio file, the current return status makes it
> difficult for an AGI script to determine that there was an error with the audio
> file.
>
> This patches changes the result to return -1 and the function returns
> RESULT_FAILURE instead of RESULT_SUCCESS.  From looking at other parts of
> res_agi, this would appear to be the proper way to handle an error.
>
> (closes issue ASTERISK-21903)
> Reported by: Ariel Wainer
> Tested by: Ariel Wainer
> Patches:
> 	asterisk-21903-return-stream-res_1.8.diff
> 					by Michael L. Young (license 5026)
>
> Review: https://reviewboard.asterisk.org/r/2625/
>
> Modified:
>      branches/1.8/res/res_agi.c
>
Please add a note to UPGRADE.txt so external libraries have a heads up.

-- 
Paul Belanger | PolyBeacon, Inc.
Jabber: paul.belanger at polybeacon.com | IRC: pabelanger (Freenode)
Github: https://github.com/pabelanger | Twitter: 
https://twitter.com/pabelanger