diff options
| author |  Cary Clark <caryclark@skia.org> | 2018-03-12 14:41:45 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-03-13 12:41:42 +0000 |
| commit | 2f06a53b257571b566cbce792032bd0be208ae5d (patch) | |
| tree | 86e457955d775ded32900968caeb862fa9ea1362 /src/pathops/SkOpCoincidence.cpp | |
| parent | 4ce53a6ffd3caa1da68d9d9c42d848cbb06f69fb (diff) | |
fix a couple pathops fuzzers
This fixes these:
src/pathops/SkOpCoincidence.cpp -- 1 crash-causing fuzzes
src/pathops/SkOpSpan.h -- 1 crash-causing fuzzes
R=kjlubick@google.com
Bug: skia:
Change-Id: I2b0dbd74757a32e522901753e2e17114a1a9f79b
Reviewed-on: https://skia-review.googlesource.com/113751
Commit-Queue: Cary Clark <caryclark@skia.org>
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Diffstat (limited to 'src/pathops/SkOpCoincidence.cpp')
| -rw-r--r-- | src/pathops/SkOpCoincidence.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/pathops/SkOpCoincidence.cpp b/src/pathops/SkOpCoincidence.cpp index 8e5c6c1d4d..8fa42a5c4e 100644 --- a/src/pathops/SkOpCoincidence.cpp +++ b/src/pathops/SkOpCoincidence.cpp @@ -692,7 +692,7 @@ bool SkOpCoincidence::addOrOverlap(SkOpSegment* coinSeg, SkOpSegment* oppSeg, return true; } SkASSERT(!cs || !cs->deleted()); - SkASSERT(!os || !os->deleted()); + FAIL_IF(os && os->deleted()); SkASSERT(!ce || !ce->deleted()); FAIL_IF(oe && oe->deleted()); const SkOpPtT* csExisting = !cs ? coinSeg->existing(coinTs, nullptr) : nullptr; @@ -1095,6 +1095,7 @@ bool SkOpCoincidence::apply(DEBUG_COIN_DECLARE_ONLY_PARAMS()) { SkDebugf("seg=%d span=%d windValue=%d oppValue=%d\n", oSegment->debugID(), oStart->debugID(), oWindValue, oOppValue); #endif + FAIL_IF(windValue == -1); start->setWindValue(windValue); start->setOppValue(oppValue); FAIL_IF(oWindValue == -1); |