diff options
| author |  Cary Clark <caryclark@google.com> | 2016-12-12 13:57:56 -0500 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-12-12 19:39:11 +0000 |
| commit | 26ae5aba592d4169c1c53aa7fafa660589a4d554 (patch) | |
| tree | edffbfce799ff88c057aeab6d0e3bb7880fd8cc8 /src/pathops | |
| parent | ee43f6f25e4ebd0a91ee208484f576dc9991a8e8 (diff) | |
fix fuzz
Don't walk off the end if the loop doesn't
contain the expected value.
R=kjlubick@google.com
BUG=skia:6047
Change-Id: I96815180dc7c92b45691037ae6c4b40beedc009a
Reviewed-on: https://skia-review.googlesource.com/5845
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Commit-Queue: Cary Clark <caryclark@google.com>
Diffstat (limited to 'src/pathops')
| -rw-r--r-- | src/pathops/SkOpCoincidence.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/pathops/SkOpCoincidence.cpp b/src/pathops/SkOpCoincidence.cpp index 1ade4446a3..557da0a56e 100644 --- a/src/pathops/SkOpCoincidence.cpp +++ b/src/pathops/SkOpCoincidence.cpp @@ -1004,6 +1004,7 @@ bool SkOpCoincidence::apply(DEBUG_COIN_DECLARE_ONLY_PARAMS()) { if (oNext == oEnd) { break; } + FAIL_IF(!oNext->upCastable()); oStart = oNext->upCast(); } while (true); } |