[asterisk-bugs] [Asterisk 0015858]: [patch] Revision 152765 introduces scoping difficulties in stdexten

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15858 
====================================================================== 
Reported By:                pprindeville
Assigned To:                lmadsen
====================================================================== 
Project:                    Asterisk
Issue ID:                   15858
Category:                   Core/Configuration
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Asterisk Version:           1.6.1.5 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-09-08 18:59 CDT
Last Modified:              2009-11-03 13:31 CST
====================================================================== 
Summary:                    [patch] Revision 152765 introduces scoping
difficulties in stdexten
Description: 
The new [stdexten] macro doesn't always seem to be properly scoped.

Testing the redfish config below, I get:

...
    -- Executing [s at redfish-pstn:27] Gosub("SIP/sip_proxy-0847e6d8",
"stdexten(111,SIP/bedroom_1&SIP/office_2&SIP/remote_2&SIP/kitchen_1&Local/2088919324 at follow-me&SIP/softphone,redfish)")
in new stack
[Sep  8 17:34:14] NOTICE[7503]: pbx.c:3233 pbx_extension_helper: No such
label 'stdexten' in extension 's' in context 'redfish-pstn'
[Sep  8 17:34:14] WARNING[7503]: pbx.c:8998 pbx_parseable_goto: Priority
'stdexten' must be a number > 0, or valid label
[Sep  8 17:34:14] ERROR[7503]: app_stack.c:279 gosub_exec: Gosub address
is invalid:
'stdexten(111,SIP/bedroom_1&SIP/office_2&SIP/remote_2&SIP/kitchen_1&Local/2088919324 at follow-me&SIP/softphone,redfish)'
  == Spawn extension (redfish-pstn, s, 27) exited non-zero on
'SIP/sip_proxy-0847e6d8'

on externally originating calls, even though it seems to work for internal
calls.

Both the [redfish-internal] and the [redfish-pstn] contexts include the
[redfish-extens] context.

Reverting part of the context to revision 147896 I once again see
success:

...
    -- Executing [s at redfish-pstn:27] Gosub("SIP/sip_proxy-08493c08",
"stdexten,s,1(111,SIP/bedroom_1&SIP/office_2&SIP/remote_2&SIP/kitchen_1&Local/2088919324 at follow-me&SIP/softphone,redfish)")
in new stack
    -- Executing [s at stdexten:1] NoOp("SIP/sip_proxy-08493c08", "Start
stdexten") in new stack
...




======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
has duplicate       0015857 [patch] Revision 152765 introduces regr...
====================================================================== 

---------------------------------------------------------------------- 
 (0113074) svnbot (reporter) - 2009-11-03 13:31
 https://issues.asterisk.org/view.php?id=15858#c113074 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 227362

_U  branches/1.6.0/
U   branches/1.6.0/configs/extensions.conf.sample

------------------------------------------------------------------------
r227362 | lmadsen | 2009-11-03 13:31:09 -0600 (Tue, 03 Nov 2009) | 19
lines

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

........
  r227361 | lmadsen | 2009-11-03 13:25:18 -0600 (Tue, 03 Nov 2009) | 11
lines
  
  Additional fixes to the extensions.conf.sample file.
  
  Update the extensions.conf.sample [stdexten] context so that we use the 
  variable instead of requiring it to be passed explicitly. Also updated
uses of
  the [stdexten] context throughout.
  
  (closes issue https://issues.asterisk.org/view.php?id=15858)
  Reported by: pprindeville
  Patches:
        stdexten-context-update.txt uploaded by lmadsen (license 10)
  Tested by: pprindeville
........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-11-03 13:31 svnbot         Checkin                                      
2009-11-03 13:31 svnbot         Note Added: 0113074                          
======================================================================