[Home]

Summary:ASTERISK-09945: [patch] [res_agi] more fixes and love, should be now completly thread and usecount safe.
Reporter:Donny Kavanagh (donnyk)Labels:
Date Opened:2007-07-24 09:54:18Date Closed:2007-07-30 13:54:07
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Resources/res_agi
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 10288-v2.patch
Description:1) Added ast_agi_(un)register_multiple (works like ast_cli_(un)register_multiple
2) renamed a variable in ast_agi_register which was incorrectly named.
3) removed MAX_COMMANDS
4) moved/created a couple of defines in agi.h
5) remove use of AST_MAX_CMD_LEN, i highly doubt anyone will be registering an agi command with 15 separate words.  Created a new define for this 'AGI_MAX_DEPTH' set it to a more conservative 8.
6) added agi_debug_cli to the agi.h such that external modules which may want to do agi debugging can do so.

res_agi should now be completely thread safe, and usecount safe.

Comments:By: Donny Kavanagh (donnyk) 2007-07-24 23:52:51

it would be pretty awesome in the future if i remembered to attach the file.
By: Donny Kavanagh (donnyk) 2007-07-25 23:26:04

new version with changes as requested by corydon on irc.

1) removed some unnecessary defines
2) renamed a function to meet proper naming convention for non-static functions.
By: Digium Subversion (svnbot) 2007-07-30 13:54:07

Repository: asterisk
Revision: 77787

------------------------------------------------------------------------
r77787 | tilghman | 2007-07-30 13:54:06 -0500 (Mon, 30 Jul 2007) | 2 lines

Cleanup of res_agi, ensuring thread safety (closes issue ASTERISK-9945)

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