qgsquadrilateral: Fix memory leak in area and polygon calls

The toPolygon() call releases the unique_ptr, the `QgsPolygon` object needs to be deleted. This achieved by using a `std::unique_ptr` which automatically frees the polygon at the end of the function.
qgis · Apr 13, 2023 · e5c2f56 · e5c2f56
1 parent d73264b
commit e5c2f56
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/core/geometry/qgsquadrilateral.cpp b/src/core/geometry/qgsquadrilateral.cpp
@@ -435,10 +435,12 @@ QString QgsQuadrilateral::toString( int pointPrecision ) const
 
 double QgsQuadrilateral::area() const
 {
-  return toPolygon()->area();
+  std::unique_ptr<QgsPolygon> polygon( toPolygon() );
+  return polygon->area();
 }
 
 double QgsQuadrilateral::perimeter() const
 {
-  return toPolygon()->perimeter();
+  std::unique_ptr<QgsPolygon> polygon( toPolygon() );
+  return polygon->perimeter();
 }