Fix possible null-pointer dereference

manisandro · manisandro · commit 97a3709798b3 · 2017-03-13T16:47:02.000+01:00
diff --git a/src/providers/arcgisrest/qgsafsprovider.cpp b/src/providers/arcgisrest/qgsafsprovider.cpp
@@ -162,7 +162,7 @@ bool QgsAfsProvider::getFeature( const QgsFeatureId &id, QgsFeature &f, bool fet
   if ( it != mCache.end() )
   {
     f = it.value();
-    return filterRect.isNull() || f.geometry()->intersects( filterRect );
+    return filterRect.isNull() || ( f.geometry() && f.geometry()->intersects( filterRect ) );
   }
 
   // Determine attributes to fetch

Original file line number	Diff line number	Diff line change
`@@ -162,7 +162,7 @@ bool QgsAfsProvider::getFeature( const QgsFeatureId &id, QgsFeature &f, bool fet`
`162`	`162`	`if ( it != mCache.end() )`
`163`	`163`	`{`
`164`	`164`	`f = it.value();`
`165`		`- return filterRect.isNull() \|\| f.geometry()->intersects( filterRect );`
	`165`	`+ return filterRect.isNull() \|\| ( f.geometry() && f.geometry()->intersects( filterRect ) );`
`166`	`166`	`}`
`167`	`167`
`168`	`168`	`// Determine attributes to fetch`