[Asterisk-bugs] [Asterisk 0010270]: [patch] more res_agi updates, upgrades and fixes

[noreply at bugs.digium.com]

The following issue has been UPDATED. 
====================================================================== 
http://bugs.digium.com/view.php?id=10270 
====================================================================== 
Reported By:                juggie
Assigned To:                juggie
====================================================================== 
Project:                    Asterisk
Issue ID:                   10270
Category:                   Resources/res_agi
Reproducibility:            N/A
Severity:                   tweak
Priority:                   normal
Status:                     feedback
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             07-22-2007 16:27 CDT
Last Modified:              07-22-2007 16:55 CDT
====================================================================== 
Summary:                    [patch] more res_agi updates, upgrades and fixes
Description: 
This patch does the following:

1) Makes res_agi and its memory space thread safe, implements linked list
macros rather then a stuct array.
2) opens the door for other asterisk modules registering/unregistering agi
commands (this was previously possible, but do to the way it was coded, a
little dangerous, furthermore this is why linked list was chosen rather
then making the existing array thread safe)
3) Deprecates DeadAGI, now you call AGI on a live or dead channel, and
EAGI only on a live channel.  DeadAGI however still works and displays a
warning as such.  As well, running DeadAGI on a live channel continues to
operate just as before so there is no functionality change over 1.4 with
the exception that when the channel hangs up, the call to AGI will become
dead access to asterisk rather then cut off your access.  This change comes
from corydon and can be found in bug http://bugs.digium.com/view.php?id=10245,
its been incorporated into
this patch. (thanks corydon)
4) numerous cli level bug fixes.
5) commands are now flagged with the ability to run on a dead channel or
not.  attempting to run a command intended for a live channel only will
result in a new agi error '511 Command Not Permitted on a dead channel'
6) agi show via the cli will display what can and cannot be run if the
channel is dead.
7) one function had to be moved in the source, this made the patch quite
hard to read, i apologize.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
parent of           0010245 Agi in live & dead channels? Don't ...
====================================================================== 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
07-22-07 16:55  juggie         Description Updated                          
======================================================================