Summary: | ASTERISK-04721: [patch] Memory Leak(tm) in app_voicemail.c | ||
Reporter: | Anthony Minessale (anthm) | Labels: | |
Date Opened: | 2005-07-28 12:19:29 | Date Closed: | 2008-01-15 15:43:07.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Applications/app_voicemail |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) voicemail_leak.diff | |
Description: | There is a memory leak in app_voicemail static struct ast_config *voicemailCfg This pointer points at the last loaded voicemail config presumably to spare disk i/o. This issue is, when you reload, the pointer is moved to point at a newly loaded config object and disregards the config object it was previously pointing to. This patch makes sure the old object is destroyed. ****** ADDITIONAL INFORMATION ****** Disclaimer On file <-- for all those who say i don't disclaim any code. anthmct@yahoo.com | ||
Comments: | By: Mark Spencer (markster) 2005-07-28 14:07:49 All the global voicemailcfg has been removed. That code should never have been merged. By: Russell Bryant (russell) 2005-08-04 20:04:24 that stuff was merged after 1.0 By: Digium Subversion (svnbot) 2008-01-15 15:43:07.000-0600 Repository: asterisk Revision: 6242 U trunk/apps/app_voicemail.c ------------------------------------------------------------------------ r6242 | markster | 2008-01-15 15:43:07 -0600 (Tue, 15 Jan 2008) | 2 lines Remove per-context maxmsg which should never have made it in (bug ASTERISK-4721) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=6242 |