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

Jens-Michael Hoffmann jensmh at gmx.de
Sat Aug 1 10:34:48 CEST 2009


SVN commit 1005467 by jmhoffmann:

Fix crash in case no network plugin was found.
BUG: 201995


 M  +8 -2      HttpDownloadManager.cpp  


--- trunk/KDE/kdeedu/marble/src/lib/HttpDownloadManager.cpp #1005466:1005467
@@ -110,7 +110,8 @@
     sourceUrl.setPath( path + relativeUrlString );
 
     HttpJob *job = createJob( sourceUrl, relativeUrlString, id );
-    addJob( job );
+    if ( job )
+        addJob( job );
 }
 
 void HttpDownloadManager::addJob( const QUrl& sourceUrl, const QString& destFileName,
@@ -120,7 +121,8 @@
         return;
 
     HttpJob *job = createJob( sourceUrl, destFileName, id );
-    addJob( job );
+    if ( job )
+        addJob( job );
 }
 
 bool HttpDownloadManager::acceptJob( HttpJob  *job )
@@ -278,6 +280,10 @@
             qDeleteAll( networkPlugins );
             m_networkPlugin->setParent( this );
         }
+        else {
+            m_downloadEnabled = false;
+            return 0;
+        }
     }
     Q_ASSERT( m_networkPlugin );
     return m_networkPlugin->createJob( sourceUrl, destFileName, id );


More information about the Marble-commits mailing list