commit 18562b8adb9108e6bcd9ae5b89cc66deb0e49c5d
author Cary Clark <caryclark@skia.org> Thu Jun 07 08:06:12 2018 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Jun 07 12:31:08 2018 +0000
tree 8b8913bc4676e632795a2cf640d8c3e92a5d3934
parent 969bdef64139c417f5c0b8f2fbf2638bb9d7dc95

fix pathops fuzzer

turn assert into fail

R=kjlubick

Change-Id: I3a2172b6b1a9eff092bcfddcf8310f32754b93f7
Reviewed-on: https://skia-review.googlesource.com/132780
Commit-Queue: Cary Clark <caryclark@skia.org>
Commit-Queue: Kevin Lubick <kjlubick@google.com>
Auto-Submit: Cary Clark <caryclark@skia.org>
Reviewed-by: Kevin Lubick <kjlubick@google.com>

src/pathops/SkOpCoincidence.cpp

diff --git a/src/pathops/SkOpCoincidence.cpp b/src/pathops/SkOpCoincidence.cpp
index ab848c8..93d0e40 100644
--- a/src/pathops/SkOpCoincidence.cpp
+++ b/src/pathops/SkOpCoincidence.cpp
@@ -789,7 +789,7 @@
         const SkOpPtT* ocs = outer->coinPtTStart();
         FAIL_IF(ocs->deleted());
         const SkOpSegment* outerCoin = ocs->segment();
-        SkASSERT(!outerCoin->done());  // if it's done, should have already been removed from list
+        FAIL_IF(outerCoin->done());
         const SkOpPtT* oos = outer->oppPtTStart();
         if (oos->deleted()) {
             return true;