Do not dereference null pointer on QgsSelectedFeature::updateGeometry

Sandro Santilli · Sandro Santilli · commit 781c9d7fc2c1 · 2016-02-01T17:30:14.000+01:00
Fixes #14188
diff --git a/src/app/nodetool/qgsselectedfeature.cpp b/src/app/nodetool/qgsselectedfeature.cpp
@@ -78,7 +78,7 @@ void QgsSelectedFeature::updateGeometry( QgsGeometry *geom )
   {
     QgsFeature f;
     mVlayer->getFeatures( QgsFeatureRequest().setFilterFid( mFeatureId ) ).nextFeature( f );
-    mGeometry = new QgsGeometry( *f.geometry() );
+    mGeometry = f.geometry() ? new QgsGeometry( *f.geometry() ) : nullptr;
   }
   else
   {

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ void QgsSelectedFeature::updateGeometry( QgsGeometry *geom )`
`78`	`78`	`{`
`79`	`79`	`QgsFeature f;`
`80`	`80`	`mVlayer->getFeatures( QgsFeatureRequest().setFilterFid( mFeatureId ) ).nextFeature( f );`
`81`		`- mGeometry = new QgsGeometry( *f.geometry() );`
	`81`	`+ mGeometry = f.geometry() ? new QgsGeometry( *f.geometry() ) : nullptr;`
`82`	`82`	`}`
`83`	`83`	`else`
`84`	`84`	`{`