commit 1f0170ce7a908a1112b60d5f2bff2132624000e1
author Kevin Lubick <kjlubick@google.com> Tue Oct 23 09:40:32 2018 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Oct 23 14:29:52 2018 +0000
tree 5eceb183a5d751e12a1667ad777da1e32d4f9b42
parent b70fd9168111a9cc090a766ec68a8e7595cfb738

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>

fuzz/FuzzPathop.cpp

diff --git a/fuzz/FuzzPathop.cpp b/fuzz/FuzzPathop.cpp
index 7aa229e..b3c6369 100644
--- a/fuzz/FuzzPathop.cpp
+++ b/fuzz/FuzzPathop.cpp
@@ -19,16 +19,16 @@
     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);