[Home]

Summary:ASTERISK-11785: [patch] AsyncAGI: AGI action with invalid channel closes AMI connection
Reporter:Stefan Reuter (srt)Labels:
Date Opened:2008-04-05 02:05:03Date Closed:2008-04-05 08:26:23
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Resources/res_agi
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) asterisk-12370.diff
Description:Sending an AGI action with an invalid channel through AMI reports a manager error. This is expected.
In addion it also closes the AMI connection which should not happen.
The attached patch fixed this.

****** ADDITIONAL INFORMATION ******

Action: AGI
Channel: SIP/123
Command: EXEC

Response: Error
Message: Channel SIP/123 does not exists or cannot get its lock

Connection closed by foreign host.
Comments:By: Digium Subversion (svnbot) 2008-04-05 08:19:33

Repository: asterisk
Revision: 112972

U   trunk/res/res_agi.c

------------------------------------------------------------------------
r112972 | tilghman | 2008-04-05 08:19:29 -0500 (Sat, 05 Apr 2008) | 6 lines

AsyncAGI should not close the manager session on error.
(closes issue ASTERISK-11785)
Reported by: srt
Patches:
      asterisk-12370.diff uploaded by srt (license 378)

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

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

By: Digium Subversion (svnbot) 2008-04-05 08:21:21

Repository: asterisk
Revision: 112973

_U  branches/1.6.0/

------------------------------------------------------------------------
r112973 | tilghman | 2008-04-05 08:21:21 -0500 (Sat, 05 Apr 2008) | 13 lines

Blocked revisions 112972 via svnmerge

........
r112972 | tilghman | 2008-04-05 08:24:12 -0500 (Sat, 05 Apr 2008) | 6 lines

AsyncAGI should not close the manager session on error.
(closes issue ASTERISK-11785)
Reported by: srt
Patches:
      asterisk-12370.diff uploaded by srt (license 378)

........

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

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

By: Digium Subversion (svnbot) 2008-04-05 08:26:23

Repository: asterisk
Revision: 112975

_U  branches/1.6.0/
U   branches/1.6.0/res/res_agi.c

------------------------------------------------------------------------
r112975 | tilghman | 2008-04-05 08:26:18 -0500 (Sat, 05 Apr 2008) | 14 lines

Merged revisions 112972 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
r112972 | tilghman | 2008-04-05 08:24:12 -0500 (Sat, 05 Apr 2008) | 6 lines

AsyncAGI should not close the manager session on error.
(closes issue ASTERISK-11785)
Reported by: srt
Patches:
      asterisk-12370.diff uploaded by srt (license 378)

........

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

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