Fix crash in network content fetcher when run from non-main thread

Because a QNetworkAccessManager is the parent for all QNetworkReplys created by it, if a reply is created in a thread then we need to ensure that this reply isn't deleted early by destruction of the thread local QgsNetworkAcessManager instance. Work around this by unsetting the parent for the QNetworkReplys owned by QgsNetworkContentFetcher objects. Fixes #19452 (cherry-picked from 7f84b6c)
qgis · Jul 28, 2018 · 1d28bbe · 1d28bbe
1 parent f20fec4
commit 1d28bbe
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/src/core/qgsnetworkcontentfetcher.cpp b/src/core/qgsnetworkcontentfetcher.cpp
@@ -55,6 +55,7 @@ void QgsNetworkContentFetcher::fetchContent( const QNetworkRequest &request )
   }
 
   mReply = QgsNetworkAccessManager::instance()->get( request );
+  mReply->setParent( nullptr ); // we don't want thread locale QgsNetworkAccessManagers to delete the reply - we want ownership of it to belong to this object
   connect( mReply, &QNetworkReply::finished, this, [ = ] { contentLoaded(); } );
   connect( mReply, &QNetworkReply::downloadProgress, this, &QgsNetworkContentFetcher::downloadProgress );
 }