diff options
| author | Kevin Lubick <kjlubick@google.com> | 2018-10-23 09:40:32 -0400 |
|---|---|---|
| committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-10-23 14:29:52 +0000 |
| commit | 1f0170ce7a908a1112b60d5f2bff2132624000e1 (patch) | |
| tree | 5eceb183a5d751e12a1667ad777da1e32d4f9b42 /fuzz | |
| parent | b70fd9168111a9cc090a766ec68a8e7595cfb738 (diff) | |
| download | skqp-1f0170ce7a908a1112b60d5f2bff2132624000e1.tar.gz | |
Have FuzzPathop make a new path every time
This will hopefully be more interesting logic.
Bug: skia:
Change-Id: I8aa6ab3d66ece4a6c1042701e1aae06d96247f32
Reviewed-on: https://skia-review.googlesource.com/c/164600
Reviewed-by: Cary Clark <caryclark@google.com>
Commit-Queue: Kevin Lubick <kjlubick@google.com>
Diffstat (limited to 'fuzz')
| -rw-r--r-- | fuzz/FuzzPathop.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/fuzz/FuzzPathop.cpp b/fuzz/FuzzPathop.cpp index 7aa229ee8c..b3c6369ba1 100644 --- a/fuzz/FuzzPathop.cpp +++ b/fuzz/FuzzPathop.cpp @@ -19,16 +19,16 @@ DEF_FUZZ(Pathop, fuzz) { fuzz->nextRange(&choice, 0, 4); switch (choice) { case 0: { - SkPath path; - FuzzEvilPath(fuzz, &path, SkPath::Verb::kDone_Verb); - SkPath::FillType ft; - fuzz->nextEnum(&ft, 0, SkPath::kInverseEvenOdd_FillType); - path.setFillType(ft); - uint8_t ops; fuzz->nextRange(&ops, 0, MAX_OPS); SkOpBuilder builder; - for (uint8_t i = 0; i < ops; i++) { + for (uint8_t i = 0; i < ops && !fuzz->exhausted(); i++) { + SkPath path; + FuzzEvilPath(fuzz, &path, SkPath::Verb::kDone_Verb); + SkPath::FillType ft; + fuzz->nextEnum(&ft, 0, SkPath::kInverseEvenOdd_FillType); + path.setFillType(ft); + SkPathOp op; fuzz->nextEnum(&op, 0, SkPathOp::kReverseDifference_SkPathOp); builder.add(path, op); |