[asterisk-bugs] [Asterisk 0017393]: [patch] AGISTATUS bug in Asterisk 1.6.2.7

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17393 
====================================================================== 
Reported By:                siby
Assigned To:                rmudgett
====================================================================== 
Project:                    Asterisk
Issue ID:                   17393
Category:                   Resources/res_agi
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
JIRA:                       SWP-2727 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-05-25 05:18 CDT
Last Modified:              2011-04-13 11:29 CDT
====================================================================== 
Summary:                    [patch] AGISTATUS bug in Asterisk 1.6.2.7
Description: 
I was just verifying different states of AGISTATUS variable i.e
SUCCESS,FAILURE,NOTFOUND and HANGUP.

I have noticed a what looks like a bug in the behaviour for teh case
HANGUP.

In this test case, I am trying to hang up the call (from caller side)
while it is still executing the AGI script.

When i do this i recive an AGISTATUS of FAILURE when I was expecting
HANGUP, as per the 'show application' explination.

 
Here is a simple setup to replicate the issue, 

extensions.conf

[test]
exten => _X.,1,Answer()
exten => _X.,n,AGI(agi-test.php)
exten => _X.,n,NoOp(AGISTATUS: ${AGISTATUS})
exten => _X.,n,Hangup()
exten => h,1,NoOp(AGISTATUS: ${AGISTATUS})

agi-test.php

#!/usr/bin/php
<?php
sleep(7);
?>

Any help/advice would be very much appreciated.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
duplicate of        0017385 AGISTATUS bug in Asterisk 1.6.1.8
====================================================================== 

---------------------------------------------------------------------- 
 (0133726) svnbot (reporter) - 2011-04-13 11:29
 https://issues.asterisk.org/view.php?id=17393#c133726 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 313579

_U  branches/1.6.2/
U   branches/1.6.2/main/channel.c
U   branches/1.6.2/res/res_agi.c

------------------------------------------------------------------------
r313579 | rmudgett | 2011-04-13 11:29:51 -0500 (Wed, 13 Apr 2011) | 48
lines

Merged revisions 313545 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r313545 | rmudgett | 2011-04-13 11:21:24 -0500 (Wed, 13 Apr 2011) | 41
lines
  
  Asterisk does not hangup a channel after endpoint hangs up.
  
  If the call that the dialplan started an AGI script for is hungup while
  the AGI script is in the middle of a command then the AGI script is not
  notified of the hangup.  There are many AGI Exec commands that this can
  happen with.  The reported applications have been: Background, Wait,
Read,
  and Dial.  Also the AGI Get Data command.
  
  * Don't wait on the Asterisk channel after it has hung up.  The channel
is
  likely to never need servicing again.
  
  * Restored the AGI script's ability to return the AGI_RESULT_HANGUP
value
  in run_agi().  It previously only could return AGI_RESULT_SUCCESS or
  AGI_RESULT_FAILURE after the DeadAGI and AGI applications were merged.
  
  (closes issue https://issues.asterisk.org/view.php?id=17954)
  Reported by: mn3250
  Patches:
        issue17954_v1.8.patch uploaded by rmudgett (license 664)
        issue17954_v1.6.2.patch uploaded by rmudgett (license 664)
        issue17954_v1.4.patch uploaded by rmudgett (license 664)
  Tested by: rmudgett
  JIRA SWP-2171
  
  (closes issue https://issues.asterisk.org/view.php?id=18492)
  Reported by: devmod
  Tested by: rmudgett
  JIRA SWP-2761
  
  (closes issue https://issues.asterisk.org/view.php?id=18935)
  Reported by: nvitaly
  Tested by: astmiv, rmudgett
  JIRA SWP-3216
  
  (closes issue https://issues.asterisk.org/view.php?id=17393)
  Reported by: siby
  Tested by: rmudgett
  JIRA SWP-2727
  
  Review: https://reviewboard.asterisk.org/r/1165/
........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-04-13 11:29 svnbot         Checkin                                      
2011-04-13 11:29 svnbot         Note Added: 0133726                          
======================================================================