Merge pull request #53 from mapbox/duplicate-point-crash

Check iterator for validity before dereferencing
mapbox · Dec 14, 2016 · eec53a6 · eec53a6
2 parents 4ee6478 + 617573b
commit eec53a6
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/include/mapbox/geometry/wagyu/build_edges.hpp b/include/mapbox/geometry/wagyu/build_edges.hpp
@@ -79,10 +79,11 @@ bool build_edge_list(mapbox::geometry::linear_ring<T> const& path_geometry, edge
     // Find next non repeated point going backwards from
     // end for pt1
     while (pt1 == pt2) {
-        pt1 = *(++itr_rev);
+        ++itr_rev;
         if (itr_rev == path_geometry.rend()) {
             return false;
         }
+        pt1 = *itr_rev;
     }
     ++itr;
     mapbox::geometry::point<value_type> pt3 = *itr;