commit 9c2ab4c467203e14a1a3cf1e29c9568112d13dc5
author Robert Phillips <robertphillips@google.com> Tue Feb 02 10:01:59 2021 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Feb 02 17:06:22 2021 +0000
tree bba6206b975f28fa49dced18158463e1711b8666
parent c858554f387bd94d87e7ca457a17f6e15b5bbea1

Just skip corrupt JPEG image draws when rendering DDLs

Bug: skia:11243
Change-Id: Ia83b1a43c4e25d62a6f3170b8ddc7e4b5e543b52
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/364577
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>

tools/DDLPromiseImageHelper.cpp

diff --git a/tools/DDLPromiseImageHelper.cpp b/tools/DDLPromiseImageHelper.cpp
index c4ff918..762ca2c 100644
--- a/tools/DDLPromiseImageHelper.cpp
+++ b/tools/DDLPromiseImageHelper.cpp
@@ -407,7 +407,9 @@
     SkYUVAPixmapInfo yuvaInfo;
     if (codec && codec->queryYUVAInfo(fSupportedYUVADataTypes, &yuvaInfo)) {
         auto yuvaPixmaps = SkYUVAPixmaps::Allocate(yuvaInfo);
-        SkAssertResult(codec->getYUVAPlanes(yuvaPixmaps));
+        if (!codec->getYUVAPlanes(yuvaPixmaps)) {
+            return -1;
+        }
         SkASSERT(yuvaPixmaps.isValid());
         newImageInfo.setYUVPlanes(std::move(yuvaPixmaps));
     } else {