[asterisk-bugs] [Asterisk 0013199]: Channel var AGISTATUS is not set for many AGI/DeadAGI failure conditions

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13199 
====================================================================== 
Reported By:                smw1218
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13199
Category:                   Resources/res_agi
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           1.6.0-beta9 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 134414 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-07-30 13:16 CDT
Last Modified:              2008-07-30 13:55 CDT
====================================================================== 
Summary:                    Channel var AGISTATUS is not set for many
AGI/DeadAGI failure conditions
Description: 
Under any network failure for FastAGI (and possibly many other failure
conditions) AGI does not set the AGISTATUS channel variable.  This make
dial plan recovery code much more difficult. This is trivial to reproduce:

exten => 150,1,Answer()
exten => 150,n,AGI(agi://dummyhost/dummyapp/dummy)
exten => 150,n,NoOp(${AGISTATUS})
exten => 150,n,HangUp()

Since "dummyhost" does not exist, AGI will fail to connect but not set the
AGISTATUS to "FAILURE"


====================================================================== 

---------------------------------------------------------------------- 
 (0090858) svnbot (reporter) - 2008-07-30 13:55
 http://bugs.digium.com/view.php?id=13199#c90858 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 134480

U   branches/1.4/res/res_agi.c

------------------------------------------------------------------------
r134480 | tilghman | 2008-07-30 13:55:36 -0500 (Wed, 30 Jul 2008) | 5
lines

launch_netscript sometimes returns -1, which fails to set AGISTATUS.  Map
failure to -1, so that AGISTATUS is always set.
(closes issue http://bugs.digium.com/view.php?id=13199)
 Reported by: smw1218

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=134480 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-07-30 13:55 svnbot         Checkin                                      
2008-07-30 13:55 svnbot         Note Added: 0090858                          
======================================================================