commit 2631558d58a86b7904d05d368729d814a4ffe9d7
author Ivan Afanasyev <ivafanas@yandex-team.ru> Mon Oct 09 10:09:53 2017 +0700
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Mon Oct 09 19:17:05 2017 +0000
tree 80e3e67b366144694f06bd4f179e0d22e43e2882
parent 7cf2fd6ff62378c5060d06a9699f3ff4510bdf53

fix use of uninitialized flag in SkCodec

SkCodec class has the fStartedIncrementalDecode field, which is not
initialized on construction, but may be used in incrementalDecode
method being uninitialized.

Bug: skia:
Change-Id: I378412a375100ad2df4aa38f2dd5682867e0e34b
Reviewed-on: https://skia-review.googlesource.com/53160
Commit-Queue: Heather Miller <hcm@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
Reviewed-by: Mike Klein <mtklein@chromium.org>

src/codec/SkCodec.cpp

diff --git a/src/codec/SkCodec.cpp b/src/codec/SkCodec.cpp
index 9b55790..c3d99f2 100644
--- a/src/codec/SkCodec.cpp
+++ b/src/codec/SkCodec.cpp
@@ -138,6 +138,7 @@
     , fDstInfo()
     , fOptions()
     , fCurrScanline(-1)
+    , fStartedIncrementalDecode(false)
 {}
 
 SkCodec::SkCodec(const SkEncodedInfo& info, const SkImageInfo& imageInfo,
@@ -151,6 +152,7 @@
     , fDstInfo()
     , fOptions()
     , fCurrScanline(-1)
+    , fStartedIncrementalDecode(false)
 {}
 
 SkCodec::~SkCodec() {}