| Summary: | ASTERISK-22368: [patch] mixmonitor_free leaks filename | ||
| Reporter: | Corey Farrell (coreyfarrell) | Labels: | |
| Date Opened: | 2013-08-22 17:34:04 | Date Closed: | 2013-08-30 11:19:03 |
| Priority: | Minor | Regression? | |
| Status: | Closed/Complete | Components: | Applications/app_mixmonitor |
| Versions: | SVN 12 11.6.0 | Frequency of Occurrence | Constant |
| Related Issues: | |||
| Environment: | Attachments: | ( 0) issueA22368_mixmonitor_free_filename.patch | |
| Description: | mixmonitor_free() doesn't free mixmonitor->filename. | ||
| Comments: | By: Corey Farrell (coreyfarrell) 2013-08-22 17:37:46.106-0500 Tested against 11. Does not apply to 1.8 since filename was not separately allocated. By: Walter Doekes (wdoekes) 2013-08-23 02:46:14.858-0500 I reordered it a bit for clarity. But yes, I say ship it. (Leak was introduced in v10 btw.) By: Corey Farrell (coreyfarrell) 2013-08-28 18:42:32.549-0500 I've removed my patch, the reorder done by Walter is better so his patch should be used instead. | ||