Opened 14 years ago

Closed 14 years ago

#140 closed patch (fixed)

Patch for a memory leak in lcddevice.cpp

Reported by: eskil <myth@…> Owned by: Isaac Richards
Priority: trivial Milestone: unknown
Component: mythtv Version: head
Severity: low Keywords: leak
Cc: Ticket locked: no

Description

While working on something else and painstakingly valgrinding mythfrontend, I encountered ;

==10470== 2688 bytes in 112 blocks are definitely lost in loss record 650 of 730 ==10470== at 0x1B90506F: operator new(unsigned) (vg_replace_malloc.c:133) ==10470== by 0x1C427A0C: LCD::startMenu(QPtrList<LCDMenuItem>*, QString, bool) (lcddevice.cpp:1013) ==10470== by 0x1C42A781: LCD::switchToMenu(QPtrList<LCDMenuItem>*, QString, bool) (lcddevice.cpp:1606) ==10470== by 0x1C48380F: ThemedMenuPrivate::updateLCD() (themedmenu.cpp:1622)

The attached patch basically sets autodelete on in the ptrlist, fixes the leak.

Attachments (1)

lcddevice.patch (480 bytes) - added by eskil <myth@…> 14 years ago.
patch for lcddevice.cpp

Download all attachments as: .zip

Change History (2)

Changed 14 years ago by eskil <myth@…>

Attachment: lcddevice.patch added

patch for lcddevice.cpp

comment:1 Changed 14 years ago by Isaac Richards

Resolution: fixed
Status: newclosed

(In [6982]) Close #140 by applying patch from 'eskil'.

Note: See TracTickets for help on using tickets.