commit 18c00976bd62938fa13bc9d490ac8fd39b769f6d
author reed <reed@chromium.org> Sat Apr 02 18:40:40 2016 -0700
committer Commit bot <commit-bot@chromium.org> Sat Apr 02 18:40:40 2016 -0700
tree d9bc6f55159bacdb130f89ce49dec9579c54ba62
parent 6db0a7bdceb6be85721bfb0db8dea7fd27db5970

check for bad params sent to discrete patheffect

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1853953002

TBR=

Review URL: https://codereview.chromium.org/1853953002

src/effects/SkDiscretePathEffect.cpp

diff --git a/src/effects/SkDiscretePathEffect.cpp b/src/effects/SkDiscretePathEffect.cpp
index 5af6ab4..1c6039c 100644
--- a/src/effects/SkDiscretePathEffect.cpp
+++ b/src/effects/SkDiscretePathEffect.cpp
@@ -14,6 +14,12 @@
 
 sk_sp<SkPathEffect> SkDiscretePathEffect::Make(SkScalar segLength, SkScalar deviation,
                                                uint32_t seedAssist) {
+    if (!SkScalarIsFinite(segLength) || !SkScalarIsFinite(deviation)) {
+        return nullptr;
+    }
+    if (segLength <= SK_ScalarNearlyZero) {
+        return nullptr;
+    }
     return sk_sp<SkPathEffect>(new SkDiscretePathEffect(segLength, deviation, seedAssist));
 }