commit a1feabd38305ea7f5e3961f96585fbe1f1c00f91
author Brian Osman <brianosman@google.com> Thu Jun 10 10:10:06 2021 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Jun 10 15:07:40 2021 +0000
tree 95607fe98598cb76fbf53d71f69bb97d304cb356
parent 6285d6402785f7c58acfd22e5fce97ec338931c2

Ignore divide by zero in SkColorSpaceXformSteps::apply

We're relying on IEEE behavior here (and checking the result), so this
is safe.

Bug: oss-fuzz:35004
Change-Id: I82cc38f7802455036b34d9afdb525662877f3809
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/417201
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>

src/core/SkColorSpaceXformSteps.cpp

diff --git a/src/core/SkColorSpaceXformSteps.cpp b/src/core/SkColorSpaceXformSteps.cpp
index 291f39a..c091d6a 100644
--- a/src/core/SkColorSpaceXformSteps.cpp
+++ b/src/core/SkColorSpaceXformSteps.cpp
@@ -98,7 +98,8 @@
         // I don't know why isfinite(x) stopped working on the Chromecast bots...
         auto is_finite = [](float x) { return x*0 == 0; };
 
-        float invA = is_finite(1.0f / rgba[3]) ? 1.0f / rgba[3] : 0;
+        float invA = sk_ieee_float_divide(1.0f, rgba[3]);
+        invA = is_finite(invA) ? invA : 0;
         rgba[0] *= invA;
         rgba[1] *= invA;
         rgba[2] *= invA;