[asterisk-bugs] [Asterisk 0017794]: [patch] segfault on dialplan reload

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17794 
====================================================================== 
Reported By:                PavelL
Assigned To:                tilghman
====================================================================== 
Project:                    Asterisk
Issue ID:                   17794
Category:                   Core/Configuration
Reproducibility:            always
Severity:                   crash
Priority:                   normal
Status:                     closed
Target Version:             1.6.2.13
Asterisk Version:           1.6.2.10 
JIRA:                       SWP-2008 
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:             2010-08-04 09:53 CDT
Last Modified:              2010-09-07 15:56 CDT
====================================================================== 
Summary:                    [patch] segfault on dialplan reload
Description: 
if context has syntax error like 
exten =>;
exten => 111;
asterisk always crashed on dialplan reload command 
here my extension.conf 
[general]

static=yes
writeprotect=no
autofallthrough=yes
clearglobalvars=no
priorityjumping=no

;[default]
;include => office-people

[macro-startrecord]
exten => s,1,Monitor(gsm,${UNIQUEID},b)

[office-people]

exten => _1[0123]XX,1,GotoIf(${DB_EXISTS(CHANPORT/${EXTEN})}?chpy:chpno)
exten => _1[0123]XX,n(chpy),Set(chanport=${DB(CHANPORT/${EXTEN})})
exten => _1[0123]XX,n,Goto(chpcont)
exten => _1[0123]XX,n(chpno),Set(chanport=SIP/${EXTEN})
exten =>; _1[0123]XX,n(chpcont),Noop
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ here error - but it does not
matter the position on wrong string  

exten => _1[0123]XX,n(main),Macro(startrecord)
exten =>
_1[0123]XX,n,GotoIf(${DB_EXISTS(VMREDIR/${EXTEN})}?vmredir:vmnoredir)
exten => _1[0123]XX,n(vmredir),Dial(SIP/${EXTEN},20,tTWw)
exten => _1[0123]XX,n,VoiceMail(${EXTEN},u)
exten => _1[0123]XX,n,Hangup
exten =>
_1[0123]XX,n(vmnoredir),GotoIf(${DB_EXISTS(REDIR/${EXTEN})}?redir:noredir)
exten => _1[0123]XX,n(redir),Dial(Local/${DB(REDIR/${EXTEN})}@office)
exten => _1[0123]XX,n(noredir),Dial(${chanport},20,tTWw)
exten => _1[0123]XX,n,GotoIf($[${DIALSTATUS} = BUSY]?hung:cont)
exten => _1[0123]XX,n(cont),GotoIf($[${DIALSTATUS} =
CHANUNAVAIL]?hung:cont2)
exten =>
_1[0123]XX,n(cont2),GotoIf(${DB_EXISTS(MOBY/${EXTEN})}?doit:wait)
exten => _1[0123]XX,n(doit),Playback(mob_go)
exten => _1[0123]XX,n,Set(number=${DB(MOB/${EXTEN})})
exten => _1[0123]XX,n,Set(CALLERID(num)=${EXTEN})
exten => _1[0123]XX,n,Goto(pstn-out,98${number},1)
exten => _1[1023]XX,n(wait),Wait(1)
exten => _1[0123]XX,n,Goto(main)
exten => _1[0123]XX,n(hung),Hangup

1.6.2.10 has this error too 
====================================================================== 

---------------------------------------------------------------------- 
 (0126702) svnbot (reporter) - 2010-09-07 15:56
 https://issues.asterisk.org/view.php?id=17794#c126702 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 285367

_U  branches/1.8/
U   branches/1.8/pbx/pbx_config.c

------------------------------------------------------------------------
r285367 | tilghman | 2010-09-07 15:56:08 -0500 (Tue, 07 Sep 2010) | 23
lines

Merged revisions 285366 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r285366 | tilghman | 2010-09-07 15:31:41 -0500 (Tue, 07 Sep 2010) | 16
lines
  
  Merged revisions 285365 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r285365 | tilghman | 2010-09-07 15:30:22 -0500 (Tue, 07 Sep 2010) | 9
lines
    
    Catch invalid extensions at the parser, instead of making the core
deal with them.
    
    (closes issue https://issues.asterisk.org/view.php?id=17794)
     Reported by: PavelL
     Patches: 
           20100820__issue17794__1.6.2.diff.txt uploaded by tilghman
(license 14)
           20100820__issue17794__1.4.diff.txt uploaded by tilghman
(license 14)
     Tested by: PavelL
  ........
................

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-09-07 15:56 svnbot         Checkin                                      
2010-09-07 15:56 svnbot         Note Added: 0126702                          
======================================================================