[Spatialite provider] Properly destroy QgsSpatiaLiteConnPool singleto…

…n at provider unloading. Should fix frequent crash in PyQgsServerWMS on Travis (fixes #17447)
qgis · Nov 15, 2017 · 581d0d3 · 581d0d3 · pblottiere · Nov 16, 2017
1 parent 7fa11c8
commit 581d0d3
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 3 deletions.
diff --git a/src/providers/spatialite/qgsspatialiteconnpool.cpp b/src/providers/spatialite/qgsspatialiteconnpool.cpp
@@ -13,11 +13,30 @@
  *                                                                         *
  ***************************************************************************/
 
+#include <QMutex>
+#include <QMutexLocker>
+
 #include "qgsspatialiteconnpool.h"
 
-QgsSpatiaLiteConnPool QgsSpatiaLiteConnPool::sInstance;
+QgsSpatiaLiteConnPool *QgsSpatiaLiteConnPool::sInstance = nullptr;
 
 QgsSpatiaLiteConnPool *QgsSpatiaLiteConnPool::instance()
 {
-  return &sInstance;
+  if ( ! sInstance )
+  {
+    static QMutex sMutex;
+    QMutexLocker locker( &sMutex );
+    if ( ! sInstance )
+    {
+      sInstance = new QgsSpatiaLiteConnPool();
+    }
+  }
+  return sInstance;
+}
+
+// static public
+void QgsSpatiaLiteConnPool::cleanupInstance()
+{
+  delete sInstance;
+  sInstance = nullptr;
 }
diff --git a/src/providers/spatialite/qgsspatialiteconnpool.h b/src/providers/spatialite/qgsspatialiteconnpool.h
@@ -68,9 +68,19 @@ class QgsSpatiaLiteConnPoolGroup : public QObject, public QgsConnectionPoolGroup
 //! SpatiaLite connection pool - singleton
 class QgsSpatiaLiteConnPool : public QgsConnectionPool<QgsSqliteHandle *, QgsSpatiaLiteConnPoolGroup>
 {
-    static QgsSpatiaLiteConnPool sInstance;
+    static QgsSpatiaLiteConnPool *sInstance;
   public:
     static QgsSpatiaLiteConnPool *instance();
+
+    // Singleton cleanup
+    //
+    // Make sure nobody is using the instance before calling
+    // this function.
+    //
+    // WARNING: concurrent call from multiple threads may result
+    //          in double-free of the instance.
+    //
+    static void cleanupInstance();
 };
 
 

diff --git a/src/providers/spatialite/qgsspatialiteprovider.cpp b/src/providers/spatialite/qgsspatialiteprovider.cpp
@@ -5900,6 +5900,7 @@ QGISEXTERN QString getStyleById( const QString &uri, QString styleId, QString &e
 
 QGISEXTERN void cleanupProvider()
 {
+  QgsSpatiaLiteConnPool::cleanupInstance();
   QgsSqliteHandle::closeAll();
 }