commit 1eece783181701bf07431180eca13c689fd13aee
author Cary Clark <caryclark@google.com> Fri Apr 20 10:14:41 2018 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Apr 20 14:52:05 2018 +0000
tree b22619b003019a84ec77cf428a6180a36675140e
parent 8f88d89796d0ab7fefb302b21e03cc186dfc8bc3

guard against NaN in path is rect

Recent rewrite of SkPath::isRect() did not check against
sequences with NaN and infinity.

R=kjlubick@google.com

Bug: oss-fuzz:7882
Change-Id: I9315f68e4c53c37fda819adc6d57d6b3e3acc1c6
Reviewed-on: https://skia-review.googlesource.com/122783
Commit-Queue: Cary Clark <caryclark@google.com>
Commit-Queue: Kevin Lubick <kjlubick@google.com>
Auto-Submit: Cary Clark <caryclark@google.com>
Reviewed-by: Kevin Lubick <kjlubick@google.com>

src/core/SkPath.cpp

diff --git a/src/core/SkPath.cpp b/src/core/SkPath.cpp
index 0683c17..54d7a69 100644
--- a/src/core/SkPath.cpp
+++ b/src/core/SkPath.cpp
@@ -480,6 +480,9 @@
                 if (lineDelta.fX && lineDelta.fY) {
                     return false; // diagonal
                 }
+                if (!lineDelta.isFinite()) {
+                    return false; // path contains infinity or NaN
+                }
                 if (lineStart == lineEnd) {
                     break; // single point on side OK
                 }