[asterisk-bugs] [Asterisk 0013041]: Asterisk crash while unloading pbx_ael.so

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13041 
====================================================================== 
Reported By:                eliel
Assigned To:                murf
====================================================================== 
Project:                    Asterisk
Issue ID:                   13041
Category:                   Core/PBX
Reproducibility:            always
Severity:                   crash
Priority:                   normal
Status:                     assigned
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 129454 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             07-09-2008 15:45 CDT
Last Modified:              07-11-2008 13:17 CDT
====================================================================== 
Summary:                    Asterisk crash while unloading pbx_ael.so
Description: 
If we unload the first module that registered a context, then asterisk
crash.
How to reproduce it:
1) Check the first module in the 'contexts' hashtab.
      > dialplan show
        .......... [pbx_ael]       1st
        ............ [pbx_config]   2nd
        etc...
    So in this example pbx_ael is the first.
2) unload the first module that registered a context.
      > module unload pbx_ael.so
3.1) Asterisk crash.
3.2) Asterisk didn't crash but if you run 'dialplan show' you continue
seeing pbx_ael contexts that werent removed.

Backtrace full uploaded.
====================================================================== 

---------------------------------------------------------------------- 
 svnbot - 07-11-08 13:17  
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 130145

U   trunk/main/pbx.c

------------------------------------------------------------------------
r130145 | murf | 2008-07-11 13:17:12 -0500 (Fri, 11 Jul 2008) | 40 lines

(closes issue http://bugs.digium.com/view.php?id=13041)
Reported by: eliel
Tested by: murf

(closes issue http://bugs.digium.com/view.php?id=12960)
Reported by: mnicholson

In this 'omnibus' fix, I **think** I solved both
the problem in 13041, where unloading pbx_ael.so
caused crashes, or incomplete removal of previous
registrar'ed entries. And I added code to completely
remove all includes, switches, and ignorepats that
had a matching registrar entry, which should
appease 12960.

I also added a lot of seemingly useless brackets
around single statement if's, which helped debug 
so much that I'm leaving them there.

I added a routine to check the correlation between
the extension tree lists and the hashtab 
tables. It can be amazingly helpful when you have
lots of dialplan stuff, and need to narrow
down where a problem is occurring. It's ifdef'd
out by default.

I cleaned up the code around the new CIDmatch code.
It was leaving hanging extens with bad ptrs, getting confused
over which objects to remove, etc. I tightened
up the code and changed the call to remove_exten
in the merge_and_delete code.

I added more conditions to check for empty context
worthy of deletion. It's not empty if there are
any includes, switches, or ignorepats present.

If I've missed anything, please re-open this bug,
and be prepared to supply example dialplan code.


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

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

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
07-11-08 13:17  svnbot         Note Added: 0090105                          
======================================================================