diff options
author | 2018-05-22 11:51:32 -0400 | |
---|---|---|
committer | 2018-05-22 16:52:45 +0000 | |
commit | 7b376940e4722f0da14807bd3b18823d0f7c1964 (patch) | |
tree | 56c6b1c3fcacb6f9a07818626afbef7bb73fa18f /src | |
parent | d26e446927051bc3eed3aed9e5e2db19e7122cf0 (diff) |
GrTessellator: fix crash on merging a dead edge.
When merging coincident vertices, sometimes merging an edge will cause
the next edge in the list to die. But since merge_vertices() retrieves
the next edge before merging the current one, we then try to merge its
lifeless corpse, bringing it back as a zombie. The fix is to leave
dead edges, dead.
This was revealed by ec79c39a770205910f0abd762de7e19a96c986ca, but was
likely a real bug that was being mishandled before that.
Bug: 844873
Bug: skia:7982
Change-Id: I752c45f7551df4278fa4cb4587e7ead48cd16a21
Reviewed-on: https://skia-review.googlesource.com/129524
Commit-Queue: Stephen White <senorblanco@chromium.org>
Reviewed-by: Robert Phillips <robertphillips@google.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/gpu/GrTessellator.cpp | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/src/gpu/GrTessellator.cpp b/src/gpu/GrTessellator.cpp index d3c2578da5..308d5bcc36 100644 --- a/src/gpu/GrTessellator.cpp +++ b/src/gpu/GrTessellator.cpp @@ -908,6 +908,7 @@ void insert_edge_below(Edge* edge, Vertex* v, Comparator& c) { } void remove_edge_above(Edge* edge) { + SkASSERT(edge->fTop && edge->fBottom); LOG("removing edge (%g -> %g) above vertex %g\n", edge->fTop->fID, edge->fBottom->fID, edge->fBottom->fID); list_remove<Edge, &Edge::fPrevEdgeAbove, &Edge::fNextEdgeAbove>( @@ -915,6 +916,7 @@ void remove_edge_above(Edge* edge) { } void remove_edge_below(Edge* edge) { + SkASSERT(edge->fTop && edge->fBottom); LOG("removing edge (%g -> %g) below vertex %g\n", edge->fTop->fID, edge->fBottom->fID, edge->fTop->fID); list_remove<Edge, &Edge::fPrevEdgeBelow, &Edge::fNextEdgeBelow>( @@ -1120,15 +1122,11 @@ void merge_vertices(Vertex* src, Vertex* dst, VertexList* mesh, Comparator& c, if (src->fPartner) { src->fPartner->fPartner = dst; } - for (Edge* edge = src->fFirstEdgeAbove; edge;) { - Edge* next = edge->fNextEdgeAbove; + while (Edge* edge = src->fFirstEdgeAbove) { set_bottom(edge, dst, nullptr, nullptr, c); - edge = next; } - for (Edge* edge = src->fFirstEdgeBelow; edge;) { - Edge* next = edge->fNextEdgeBelow; + while (Edge* edge = src->fFirstEdgeBelow) { set_top(edge, dst, nullptr, nullptr, c); - edge = next; } mesh->remove(src); } |