[Home]

Summary:ASTERISK-05437: error parsing multiple variables in Manager Originate
Reporter:Abhishek Tiwari (abhi)Labels:
Date Opened:2005-11-03 06:55:14.000-0600Date Closed:2008-01-15 15:54:41.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/ManagerInterface
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) manager.c.diff
Description:The function astman_get_variables assumes only single variable in the Variables line of the manager command and breaks the older functionality. Attached patch fixes this to read multiple variables (seperated by "|").
Comments:By: Russell Bryant (russell) 2005-11-03 08:06:01.000-0600

If we're going to make this backwards compatible, it needs to be aware of parenthesis.  You can set dialplan functions that use '|' to seperate arguments.  So, it should only split them when not inside parenthesis.
By: Russell Bryant (russell) 2005-11-03 08:08:59.000-0600

As a matter of fact, you can just use ast_separate_app_args to do all of this for you.
By: Kevin P. Fleming (kpfleming) 2005-11-07 21:20:54.000-0600

Fixed in CVS HEAD using existing parser.
By: Digium Subversion (svnbot) 2008-01-15 15:54:41.000-0600

Repository: asterisk
Revision: 7012

U   trunk/ChangeLog
U   trunk/manager.c

------------------------------------------------------------------------
r7012 | kpfleming | 2008-01-15 15:54:40 -0600 (Tue, 15 Jan 2008) | 2 lines

issue ASTERISK-5437

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

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