tree 86f62e412e070bef1227b1f38c187b979499c1b0
parent d384ba5cdd182643350b38f6ca23ac72c7bb6f19
author wfeng6 <wei.feng@intel.com> 1376045503 +0800
committer Patrick Tjin <pattjin@google.com> 1406004699 -0700

To fix two critical Klocwork issues in libmix

BZ: 129623

Two critical issues in libmix:
1. Null pointer 'data' that comes from line 160
    may be dereferenced at line 182. Also there
    is one similar error on line 190.
    File: libmix/videodecoder/securevideo/
    baytrail/VideoDecoderAVCSecure.cpp:182 | decode()

2.  Array 'op' of size 16 may use index value(s) 16.
    Also there are 6 similar errors on line(s) 542,
    547, 552, 556, 557, 562.
    File: libmix/mix_vbp/viddec_fw/fw/codecs/
    h264/parser/viddec_h264secure_parse.c:538
    | h264secure_Parse_Dec_Ref_Pic_Marking()

Fix:
1.  Check the 'data' pointer before dereference it
2.  Limit the idx value to be smaller than 16

Change-Id: Iba48665147822a23873ca8ef3c0fc5b0f3de03ee
Signed-off-by: wfeng6 <wei.feng@intel.com>
Reviewed-on: http://android.intel.com:8080/124651
Reviewed-by: Shi, PingX <pingx.shi@intel.com>
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>