[geos-devel] [GEOS] #830: Possible memory leak in WKTReader with partial MULTIPOLYGON

[GEOS]

#830: Possible memory leak in WKTReader with partial MULTIPOLYGON
-------------------------+---------------------------
 Reporter:  goatbar      |       Owner:  geos-devel@…
     Type:  defect       |      Status:  new
 Priority:  minor        |   Milestone:
Component:  Default      |     Version:  3.5.0
 Severity:  Unassigned   |  Resolution:
 Keywords:  wkt fuzzing  |
-------------------------+---------------------------

Comment (by goatbar):

 Working on a proposed simple patch of this form.  I'll have it ready in
 the next day.
 {{{#!c++
         Polygon *polygon = NULL;
         try {
                 polygon=readPolygonText(tokenizer);
         } catch (ParseException const& e) {
                 delete polygons;
                 throw ParseException(e.what());
         }
 }}}
 This happens a number of locations, e.g. multilinestring too.

 mloskot: Agreed, the whole move semantics world will make this a lot
 simple in the long run.

--
Ticket URL: <https://trac.osgeo.org/geos/ticket/830#comment:4>
GEOS <http://trac.osgeo.org/geos>
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite (JTS).