Opened 16 years ago

Closed 16 years ago

Last modified 14 years ago

#18 closed patch (fixed)

Fix memory leak in scheduler.cpp

Reported by: neil@… Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythtv Version: head
Severity: low Keywords:
Cc: Ticket locked: no


scheduler.cpp appears to have a memory leak in getAllPending(QStringList &strList) - retList is allocated but never freed. The leak won't be obvious unless you're calling getAllPending fairly frequently (I have a monitoring script that calls it twice a minute). The attached patch fixes it for me. It was diff'd against CVS from 15/06 (I haven't moved to svn yet), although hopefully it will patch cleanly against current head.

Attachments (1)

scheduler.diff (550 bytes) - added by neil@… 16 years ago.

Download all attachments as: .zip

Change History (2)

Changed 16 years ago by neil@…

Attachment: scheduler.diff added

comment:1 Changed 16 years ago by Isaac Richards

Resolution: fixed
Status: newclosed

(In [6725]) Simpler variation of the patch from neil @ Fixes a leak in Scheduler::getAllPending.

Closes #18.

Note: See TracTickets for help on using tickets.