[asterisk-dev] [Code Review] 3809: features config: prevent assert on no featuregroups or applicationmap

Scott Griepentrog reviewboard at asterisk.org
Wed Jul 16 14:50:47 CDT 2014


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

Review request for Asterisk Developers.


Repository: Asterisk


Description
-------

During testing, encountered an assert due to a null featuregroup passed to ao2_find().  This patch adds null checks around cfg->featuregroups and cfg->applicationmap, since they are not duplicated or initialized when features are copied.


Diffs
-----

  /team/group/media_formats-reviewed-trunk/main/features_config.c 418781 

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


Testing
-------

Repeated previously asserting test sequence with success.


Thanks,

Scott Griepentrog

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140716/8914ae80/attachment.html>


More information about the asterisk-dev mailing list