[Home]

Summary:ASTERISK-04204: [patch] Old style of app,appdata not parsed correctly when appdata contains a (
Reporter:Brian West (bkw918)Labels:
Date Opened:2005-05-17 10:36:16Date Closed:2011-06-07 14:10:50
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20050517__fix_4306.diff.txt
Description:exten => 578,1,NoOp(${DB(TEST/blah)})
exten => 578,2,NoOp,${DB(TEST/blah)}

priority 1 works fine.
priority 2 results in:

No application 'NoOp,${DB' for extension (default, 578, 2)

Tiny parse error I say in pbx_config.c  I'll see if I have time to find it.. if not its here for documentation.

Either we fix this or depricate the abilitiy to not use () on app's
/b
Comments:By: Tilghman Lesher (tilghman) 2005-05-17 10:48:41

Easy fix.  The problem is that parsing of parentheses took precedence over commas, instead of checking which came first.
By: Russell Bryant (russell) 2005-05-18 21:14:54

fixed in cvs head, thanks!
By: Digium Subversion (svnbot) 2008-01-15 15:35:23.000-0600

Repository: asterisk
Revision: 5721

U   trunk/pbx/pbx_config.c

------------------------------------------------------------------------
r5721 | russell | 2008-01-15 15:35:22 -0600 (Tue, 15 Jan 2008) | 3 lines

check to see if a comma or an open paren came first when splitting the application
from the application arguments (bug ASTERISK-4204)

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

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