Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#13438 closed Bug Report - Memory Leak (fixed)

Potential memory leak in RecorderBase::CheckForRingBufferSwitch

Reported by: David Hampton Owned by: David Hampton
Priority: minor Milestone: 30.1
Component: MythTV - Recording Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

At the end of the RecorderBase::CheckForRingBufferSwitch function, m_tvrec->RingBufferChanged is called with recq as an argument, and that function ends up freeing the memory pointed to by recq. Its theoretically possible to get to this point and have m_tvrec be a nullptr, in which case the function isn't called and recq isn't freed. The code needs to delete recq in that case.

Change History (2)

comment:1 by David Hampton <mythtv@…>, 7 years ago

Resolution: → fixed
Status: assigned → closed

In e6a39281b/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:2 by David Hampton <mythtv@…>, 7 years ago

In 3af3cad3d7/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available
Note: See TracTickets for help on using tickets.