| Summary: | ASTERISK-22565: [patch] res_rtp_asterisk leaks reference to rtcp_report in ast_rtcp_read | ||
| Reporter: | Corey Farrell (coreyfarrell) | Labels: | |
| Date Opened: | 2013-09-20 23:17:07 | Date Closed: | 2013-09-20 23:59:03 |
| Priority: | Major | Regression? | |
| Status: | Closed/Complete | Components: | Resources/res_rtp_asterisk |
| Versions: | SVN 12.0.0-alpha1 | Frequency of Occurrence | Constant |
| Related Issues: | |||
| Environment: | Attachments: | ( 0) rtcp_report-leak.patch | |
| Description: | Found leak of rtcp_report found by valgrind and REF_DEBUG. Also seems possible for message_blob to leak in the same way.
Moved rtcp_report RAII_VAR declaration into the loop so it is unref'ed after every loop. Moved message_blob to loop and switched it to a regular variable. The regular variable was used since message_blob is used in a very contained way. Patch applies to 12 and trunk. Tested a call with trunk (using chan_sip), verified the leak no longer occurs. | ||
| Comments: | |||