diff options
| author |  Mike Reed <reed@google.com> | 2018-05-11 09:04:12 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-05-11 16:47:27 +0000 |
| commit | b35002d323134e2a441ce2f912a305cd9b3bd321 (patch) | |
| tree | 13bbc7db0817f27e2460c29c553e9eb93e96ff2e /src/core/SkDraw.cpp | |
| parent | 103d6f616b4081d29469b4c1386972bb5b32e0d6 (diff) | |
reject large paths to avoid potential float overflows
I think this change can catch a host of potential fuzzer issues up-front,
rather than adding finite tests in lots and lots of places down-stream.
Bug: oss-fuzz:8131
Change-Id: I421aa72c6ca3df57b40dd32b805d6c847d8e8d29
Reviewed-on: https://skia-review.googlesource.com/127388
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Reed <reed@google.com>
Diffstat (limited to 'src/core/SkDraw.cpp')
| -rw-r--r-- | src/core/SkDraw.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/core/SkDraw.cpp b/src/core/SkDraw.cpp index 34f5da5cfe..c465b8fd8e 100644 --- a/src/core/SkDraw.cpp +++ b/src/core/SkDraw.cpp @@ -951,6 +951,9 @@ SkScalar SkDraw::ComputeResScaleForStroking(const SkMatrix& matrix) { void SkDraw::drawDevPath(const SkPath& devPath, const SkPaint& paint, bool drawCoverage, SkBlitter* customBlitter, bool doFill, SkInitOnceData* iData) const { + if (SkPathPriv::TooBigForMath(devPath)) { + return; + } SkBlitter* blitter = nullptr; SkAutoBlitterChoose blitterStorage; if (nullptr == customBlitter) { |