[asterisk-bugs] [Asterisk 0013343]: Local channel does not support exten/callerid style dialplan entries (ast_exists_extension placement).

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13343 
====================================================================== 
Reported By:                efutch
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13343
Category:                   Channels/chan_local
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           1.4.21.2 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-08-19 15:25 CDT
Last Modified:              2008-09-16 10:34 CDT
====================================================================== 
Summary:                    Local channel does not support exten/callerid style
dialplan entries (ast_exists_extension placement).
Description: 
local_alloc in chan_local.c calls ast_exists_extension before any CallerID
information for the local channel is populated.  Therefore, it cannot pass
the CallerID number to ast_exists_extensions like other channel techs do. 
So calls will not match the exten/callerid style dialplan entries like
these:
7002001234/_212NXXNXXXXX,1,Noop(Call from 212 area.)

Either the CallerID information should be populated before
ast_exists_extension in local_alloc is called, or the call to
ast_exists_extension to be placed somewhere after the CallerID information
is established in local_call.

This problem also seems to exist in 1.6, after quickly checking
chan_local.c
====================================================================== 

---------------------------------------------------------------------- 
 (0092562) efutch (reporter) - 2008-09-16 10:34
 http://bugs.digium.com/view.php?id=13343#c92562 
---------------------------------------------------------------------- 
This patch looks good to me.  The only problem with it is that tmp->context
and tmp->exten need to be p->chan->context and p->chan->exten for
ast_exists_extension in the first chunk.  You got them right on the
following line though (ast_log).  Once I changed that it compiled and
tested ok. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-09-16 10:34 efutch         Note Added: 0092562                          
======================================================================