[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [messagelib/Applications/16.08] mimetreeparser/src/viewer: Fix memory leak when deleting MessageView
From:       David Faure <faure () kde ! org>
Date:       2016-07-31 20:26:27
Message-ID: E1bTxJb-0002Jc-It () code ! kde ! org
[Download RAW message or body]

Git commit fc7d6e69f398e8cef081acfcce60aee2e5977eed by David Faure.
Committed on 31/07/2016 at 20:26.
Pushed by dfaure into branch 'Applications/16.08'.

Fix memory leak when deleting MessageViewer and its NodeHelper.

The Interface::BodyPartMemento(s) were not deleted.

Since they are deleted when switching messages, this is no severe
leak, it just showed up in a unittest.

M  +1    -0    mimetreeparser/src/viewer/nodehelper.cpp

http://commits.kde.org/messagelib/fc7d6e69f398e8cef081acfcce60aee2e5977eed

diff --git a/mimetreeparser/src/viewer/nodehelper.cpp b/mimetreeparser/src/viewer/nodehelper.cpp
index ecf39f6..5942360 100644
--- a/mimetreeparser/src/viewer/nodehelper.cpp
+++ b/mimetreeparser/src/viewer/nodehelper.cpp
@@ -85,6 +85,7 @@ NodeHelper::~NodeHelper()
         delete mAttachmentFilesDir;
         mAttachmentFilesDir = 0;
     }
+    clear();
 }
 
 void NodeHelper::setNodeProcessed(KMime::Content *node, bool recurse)
[prev in list] [next in list] [prev in thread] [next in thread]