[Marble-devel] KDE/kdeedu/marble/src/lib

[Michael Jansen]

SVN commit 998750 by mjansen:

Fix memory leak.

==7385== 1,904 (192 direct, 1,712 indirect) bytes in 2 blocks are definitely lost in loss record 218 of 508
==7385==    at 0x4C2506C: operator new(unsigned long) (in /usr/lib64/valgrind/amd64-linux/vgpreload_memcheck.so)
==7385==    by 0x5416B6B: Marble::AbstractDataPluginModel::AbstractDataPluginModel(QString const&, QObject*) (AbstractDataPluginModel.cpp:108)
==7385==    by 0x149DB04E: Marble::WikipediaModel::WikipediaModel(QObject*) (WikipediaModel.cpp:39)
==7385==    by 0x149D8B28: Marble::WikipediaPlugin::initialize() (WikipediaPlugin.cpp:79)
==7385==    by 0x53D241B: Marble::LayerManager::LayerManager(Marble::MarbleDataFacade*, QObject*) (LayerManager.cpp:79)
==7385==    by 0x539F539: Marble::MarbleModel::MarbleModel(QObject*) (MarbleModel.cpp:151)
==7385==    by 0x53A49A9: Marble::MarbleMap::MarbleMap() (MarbleMap.cpp:350)
==7385==    by 0x539B493: Marble::MarbleWidget::MarbleWidget(QWidget*) (MarbleWidget.cpp:129)
==7385==    by 0x410AD4: Marble::ControlView::ControlView(QWidget*) (ControlView.cpp:43)
==7385==    by 0x41A7CD: Marble::MarblePart::MarblePart(QWidget*, QObject*, QStringList const&) (marble_part.cpp:114)
==7385==    by 0x41515B: Marble::MainWindow::MainWindow(QString const&, QWidget*) (KdeMainWindow.cpp:40)
==7385==    by 0x414956: main (kdemain.cpp:244)

CCMAIL:marble-devel at kde.org

 M  +4 -1      AbstractDataPluginModel.cpp  


--- trunk/KDE/kdeedu/marble/src/lib/AbstractDataPluginModel.cpp #998749:998750
@@ -59,7 +59,8 @@
           m_lastDataFacade( 0 ),
           m_downloadTimer( new QTimer( m_parent ) ),
           m_descriptionFileNumber( 0 ),
-          m_itemSettings()
+          m_itemSettings(),
+          m_downloadManager( 0 )
     {
     }
     
@@ -76,6 +77,8 @@
         
         m_storagePolicy->clearCache();
         delete m_storagePolicy;
+
+        delete m_downloadManager;
     }
     
     AbstractDataPluginModel *m_parent;