[postgis-tickets] r17671 - ST_GeomFromGeoJSON crash on empty rings

[Darafei]

Author: komzpa
Date: 2019-08-04 00:42:45 -0700 (Sun, 04 Aug 2019)
New Revision: 17671

Modified:
   branches/2.3/NEWS
   branches/2.3/liblwgeom/lwin_geojson.c
Log:
ST_GeomFromGeoJSON crash on empty rings

Closes #4470


Modified: branches/2.3/NEWS
===================================================================
--- branches/2.3/NEWS	2019-08-04 07:39:32 UTC (rev 17670)
+++ branches/2.3/NEWS	2019-08-04 07:42:45 UTC (rev 17671)
@@ -11,8 +11,8 @@
   - #4466, Fix undefined behaviour in _postgis_gserialized_stats (Raúl Marín)
   - #4209, Handle NULL geometry values in pgsql2shp (Paul Ramsey)
   - #4437, Handle POINT EMPTY in shape loader/dumper (Paul Ramsey)
+  - #4470, ST_GeomFromGeoJSON crash on empty rings (Darafei Praliaskouski)
 
-
 PostGIS 2.3.9
 2019/03/11
 

Modified: branches/2.3/liblwgeom/lwin_geojson.c
===================================================================
--- branches/2.3/liblwgeom/lwin_geojson.c	2019-08-04 07:39:32 UTC (rev 17670)
+++ branches/2.3/liblwgeom/lwin_geojson.c	2019-08-04 07:42:45 UTC (rev 17671)
@@ -256,9 +256,6 @@
 		}
 		nPoints = json_object_array_length(points);
 		
-		/* Skip empty rings */
-		if ( nPoints == 0 ) continue;
-		
 		if ( ! ppa )
 			ppa = (POINTARRAY**)lwalloc(sizeof(POINTARRAY*) * nRings);