diff options
author | Mike Reed <reed@google.com> | 2018-03-08 14:20:52 -0500 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-03-08 19:51:41 +0000 |
commit | 1eb5ca41d9db5ca72f44f957fdd483d3f260f051 (patch) | |
tree | a45afed3d5fa16356649f10487ee8fbfb36676a8 /src/shaders/SkImageShader.cpp | |
parent | 8b0ef635fdec43d14b2be58ab95afaea5323c6dd (diff) |
tighten check when we might overflow legacy shader math
Bug: skia:7700
Change-Id: I2ca06b7ba36f2e17d6addd31b1f67a07d478e207
Reviewed-on: https://skia-review.googlesource.com/113264
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Reed <reed@google.com>
Diffstat (limited to 'src/shaders/SkImageShader.cpp')
-rw-r--r-- | src/shaders/SkImageShader.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/shaders/SkImageShader.cpp b/src/shaders/SkImageShader.cpp index d42b417704..a76edc053c 100644 --- a/src/shaders/SkImageShader.cpp +++ b/src/shaders/SkImageShader.cpp @@ -86,8 +86,8 @@ static bool legacy_shader_can_handle(const SkMatrix& a, const SkMatrix& b) { SkRect src; SkAssertResult(inv.mapRect(&src, SkRect::MakeWH(max_dev_coord, max_dev_coord))); - // take 1/2 of max signed 32bits so we have room to subtract coordinates - const SkScalar max_fixed32dot32 = SK_MaxS32 * 0.5f; + // take 1/4 of max signed 32bits so we have room to subtract local values + const SkScalar max_fixed32dot32 = SK_MaxS32 * 0.25f; if (!SkRect::MakeLTRB(-max_fixed32dot32, -max_fixed32dot32, max_fixed32dot32, max_fixed32dot32).contains(src)) { return false; |