[mythtv-commits] Ticket #10807: [PATCH] libmythui: Prevent memory leak and dangling reference to MythUIType children

[MythTV]

#10807: [PATCH] libmythui: Prevent memory leak and dangling reference to MythUIType
children
------------------------------------+-----------------------------
 Reporter:  Lawrence Rust <lvr@…>   |          Owner:
     Type:  Patch - Bug Fix         |         Status:  new
 Priority:  minor                   |      Milestone:  unknown
Component:  MythTV - General        |        Version:  Master Head
 Severity:  medium                  |     Resolution:
 Keywords:  MythUIType memory leak  |  Ticket locked:  0
------------------------------------+-----------------------------

Comment (by Lawrence Rust <lvr@…>):

 {{{
 ==9163== 96 bytes in 6 blocks are possibly lost in loss record 1,446 of
 2,430
 ==9163==    at 0x4026864: malloc (vg_replace_malloc.c:236)
 ==9163==    by 0x7B6765C: qMalloc(unsigned int) (qmalloc.cpp:55)
 ==9163==    by 0x7B99611: QMapData::node_create(QMapData::Node**, int,
 int) (qmap.cpp:140)
 ==9163==    by 0x5857220: QMap<int, MythUIType*>::node_create(QMapData*,
 QMapData::Node**, int const&, MythUIType* const&) (qmap.h:449)
 ==9163==    by 0x586E1B8: QMap<int, MythUIType*>::operator[](int const&)
 (qmap.h:530)
 ==9163==    by 0x586CD9D:
 MythUIStateType::AddObject(MythUIStateType::StateType, MythUIType*)
 (mythuistatetype.cpp:78)
 ==9163==    by 0x586DCAC: MythUIStateType::CopyFrom(MythUIType*)
 (mythuistatetype.cpp:286)
 ==9163==    by 0x586DDC1: MythUIStateType::CreateCopy(MythUIType*)
 (mythuistatetype.cpp:294)
 ==9163==    by 0x5855252: MythUIType::CopyFrom(MythUIType*)
 (mythuitype.cpp:838)
 ==9163==    by 0x589ED4C: MythUIGroup::CopyFrom(MythUIType*)
 (mythuigroup.cpp:24)
 ==9163==    by 0x5835D38: XMLParseBase::ParseUIType(QString const&,
 QDomElement&, QString const&, MythUIType*, MythScreenType*, bool)
 (xmlparsebase.cpp:499)
 ==9163==    by 0x586D7E1: MythUIStateType::ParseElement(QString const&,
 QDomElement&, bool) (mythuistatetype.cpp:229)
 ==9163==
 ==9163== 96 bytes in 6 blocks are possibly lost in loss record 1,447 of
 2,430
 ==9163==    at 0x4026864: malloc (vg_replace_malloc.c:236)
 ==9163==    by 0x7B6765C: qMalloc(unsigned int) (qmalloc.cpp:55)
 ==9163==    by 0x7B99611: QMapData::node_create(QMapData::Node**, int,
 int) (qmap.cpp:140)
 ==9163==    by 0x586E6D5: QMap<QString,
 MythUIType*>::node_create(QMapData*, QMapData::Node**, QString const&,
 MythUIType* const&) (qmap.h:449)
 ==9163==    by 0x586E0F8: QMap<QString, MythUIType*>::operator[](QString
 const&) (qmap.h:530)
 ==9163==    by 0x586C9A9: MythUIStateType::AddObject(QString const&,
 MythUIType*) (mythuistatetype.cpp:46)
 ==9163==    by 0x586DBAF: MythUIStateType::CopyFrom(MythUIType*)
 (mythuistatetype.cpp:275)
 ==9163==    by 0x586DDC1: MythUIStateType::CreateCopy(MythUIType*)
 (mythuistatetype.cpp:294)
 ==9163==    by 0x5855252: MythUIType::CopyFrom(MythUIType*)
 (mythuitype.cpp:838)
 ==9163==    by 0x58973FD: MythUIButtonList::CopyFrom(MythUIType*)
 (mythuibuttonlist.cpp:2431)
 ==9163==    by 0x588A816: MythUISpinBox::CopyFrom(MythUIType*)
 (mythuispinbox.cpp:160)
 ==9163==    by 0x5835D38: XMLParseBase::ParseUIType(QString const&,
 QDomElement&, QString const&, MythUIType*, MythScreenType*, bool)
 (xmlparsebase.cpp:499)
 }}}

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/10807#comment:2>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center