Address ASAN fuzzer complaint

src/core/SkPicturePlayback.cpp

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkCanvas.h"
 #include "SkPatchUtils.h"
 #include "SkPictureData.h"
@@ skipping 249 matching lines @@
         case DRAW_DRAWABLE:
             canvas->drawDrawable(fPictureData->getDrawable(reader));
             break;
         case DRAW_DRAWABLE_MATRIX: {
             SkMatrix matrix;
             reader->readMatrix(&matrix);
             SkDrawable* drawable = fPictureData->getDrawable(reader);
             canvas->drawDrawable(drawable, &matrix);
         } break;
         case DRAW_DRRECT: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             SkRRect outer, inner;
             reader->readRRect(&outer);
             reader->readRRect(&inner);
-            canvas->drawDRRect(outer, inner, paint);
+            if (paint) {
+                canvas->drawDRRect(outer, inner, *paint);
+            }
         } break;
         case BEGIN_COMMENT_GROUP: {
             SkString tmp;
             reader->readString(&tmp);
             // deprecated (M44)
             break;
         }
         case COMMENT: {
             SkString tmp;
             reader->readString(&tmp);
@@ skipping 30 matching lines @@
             reader->readRect(&dst);     // required
             // DRAW_IMAGE_RECT_STRICT assumes this constraint, and doesn't store it
             SkCanvas::SrcRectConstraint constraint = SkCanvas::kStrict_SrcRectConstraint;
             if (DRAW_IMAGE_RECT == op) {
                 // newer op-code stores the constraint explicitly
                 constraint = (SkCanvas::SrcRectConstraint)reader->readInt();
             }
             canvas->legacy_drawImageRect(image, src, dst, paint, constraint);
         } break;
         case DRAW_OVAL: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             SkRect rect;
             reader->readRect(&rect);
-            canvas->drawOval(rect, paint);
+            if (paint) {
+                canvas->drawOval(rect, *paint);
+            }
         } break;
-        case DRAW_PAINT:
-            canvas->drawPaint(*fPictureData->getPaint(reader));
-            break;
+        case DRAW_PAINT: {
+            const SkPaint* paint = fPictureData->getPaint(reader);
+            if (paint) {
+                canvas->drawPaint(*paint);
+            }
+        } break;
         case DRAW_PATCH: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
 
             const SkPoint* cubics = (const SkPoint*)reader->skip(SkPatchUtils::kNumCtrlPts *
                                                                  sizeof(SkPoint));
             uint32_t flag = reader->readInt();
             const SkColor* colors = nullptr;
             if (flag & DRAW_VERTICES_HAS_COLORS) {
                 colors = (const SkColor*)reader->skip(SkPatchUtils::kNumCorners * sizeof(SkColor));
             }
             const SkPoint* texCoords = nullptr;
             if (flag & DRAW_VERTICES_HAS_TEXS) {
                 texCoords = (const SkPoint*)reader->skip(SkPatchUtils::kNumCorners *
                                                          sizeof(SkPoint));
             }
             sk_sp<SkXfermode> xfer;
             if (flag & DRAW_VERTICES_HAS_XFER) {
                 int mode = reader->readInt();
                 if (mode < 0 || mode > SkXfermode::kLastMode) {
                     mode = SkXfermode::kModulate_Mode;
                 }
                 xfer = SkXfermode::Make((SkXfermode::Mode)mode);
             }
-            canvas->drawPatch(cubics, colors, texCoords, std::move(xfer), paint);
+            if (paint) {
+                canvas->drawPatch(cubics, colors, texCoords, std::move(xfer), *paint);
+            }
         } break;
         case DRAW_PATH: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
