[Home]

Summary:ASTERISK-11212: [patch] agi RECORD FILE incorrect result code on hangup
Reporter:David Van Ginneken (davevg)Labels:
Date Opened:2008-01-11 13:05:56.000-0600Date Closed:2008-01-11 13:28:27.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Resources/res_agi
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) res_agi.diff
Description:If you hangup while an AGI is performing a RECORD FILE, it returns a result code of 0 instead of -1.  

AGI Rx << RECORD FILE /tmp/test ulaw # -1 0 1 s=3
   -- <SIP/1223-098cdb80> Playing 'beep' (language 'en')
AGI Tx >> 200 result=0 (hangup) endpos=8160

All of the other commands appear to exit with -1 on hangup.

This affects both trunk and 1.4.  I didn't check 1.2
Comments:By: Digium Subversion (svnbot) 2008-01-11 13:26:24.000-0600

Repository: asterisk
Revision: 98317

U   branches/1.4/res/res_agi.c

------------------------------------------------------------------------
r98317 | file | 2008-01-11 13:26:22 -0600 (Fri, 11 Jan 2008) | 6 lines

If the channel is hungup during RECORD FILE send a result code of -1 to be uniform with everything else.
(closes issue ASTERISK-11212)
Reported by: davevg
Patches:
     res_agi.diff uploaded by davevg (license 209)

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

http://svn.digium.com/view/asterisk?view=rev&revision=98317
By: Digium Subversion (svnbot) 2008-01-11 13:28:27.000-0600

Repository: asterisk
Revision: 98318

_U  trunk/
U   trunk/res/res_agi.c

------------------------------------------------------------------------
r98318 | file | 2008-01-11 13:28:26 -0600 (Fri, 11 Jan 2008) | 14 lines

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

........
r98317 | file | 2008-01-11 15:28:30 -0400 (Fri, 11 Jan 2008) | 6 lines

If the channel is hungup during RECORD FILE send a result code of -1 to be uniform with everything else.
(closes issue ASTERISK-11212)
Reported by: davevg
Patches:
     res_agi.diff uploaded by davevg (license 209)

........

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

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