diff options
| author | Android Build Merger (Role) <noreply-android-build-merger@google.com> | 2018-02-09 23:16:04 +0000 |
|---|---|---|
| committer | Android Build Merger (Role) <noreply-android-build-merger@google.com> | 2018-02-09 23:16:04 +0000 |
| commit | cdb13addd31cc3328b9477e0424d11271b4dcf5c (patch) | |
| tree | 7df7caed5057fbeab822683980940132a144f586 | |
| parent | c3621ebb0985ed3d6298d9aefd2ddf323f4f5bb0 (diff) | |
| parent | 1e74900d02f2795bb51d6019557b4475b73c2232 (diff) | |
| download | ex-cdb13addd31cc3328b9477e0424d11271b4dcf5c.tar.gz | |
[automerger] Add bounds checking for transparency lookup am: 53f97a2dd6 am: f171582d41 am: 7c096751ec am: 964b4bc316 am: 1e74900d02
Change-Id: I2f8ad51f9fac12230280a80f770b8d5e213714d6
| -rw-r--r-- | framesequence/jni/FrameSequence_gif.cpp | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/framesequence/jni/FrameSequence_gif.cpp b/framesequence/jni/FrameSequence_gif.cpp index 3ba09dca..ed478342 100644 --- a/framesequence/jni/FrameSequence_gif.cpp +++ b/framesequence/jni/FrameSequence_gif.cpp @@ -111,12 +111,14 @@ FrameSequence_gif::FrameSequence_gif(Stream* stream) : } #endif - if (mGif->SColorMap) { + const ColorMapObject* cmap = mGif->SColorMap; + if (cmap) { // calculate bg color GraphicsControlBlock gcb; DGifSavedExtensionToGCB(mGif, 0, &gcb); - if (gcb.TransparentColor == NO_TRANSPARENT_COLOR) { - mBgColor = gifColorToColor8888(mGif->SColorMap->Colors[mGif->SBackGroundColor]); + if (gcb.TransparentColor == NO_TRANSPARENT_COLOR + && mGif->SBackGroundColor < cmap->ColorCount) { + mBgColor = gifColorToColor8888(cmap->Colors[mGif->SBackGroundColor]); } } } @@ -358,4 +360,3 @@ static RegistryEntry gEntry = { acceptsBuffers, }; static Registry gRegister(gEntry); - |