-            canvas->drawPath(fPictureData->getPath(reader), paint);
+            const SkPaint* paint = fPictureData->getPaint(reader);
+            if (paint) {
+                canvas->drawPath(fPictureData->getPath(reader), *paint);
+            }
         } break;
         case DRAW_PICTURE:
             canvas->drawPicture(fPictureData->getPicture(reader));
             break;
         case DRAW_PICTURE_MATRIX_PAINT: {
             const SkPaint* paint = fPictureData->getPaint(reader);
             SkMatrix matrix;
             reader->readMatrix(&matrix);
             const SkPicture* pic = fPictureData->getPicture(reader);
             canvas->drawPicture(pic, &matrix, paint);
         } break;
         case DRAW_POINTS: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             SkCanvas::PointMode mode = (SkCanvas::PointMode)reader->readInt();
             size_t count = reader->readInt();
             const SkPoint* pts = (const SkPoint*)reader->skip(sizeof(SkPoint)* count);
-            canvas->drawPoints(mode, count, pts, paint);
+            if (paint) {
+                canvas->drawPoints(mode, count, pts, *paint);
+            }
         } break;
         case DRAW_POS_TEXT: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             TextContainer text;
             get_text(reader, &text);
             size_t points = reader->readInt();
             const SkPoint* pos = (const SkPoint*)reader->skip(points * sizeof(SkPoint));
-            canvas->drawPosText(text.text(), text.length(), pos, paint);
+            if (paint) {
+                canvas->drawPosText(text.text(), text.length(), pos, *paint);
+            }
         } break;
         case DRAW_POS_TEXT_TOP_BOTTOM: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             TextContainer text;
             get_text(reader, &text);
             size_t points = reader->readInt();
             const SkPoint* pos = (const SkPoint*)reader->skip(points * sizeof(SkPoint));
             const SkScalar top = reader->readScalar();
             const SkScalar bottom = reader->readScalar();
-            if (!canvas->quickRejectY(top, bottom)) {
-                canvas->drawPosText(text.text(), text.length(), pos, paint);
+            if (!canvas->quickRejectY(top, bottom) && paint) {
+                canvas->drawPosText(text.text(), text.length(), pos, *paint);
             }
         } break;
         case DRAW_POS_TEXT_H: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             TextContainer text;
             get_text(reader, &text);
             size_t xCount = reader->readInt();
             const SkScalar constY = reader->readScalar();
             const SkScalar* xpos = (const SkScalar*)reader->skip(xCount * sizeof(SkScalar));
-            canvas->drawPosTextH(text.text(), text.length(), xpos, constY, paint);
+            if (paint) {
+                canvas->drawPosTextH(text.text(), text.length(), xpos, constY, *paint);
+            }
         } break;
         case DRAW_POS_TEXT_H_TOP_BOTTOM: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             TextContainer text;
             get_text(reader, &text);
             size_t xCount = reader->readInt();
             const SkScalar* xpos = (const SkScalar*)reader->skip((3 + xCount) * sizeof(SkScalar));
             const SkScalar top = *xpos++;
             const SkScalar bottom = *xpos++;
             const SkScalar constY = *xpos++;
-            if (!canvas->quickRejectY(top, bottom)) {
-                canvas->drawPosTextH(text.text(), text.length(), xpos, constY, paint);
+            if (!canvas->quickRejectY(top, bottom) && paint) {
+                canvas->drawPosTextH(text.text(), text.length(), xpos, constY, *paint);
             }
         } break;
         case DRAW_RECT: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             SkRect rect;
             reader->readRect(&rect);
-            canvas->drawRect(rect, paint);
+            if (paint) {
+                canvas->drawRect(rect, *paint);
+            }
         } break;
         case DRAW_RRECT: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             SkRRect rrect;
             reader->readRRect(&rrect);
-            canvas->drawRRect(rrect, paint);
+            if (paint) {
+                canvas->drawRRect(rrect, *paint);
+            }
         } break;
         case DRAW_SPRITE: {
             /* const SkPaint* paint = */ fPictureData->getPaint(reader);
             /* const SkBitmap bitmap = */ shallow_copy(fPictureData->getBitmap(reader));
             /* int left = */ reader->readInt();
             /* int top = */ reader->readInt();
             // drawSprite removed dec-2015
         } break;
         case DRAW_TEXT: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             TextContainer text;
             get_text(reader, &text);
             SkScalar x = reader->readScalar();
             SkScalar y = reader->readScalar();
