commit cac53857799091a1922d0e5aa39e97c1bf53800f
author Harish Mahendrakar <harish.mahendrakar@ittiam.com> Wed Feb 20 10:59:10 2019 -0800
committer Wonsik Kim <wonsik@google.com> Thu Feb 21 11:38:40 2019 -0800
tree 1b19450a851422225d471482b3fda2c39dd753cc
parent 05677f7f44a753b756709ddbd7eebdffa34f62f4

Codec2Buffer: Initialize mediaImage to zero in GraphicView2MediaImageConverter

MediaImage2 structure was initialized only for valid planes, but entire
structure was compared to signal change in output format.
This was resulting in multiple change in output formats signalled in
dequeueOutputBuffer()

Bug: 124799741
Test: Manually verified that only one ouput format changed is signalled\
 per clip using AMediaCodec based decoder application

Change-Id: Ib4139fc80f3493e3cb2b019d3890c50b05076469

media/codec2/sfplugin/Codec2Buffer.cpp

diff --git a/media/codec2/sfplugin/Codec2Buffer.cpp b/media/codec2/sfplugin/Codec2Buffer.cpp
index 13b63c9..0fd5731 100644
--- a/media/codec2/sfplugin/Codec2Buffer.cpp
+++ b/media/codec2/sfplugin/Codec2Buffer.cpp
@@ -224,6 +224,7 @@
             mInitCheck = BAD_VALUE;
             return;
         }
+        memset(mediaImage, 0, sizeof(*mediaImage));
         mAllocatedDepth = layout.planes[0].allocatedDepth;
         uint32_t bitDepth = layout.planes[0].bitDepth;