[asterisk-dev] [Code Review] 2443: Fix app_voicemail segfaulting and a few memory leaks

[Michael Young]

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

(Updated April 12, 2013, 6:25 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers and Jaco Kroon.


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


Repository: Asterisk


Description
-------

This patch is the result of a combined effort by Jaco (jkroon) and myself.

The original report was that app_voicemail would crash when there was an error generated by having an invalid configuration file.

A patch was created to solve that problem by checking that the loaded configuration file was not invalid.

Jaco pointed out some areas where there were some memory leaks as well in app_voicemail.


Diffs
-----

  branches/11/apps/app_voicemail.c 384682 

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


Testing
-------

Tested on my dev machine after reproducing the crash.

I think that Jaco has also tested this patch out in his environment.


Thanks,

Michael Young

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130412/8ff29c01/attachment.htm>