Opened 14 years ago

Closed 14 years ago

Last modified 13 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

Description

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@… 14 years ago.

Download all attachments as: .zip

Change History (2)

Changed 14 years ago by neil@…

Attachment: scheduler.diff added

comment:1 Changed 14 years ago by Isaac Richards

Resolution: fixed
Status: newclosed

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

Closes #18.

Note: See TracTickets for help on using tickets.