diff options
| author | wfeng6 <wei.feng@intel.com> | 2013-08-09 16:08:41 +0800 |
|---|---|---|
| committer | Patrick Tjin <pattjin@google.com> | 2014-07-21 22:02:52 -0700 |
| commit | e971a6d8555f8e05e86836705734880f41c91d11 (patch) | |
| tree | e8707dc75c7aac778e614d2fa57ee2375af53157 /videocodec | |
| parent | 9bf33af84b0d334e97dbdacf74bde071402c31c5 (diff) | |
| download | android_hardware_intel_common_omx-components-e971a6d8555f8e05e86836705734880f41c91d11.tar.gz android_hardware_intel_common_omx-components-e971a6d8555f8e05e86836705734880f41c91d11.tar.bz2 android_hardware_intel_common_omx-components-e971a6d8555f8e05e86836705734880f41c91d11.zip | |
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 <wei.feng@intel.com>
Reviewed-on: http://android.intel.com:8080/124619
Reviewed-by: Guo, Nana N <nana.n.guo@intel.com>
Tested-by: Ding, Haitao <haitao.ding@intel.com>
Reviewed-by: cactus <cactus@intel.com>
Tested-by: cactus <cactus@intel.com>
Diffstat (limited to 'videocodec')
| -rw-r--r-- | videocodec/securevideo/baytrail/OMXVideoDecoderAVCSecure.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
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); |