[postgis-tickets] [PostGIS] #3463: Crash loading polygons in topology

[PostGIS]

#3463: Crash loading polygons in topology
-----------------------+---------------------------
  Reporter:  strk      |      Owner:  strk
      Type:  defect    |     Status:  new
  Priority:  blocker   |  Milestone:  PostGIS 2.2.2
 Component:  topology  |    Version:  2.2.x
Resolution:            |   Keywords:
-----------------------+---------------------------

Comment (by strk):

 The bug is in unhandled NULL return from lwt_GetFaceGeometry within
 lwt_ChangeEdgeGeom:

 {{{
  /*
   -- Update faces MBR of left and right faces
   -- TODO: think about ways to optimize this part, like see if
   --       the old edge geometry partecipated in the definition
   --       of the current MBR (for shrinking) or the new edge MBR
   --       would be larger than the old face MBR...
   --
   */
   int facestoupdate = 0;
   LWT_ISO_FACE faces[2];
   LWGEOM *nface1 = NULL;
   LWGEOM *nface2 = NULL;
   if ( oldedge->face_left != 0 )
   {
     nface1 = lwt_GetFaceGeometry(topo, oldedge->face_left);
     lwgeom_add_bbox(nface1);    /// <--- nface1 is NULL here
 }}}

--
Ticket URL: <https://trac.osgeo.org/postgis/ticket/3463#comment:2>
PostGIS <http://trac.osgeo.org/postgis/>
The PostGIS Trac is used for bug, enhancement & task tracking, a user and developer wiki, and a view into the subversion code repository of PostGIS project.