Summary:ASTERISK-22414: [patch] voicemail and test_voicemail_api leaks
Reporter:Corey Farrell (coreyfarrell)Labels:
Date Opened:2013-08-27 14:57:14Date Closed:2013-09-04 16:34:01
Versions:SVN 12 11.6.0 Frequency of
Environment:Attachments:( 0) test_voicemail_api-leaks-11.patch
( 1) test_voicemail_api-leaks-12up.patch
( 2) voicemail-leaks-11up.patch
Description:app_voicemail.c: message_range_and_existence_check() leaks config object's when msg_id doesn't match.

test_voicemail_api.c: mock snapshot object's are ast_free'd, they need a destructor to cleanup stringfields.

Patch for asterisk 12 is also for trunk.
Comments:By: Corey Farrell (coreyfarrell) 2013-08-27 15:05:21.782-0500

Test run done for 11.  Compile only test for 12/trunk.

By: Jonathan Rose (jrose) 2013-09-04 16:30:04.813-0500

I agree with these patches and am currently committing them. I've made a small adjustment on the tests in that I've made the allocation function for snapshots release the object and return NULL if the string field can't be allocated.