]> cloud.milkyroute.net Git - dolphin.git/commitdiff
Fixes memory leak of KItemListHeader
authorDavid Hallas <david@davidhallas.dk>
Sun, 29 Jul 2018 10:10:16 +0000 (12:10 +0200)
committerDavid Hallas <david@davidhallas.dk>
Sun, 24 Feb 2019 17:22:55 +0000 (18:22 +0100)
Summary:
Fixes memory leak of KItemListHeader
The KItemListHeader passed the listView parent object to the QObject
base class, but that pointer seems to always be nullptr causing the
KItemListHeader to not be memory managed by anything. Instead simple
use the listView as parent pointer.

Test Plan: This leak was found using Address Sanitizer

Reviewers: elvisangelaccio

Reviewed By: elvisangelaccio

Subscribers: elvisangelaccio, kfm-devel

Tags: #dolphin

Differential Revision: https://phabricator.kde.org/D14453

src/kitemviews/kitemlistheader.cpp

index cfc2c2b729177feec126c285f6c142a45d50c27c..a82af44e764eb4184d44901d737f8e4c71f19387 100644 (file)
@@ -74,7 +74,7 @@ qreal KItemListHeader::preferredColumnWidth(const QByteArray& role) const
 }
 
 KItemListHeader::KItemListHeader(KItemListView* listView) :
 }
 
 KItemListHeader::KItemListHeader(KItemListView* listView) :
-    QObject(listView->parent()),
+    QObject(listView),
     m_view(listView)
 {
     m_headerWidget = m_view->m_headerWidget;
     m_view(listView)
 {
     m_headerWidget = m_view->m_headerWidget;