Opened 9 years ago
Closed 9 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 , 9 years ago
Milestone: | → 1.11.2 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
trunk r27907, branches/1.11 r27908 "Fix incorrect geometry cast when reading GML topogeometries (#5715)"