-            canvas->drawText(text.text(), text.length(), x, y, paint);
+            if (paint) {
+                canvas->drawText(text.text(), text.length(), x, y, *paint);
+            }
         } break;
         case DRAW_TEXT_BLOB: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             const SkTextBlob* blob = fPictureData->getTextBlob(reader);
             SkScalar x = reader->readScalar();
             SkScalar y = reader->readScalar();
-            canvas->drawTextBlob(blob, x, y, paint);
+            if (paint) {
+                canvas->drawTextBlob(blob, x, y, *paint);
+            }
         } break;
         case DRAW_TEXT_TOP_BOTTOM: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             TextContainer text;
             get_text(reader, &text);
             const SkScalar* ptr = (const SkScalar*)reader->skip(4 * sizeof(SkScalar));
             // ptr[0] == x
             // ptr[1] == y
             // ptr[2] == top
             // ptr[3] == bottom
-            if (!canvas->quickRejectY(ptr[2], ptr[3])) {
-                canvas->drawText(text.text(), text.length(), ptr[0], ptr[1], paint);
+            if (!canvas->quickRejectY(ptr[2], ptr[3]) && paint) {
+                canvas->drawText(text.text(), text.length(), ptr[0], ptr[1], *paint);
             }
         } break;
         case DRAW_TEXT_ON_PATH: {
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             TextContainer text;
             get_text(reader, &text);
             const SkPath& path = fPictureData->getPath(reader);
             SkMatrix matrix;
             reader->readMatrix(&matrix);
-            canvas->drawTextOnPath(text.text(), text.length(), path, &matrix, paint);
+            if (paint) {
+                canvas->drawTextOnPath(text.text(), text.length(), path, &matrix, *paint);
+            }
         } break;
         case DRAW_VERTICES: {
             sk_sp<SkXfermode> xfer;
-            const SkPaint& paint = *fPictureData->getPaint(reader);
+            const SkPaint* paint = fPictureData->getPaint(reader);
             DrawVertexFlags flags = (DrawVertexFlags)reader->readInt();
             SkCanvas::VertexMode vmode = (SkCanvas::VertexMode)reader->readInt();
             int vCount = reader->readInt();
             const SkPoint* verts = (const SkPoint*)reader->skip(vCount * sizeof(SkPoint));
             const SkPoint* texs = nullptr;
             const SkColor* colors = nullptr;
             const uint16_t* indices = nullptr;
             int iCount = 0;
             if (flags & DRAW_VERTICES_HAS_TEXS) {
                 texs = (const SkPoint*)reader->skip(vCount * sizeof(SkPoint));
             }
             if (flags & DRAW_VERTICES_HAS_COLORS) {
                 colors = (const SkColor*)reader->skip(vCount * sizeof(SkColor));
             }
             if (flags & DRAW_VERTICES_HAS_INDICES) {
                 iCount = reader->readInt();
                 indices = (const uint16_t*)reader->skip(iCount * sizeof(uint16_t));
             }
             if (flags & DRAW_VERTICES_HAS_XFER) {
                 int mode = reader->readInt();
                 if (mode < 0 || mode > SkXfermode::kLastMode) {
                     mode = SkXfermode::kModulate_Mode;
                 }
                 xfer = SkXfermode::Make((SkXfermode::Mode)mode);
             }
-            canvas->drawVertices(vmode, vCount, verts, texs, colors, xfer, indices, iCount, paint);
+            if (paint) {
+                canvas->drawVertices(vmode, vCount, verts, texs, colors,
+                                     xfer, indices, iCount, *paint);
+            }
         } break;
         case RESTORE:
             canvas->restore();
             break;
         case ROTATE:
             canvas->rotate(reader->readScalar());
             break;
         case SAVE:
             // SKPs with version < 29 also store a SaveFlags param.
             if (size > 4) {
@@ skipping 53 matching lines @@
         } break;
         case TRANSLATE: {
             SkScalar dx = reader->readScalar();
             SkScalar dy = reader->readScalar();
             canvas->translate(dx, dy);
         } break;
         default:
             SkASSERTF(false, "Unknown draw type: %d", op);
     }
 }