diff options
| author |  Cary Clark <caryclark@google.com> | 2016-12-08 16:17:56 -0500 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-12-08 22:22:12 +0000 |
| commit | 59ed482af72beec6812b28d833d8bdf80ba32df7 (patch) | |
| tree | 7f993451a2ed6eada726393f3067bae1f2dcfe82 /src/pathops/SkOpCoincidence.cpp | |
| parent | 8e7432b7f98dd592e529a0c8bb038d73ebfb0478 (diff) | |
fix fuzzers
Two crashes and a hang. Abort and return false
in these cases.
TBR=kjlubick@google.com
BUG=skia:5983
Change-Id: I41ac3e56d47ee423b634b2f4886085b9caada76b
Reviewed-on: https://skia-review.googlesource.com/5716
Reviewed-by: Cary Clark <caryclark@google.com>
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Commit-Queue: Cary Clark <caryclark@google.com>
Diffstat (limited to 'src/pathops/SkOpCoincidence.cpp')
| -rw-r--r-- | src/pathops/SkOpCoincidence.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/pathops/SkOpCoincidence.cpp b/src/pathops/SkOpCoincidence.cpp index d7ef35f5c4..1ade4446a3 100644 --- a/src/pathops/SkOpCoincidence.cpp +++ b/src/pathops/SkOpCoincidence.cpp @@ -1314,7 +1314,7 @@ bool SkOpCoincidence::mark(DEBUG_COIN_DECLARE_ONLY_PARAMS()) { FAIL_IF(!coin->ordered(&ordered)); while ((next = next->upCast()->next()) != end) { FAIL_IF(!next->upCastable()); - SkAssertResult(next->upCast()->insertCoincidence(oSegment, flipped, ordered)); + FAIL_IF(!next->upCast()->insertCoincidence(oSegment, flipped, ordered)); } while ((oNext = oNext->upCast()->next()) != oEnd) { FAIL_IF(!oNext->upCastable()); |