Opened 3 years ago
Closed 3 years ago
#5028 closed defect (fixed)
ST_AsFlatGeobuf crashes on mixed geometry input
| Reported by: | Björn Harrtell | Owned by: | Björn Harrtell |
|---|---|---|---|
| Priority: | medium | Milestone: | PostGIS 3.2.0 |
| Component: | postgis | Version: | master |
| Keywords: | Cc: |
Description
Change History (4)
comment:1 by , 3 years ago
| Version: | 2.5.x → master |
|---|
comment:2 by , 3 years ago
| Milestone: | PostGIS 3.1.5 → PostGIS 3.2.0 |
|---|
comment:3 by , 3 years ago
comment:4 by , 3 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Fix and regression test added with https://git.osgeo.org/gitea/postgis/postgis/commit/d71751919c84ae83fa32268197caced462154ab2.
Note:
See TracTickets
for help on using tickets.
This is because I use the first geometry to determine geometry type at https://github.com/postgis/postgis/blob/eda70258f373c1ad88e1851f194c891ad8d9aff4/deps/flatgeobuf/flatgeobuf_c.cpp#L56.
Adding an option to override this doesn't feel right.
At least I should be able to detect this situation and bail to avoid the crash.