Opened 10 years ago

Closed 10 years ago

#5715 closed defect (fixed)

Incorrect geometry cast when reading GML topogeometries

Reported by: Even Rouault Owned by: Even Rouault
Priority: normal Milestone: 1.11.2
Component: OGR_SF Version: unspecified
Severity: normal Keywords:
Cc:

Description

Around line 2064 of gml2ogrgeometries.cpp, there's a cast poTS = (OGRMultiPolygon *)poUnion; done on poUnion that is the result of a Union. The result of this union may be a OGRPolygon* ... And there are calls to poTS->getNumGeometries(). By chance, they currently return non 0 value, which avoid poTS->addGeometryDirectly() at line 2050 to be called. But this is really unsafe ! And changing the geometry class hierarchy for SQL/MM triggers this bug that makes crash ogr_gml_read_35

Change History (1)

comment:1 by Even Rouault, 10 years ago

Milestone: → 1.11.2
Resolution: → fixed
Status: new → closed

trunk r27907, branches/1.11 r27908 "Fix incorrect geometry cast when reading GML topogeometries (#5715)"

Note: See TracTickets for help on using tickets.