diff options
author | caryclark <caryclark@google.com> | 2016-09-23 09:32:26 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-09-23 09:32:27 -0700 |
commit | cc09372730301be78b9d26c1198db1584622cdd9 (patch) | |
tree | 99be1f7a5c3cfefbb75b47f03ddf5d377a5c4fd8 /tests/PathOpsOpTest.cpp | |
parent | 94663795410c214db5232162076b8aea7e5f62d2 (diff) |
fix msan bug in pathops
Msan and Valgrind found an uninitialized memory mistake in
pathops. This also fixes similar bugs where not all parts
of the geometry were covered in the loop iteration.
R=borenet@google.com
NOTREECHECKS=true
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2366893003
Review-Url: https://codereview.chromium.org/2366893003
Diffstat (limited to 'tests/PathOpsOpTest.cpp')
-rw-r--r-- | tests/PathOpsOpTest.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/tests/PathOpsOpTest.cpp b/tests/PathOpsOpTest.cpp index 9bb3a0fbb6..92ef105ce8 100644 --- a/tests/PathOpsOpTest.cpp +++ b/tests/PathOpsOpTest.cpp @@ -7990,12 +7990,10 @@ DEF_TEST(PathOpsFailOp, reporter) { } static struct TestDesc repTests[] = { - TEST(loops44i), - TEST(loops45i), - TEST(loops46i), + TEST(fuzz763_5a), }; DEF_TEST(PathOpsRepOp, reporter) { - for (int index = 0; index < 2; ++index) + for (int index = 0; index < 1; ++index) RunTestSet(reporter, repTests, SK_ARRAY_COUNT(repTests), nullptr, nullptr, nullptr, false); } |