[processing] Guard QGIS from crashing due to erroneous extent / inval…

…id CRS in XYZ creation algorithms
qgis · Jan 21, 2025 · 202ff1f · 202ff1f
1 parent c98dcb2
commit 202ff1f
Showing 1 changed file with 26 additions and 4 deletions.
diff --git a/src/analysis/processing/qgsalgorithmxyztiles.cpp b/src/analysis/processing/qgsalgorithmxyztiles.cpp
@@ -149,7 +149,15 @@ bool QgsXyzTilesBaseAlgorithm::prepareAlgorithm( const QVariantMap &parameters,
   QgsRectangle extent = parameterAsExtent( parameters, QStringLiteral( "EXTENT" ), context );
   QgsCoordinateReferenceSystem extentCrs = parameterAsExtentCrs( parameters, QStringLiteral( "EXTENT" ), context );
   QgsCoordinateTransform ct( extentCrs, project->crs(), context.transformContext() );
-  mExtent = ct.transformBoundingBox( extent );
+  try
+  {
+    mExtent = ct.transformBoundingBox( extent );
+  }
+  catch ( QgsCsException & )
+  {
+    feedback->reportError( QObject::tr( "Could not transform the extent into the project CRS" ), true );
+    return false;
+  }
 
   mMinZoom = parameterAsInt( parameters, QStringLiteral( "ZOOM_MIN" ), context );
   mMaxZoom = parameterAsInt( parameters, QStringLiteral( "ZOOM_MAX" ), context );
@@ -167,8 +175,15 @@ bool QgsXyzTilesBaseAlgorithm::prepareAlgorithm( const QVariantMap &parameters,
   mMercatorCrs = QgsCoordinateReferenceSystem( "EPSG:3857" );
   mSrc2Wgs = QgsCoordinateTransform( project->crs(), mWgs84Crs, context.transformContext() );
   mWgs2Mercator = QgsCoordinateTransform( mWgs84Crs, mMercatorCrs, context.transformContext() );
-
-  mWgs84Extent = mSrc2Wgs.transformBoundingBox( mExtent );
+  try
+  {
+    mWgs84Extent = mSrc2Wgs.transformBoundingBox( mExtent );
+  }
+  catch ( QgsCsException & )
+  {
+    feedback->reportError( QObject::tr( "Could not transform the extent into WGS84" ), true );
+    return false;
+  }
 
   if ( parameters.contains( QStringLiteral( "TILE_WIDTH" ) ) )
   {
@@ -212,7 +227,14 @@ void QgsXyzTilesBaseAlgorithm::startJobs()
     MetaTile metaTile = mMetaTiles.takeFirst();
 
     QgsMapSettings settings;
-    settings.setExtent( mWgs2Mercator.transformBoundingBox( metaTile.extent() ) );
+    try
+    {
+      settings.setExtent( mWgs2Mercator.transformBoundingBox( metaTile.extent() ) );
+    }
+    catch ( QgsCsException & )
+    {
+      continue;
+    }
     settings.setOutputImageFormat( QImage::Format_ARGB32_Premultiplied );
     settings.setTransformContext( mTransformContext );
     settings.setDestinationCrs( mMercatorCrs );