]> Git trees. - libqmvoc.git/commitdiff
Fix potential mem leak
authorLaurent Montel <montel@kde.org>
Sun, 20 Jul 2008 15:31:14 +0000 (15:31 +0000)
committerLaurent Montel <montel@kde.org>
Sun, 20 Jul 2008 15:31:14 +0000 (15:31 +0000)
svn path=/trunk/KDE/kdeedu/libkdeedu/; revision=835439

keduvocdocument/keduvocdocument.cpp

index 97a17f81bb8e0a680ed42a58074198613e03b88a..564a4cd56a3d4623079b7d540fa3836db7c37e57 100644 (file)
@@ -108,14 +108,10 @@ KEduVocDocument::KEduVocDocumentPrivate::~KEduVocDocumentPrivate()
 
 void KEduVocDocument::KEduVocDocumentPrivate::init()
 {
-    if ( m_lessonContainer ) {
-        delete m_lessonContainer;
-    }
+    delete m_lessonContainer;
     m_lessonContainer = new KEduVocLesson(i18nc("The top level lesson which contains all other lessons of the document.", "Document Lesson"));
     m_lessonContainer->setContainerType(KEduVocLesson::Lesson);
-    if ( m_wordTypeContainer ) {
-        delete m_wordTypeContainer;
-    }
+    delete m_wordTypeContainer;
     m_wordTypeContainer = new KEduVocWordType(i18n( "Word types" ));
 
     m_tenseDescriptions.clear();
@@ -253,6 +249,7 @@ int KEduVocDocument::open( const KUrl& url )
 
         if ( !f->open( QIODevice::ReadOnly ) ) {
             kError() << errorMessage;
+            delete f;
             return FileCannotRead;
         }
 
@@ -337,6 +334,7 @@ int KEduVocDocument::open( const KUrl& url )
             QString msg = i18n( "Could not open or properly read \"%1\"\n(Error reported: %2)", url.path(), errorMessage );
             kError() << msg << i18n( "Error Opening File" );
             ///@todo make the readers return int, pass on the error message properly
+            delete f;
             return FileReaderFailed;
         }