memory leak in gmlWriteGeometry_GML2

[ross.elliott@…]

in gmlWriteGeometry_GML2 (mapgml.c) outerlist is allocated twice (see extract below)

-----------------------------------------------

case(MS_SHAPE_POLYGON): /* this gets nasty, since our shapes are so flexible */
    geometry_simple_index = msGMLGeometryLookup(geometryList, "polygon");
    geometry_aggregate_index = msGMLGeometryLookup(geometryList, "multipolygon");
    if(geometry_simple_index >= 0) geometry_simple_name =
geometryList->geometries[geometry_simple_index].name;
    if(geometry_aggregate_index >= 0) geometry_aggregate_name =
geometryList->geometries[geometry_aggregate_index].name;

    /* get a list of outter rings for this polygon */
    outerlist = msGetOuterList(shape);

    /* get a list of outter rings for this polygon */
    outerlist = msGetOuterList(shape);

[tomkralidis]

I can't find this in latest svn mapgml.c anymore. Can you confirm?

[relliott]

I've just checked the builds I have, it was there in 4.8.1 but not in 4.8.4 or later, so problem fixed.