diff options
author | Mike Reed <reed@google.com> | 2018-05-11 17:19:32 +0000 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-05-11 17:19:37 +0000 |
commit | b8f0709aac43cfdefdbf3a307f8c2ecfc0c55d27 (patch) | |
tree | caef9b5ce1f8f63b192551c6122f2c1baadbc1eb /src/core/SkDraw.cpp | |
parent | 1b17adb0f8bb19445aa66754a5dce9629f7ac560 (diff) |
Revert "reject large paths to avoid potential float overflows"
This reverts commit b35002d323134e2a441ce2f912a305cd9b3bd321.
Reason for revert: crashing in threadedraster, may need to clean up the initonce payload if I early exit
Original change's description:
> 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>
TBR=mtklein@google.com,jvanverth@google.com,bsalomon@google.com,caryclark@google.com,csmartdalton@google.com,fmalita@chromium.org,liyuqian@google.com,reed@google.com
Change-Id: I12ff8664113ba5a36ee91cbe2000d7d445d4bc7e
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: oss-fuzz:8131
Reviewed-on: https://skia-review.googlesource.com/127580
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, 0 insertions, 3 deletions
diff --git a/src/core/SkDraw.cpp b/src/core/SkDraw.cpp index c465b8fd8e..34f5da5cfe 100644 --- a/src/core/SkDraw.cpp +++ b/src/core/SkDraw.cpp @@ -951,9 +951,6 @@ 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) { |