-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3222/
-----------------------------------------------------------

(Updated Feb. 15, 2014, 9:24 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Bugs: ASTERISK-23297
    https://issues.asterisk.org/jira/browse/ASTERISK-23297


Repository: Asterisk


Description
-------

Currently, if you load Asterisk with an empty extensions.conf, and you noload 
all other pbx modules (such as pbx_ael), then loading a later module - such as 
res_parking - that attempts to add something to the dialplan will explode.

This is because ast_merge_contexts_and_delete assumes there is at least one 
context loaded.

This patch modifies the routine to simply set the contexts_table/contexts 
pointers to the merged entries if they don't exist yet.

Note that this is probably a problem in 1.8/11, however, it probably never 
occurred due to features.c creating the parking extensions. Now that the core 
no longer does that, it's slightly more plausible to end up in this situation. 
It still wouldn't hurt to apply this patch to 1.8/11 as well.


Diffs
-----

  ./branches/12/main/pbx.c 407935 

Diff: https://reviewboard.asterisk.org/r/3222/diff/


Testing
-------

The slightly contrived scenario no longer crashes.


Thanks,

Matt Jordan

-- 
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

Reply via email to