Summary: | ASTERISK-05557: [patch] Memleak in app_rpt | ||
Reporter: | Tilghman Lesher (tilghman) | Labels: | |
Date Opened: | 2005-11-10 12:03:14.000-0600 | Date Closed: | 2008-01-15 15:55:16.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Applications/app_rpt |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) 20051110__app_rpt_memleak__2.diff.txt ( 1) 20051110__app_rpt_memleak.diff.txt | |
Description: | app_rpt does not properly call ast_config_destroy to deallocate memory. | ||
Comments: | By: hwstar (hwstar) 2005-11-10 14:55:23.000-0600 Adding the first two ast_config_destroys() in your patch will not break anything, but the third one will. App_rpt needs access to the *cfg while it is running as we access settings located in *cfg in real time. Steve. By: Tilghman Lesher (tilghman) 2005-11-10 15:24:25.000-0600 How about if we move it down to just above the pthread_exit ? By: Kevin P. Fleming (kpfleming) 2005-11-10 18:05:13.000-0600 Committed to CVS HEAD, thanks! By: Digium Subversion (svnbot) 2008-01-15 15:55:16.000-0600 Repository: asterisk Revision: 7049 U trunk/ChangeLog U trunk/apps/app_rpt.c ------------------------------------------------------------------------ r7049 | kpfleming | 2008-01-15 15:55:15 -0600 (Tue, 15 Jan 2008) | 2 lines issue ASTERISK-5557 ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=7049 |