[WMS provider] Fix crash on WMS-T layer uri without timeDimensionExtent

Fixes qgis#43158
rouault · May 20, 2021 · 4743ee5 · 4743ee5
1 parent 50aa9d4
commit 4743ee5
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/providers/wms/qgswmscapabilities.cpp b/src/providers/wms/qgswmscapabilities.cpp
@@ -90,7 +90,8 @@ bool QgsWmsSettings::parseUri( const QString &uriString )
     mTemporalExtent = uri.param( QStringLiteral( "timeDimensionExtent" ) );
     mTimeDimensionExtent = parseTemporalExtent( mTemporalExtent );
 
-    if ( !mTimeDimensionExtent.datesResolutionList.constFirst().dates.dateTimes.empty() )
+    if ( !mTimeDimensionExtent.datesResolutionList.isEmpty() &&
+         !mTimeDimensionExtent.datesResolutionList.constFirst().dates.dateTimes.empty() )
     {
       QDateTime begin = mTimeDimensionExtent.datesResolutionList.constFirst().dates.dateTimes.first();
       QDateTime end = mTimeDimensionExtent.datesResolutionList.constLast().dates.dateTimes.last();

diff --git a/tests/src/providers/testqgswmsprovider.cpp b/tests/src/providers/testqgswmsprovider.cpp
@@ -257,6 +257,13 @@ class TestQgsWmsProvider: public QObject
       return myResultFlag;
     }
 
+    void testParseWmstUriWithoutTemporalExtent()
+    {
+      // test fix for https://github.com/qgis/QGIS/issues/43158
+      // we just check we don't crash
+      QgsWmsProvider provider( QStringLiteral( "allowTemporalUpdates=true&temporalSource=provider&type=wmst&layers=foostyles=bar&crs=EPSG:3857&format=image/png&url=file:///dummy" ), QgsDataProvider::ProviderOptions(), mCapabilities );
+    }
+
   private:
     QgsWmsCapabilities *mCapabilities = nullptr;