Summary: | ASTERISK-22414: [patch] voicemail and test_voicemail_api leaks | ||
Reporter: | Corey Farrell (coreyfarrell) | Labels: | |
Date Opened: | 2013-08-27 14:57:14 | Date Closed: | 2013-09-04 16:34:01 |
Priority: | Major | Regression? | |
Status: | Closed/Complete | Components: | Applications/app_voicemail |
Versions: | SVN 12 11.6.0 | Frequency of Occurrence | |
Related Issues: | |||
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. |