commit 7f2b6fa6657513e470fa73bebdf4f0230ce8b19f
author Cary Clark <caryclark@skia.org> Wed Nov 14 08:30:58 2018 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Wed Nov 14 14:00:05 2018 +0000
tree 3c7bbac5f7fa83c882005e1569352da08b67bd94
parent b45b53f01cf3d883eeb0e01205ec078cbe9ace9a

fix pathops fuzzes

Turn asserts into early exits.

R=kjlubick@google.com

Bug: skia:
Change-Id: I5da57e510774038f04381571aa2cebfbd26f03a1
Reviewed-on: https://skia-review.googlesource.com/c/171000
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/SkPathOpsSimplify.cpp

diff --git a/src/pathops/SkPathOpsSimplify.cpp b/src/pathops/SkPathOpsSimplify.cpp
index a84612e..0c7a2e4 100644
--- a/src/pathops/SkPathOpsSimplify.cpp
+++ b/src/pathops/SkPathOpsSimplify.cpp
@@ -50,7 +50,9 @@
                 if (current->activeWinding(start, end) && !writer->isClosed()) {
                     SkOpSpan* spanStart = start->starter(end);
                     if (!spanStart->done()) {
-                        SkAssertResult(current->addCurveTo(start, end, writer));
+                        if (!current->addCurveTo(start, end, writer)) {
+                            return false;
+                        }
                         current->markDone(spanStart);
                     }
                 }