From e971a6d8555f8e05e86836705734880f41c91d11 Mon Sep 17 00:00:00 2001 From: wfeng6 Date: Fri, 9 Aug 2013 16:08:41 +0800 Subject: Fix in the Klocwork issue in OMXVideoDecoderAVCSecure BZ: 129622 Problem: 'output.parsed_data_size' might be used uninitialized in PrepareDecodeBuffer in file videocodec/securevideo /baytrail/OMXVideoDecoderAVCSecure.cpp:354. Fix: initialize the output structure with 0's. Change-Id: I41c51ddf203ce575f38ccb753ed66128bf67e98a Signed-off-by: wfeng6 Reviewed-on: http://android.intel.com:8080/124619 Reviewed-by: Guo, Nana N Tested-by: Ding, Haitao Reviewed-by: cactus Tested-by: cactus --- videocodec/securevideo/baytrail/OMXVideoDecoderAVCSecure.cpp | 2 ++ 1 file changed, 2 insertions(+) (limited to 'videocodec/securevideo') diff --git a/videocodec/securevideo/baytrail/OMXVideoDecoderAVCSecure.cpp b/videocodec/securevideo/baytrail/OMXVideoDecoderAVCSecure.cpp index f116d85..467b084 100644 --- a/videocodec/securevideo/baytrail/OMXVideoDecoderAVCSecure.cpp +++ b/videocodec/securevideo/baytrail/OMXVideoDecoderAVCSecure.cpp @@ -319,6 +319,8 @@ OMX_ERRORTYPE OMXVideoDecoderAVCSecure::PrepareDecodeBuffer(OMX_BUFFERHEADERTYPE input.metadata_offset = 1024 * 1024; //Metadata offset is 1MB input.header_offset = (1024*1024)+512; //Header offset is 1M + 512 + memset(&output, 0, sizeof(wv_heci_process_video_frame_out)); + for(int pes_count=0, pesoffset =0, dmaoffset=0; pes_count < secBuffer->num_entries; pes_count++) { dmaoffset = WV_CEILING(dmaoffset,32); -- cgit v1.2.3