History log of /external/libmpeg2/decoder/impeg2d_pnb_pic.c
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
55afb9d20dc6c001fd6667f6a4d7d3399aa2e62c 15-Mar-2018 Venkatarama NG. Avadhani <venkatarama.avadhani@ittiam.com> Adding Check For Number of Skip MBs

Adding check for the value parsed into number of skip MBs. The skip MBs
function should be called only if the number of skip MBs is more than 0.

Bug: 74016277
Test: POC before/after
Change-Id: Iba1cc7b7e214efaeba510d180495a61105bf2125
(cherry picked from commit 414cd5db73a8cc3cd64bdad954475becffa40478)
/external/libmpeg2/decoder/impeg2d_pnb_pic.c
4262d8eeee23d169ab0a141f103592f7172d95bc 12-Jul-2017 Venkatarama Avadhani <venkatarama.avadhani@ittiam.com> Update num_mbs_left When mb_x is Reset.

When we reset the mb_x values, the num_mbs_left should also be
reset accordingly. Added the code for the same in
impeg2d_dec_pnb_mb_params. Also, there was nothing to do in p
frames when the first_mb was set to 1. Added code for setting
mb_x value and updating the num_mbs left similar to
"impeg2d_dec_pnb_mb_params" function.

Bug: 63874456
Test: run PoC through before/after ASAN build
Change-Id: I7a0bfc33e22c0e8cb93ff29198f30052f7f96546
/external/libmpeg2/decoder/impeg2d_pnb_pic.c
89b4c1cf9e2d18c27c2d9c8c7504e5e2d79ef289 27-Jun-2017 Venkatarama Avadhani <venkatarama.avadhani@ittiam.com> Check Number of MBs to Skip.

The number of skip mbs was parsed as 0 and impeg2d_dec_skip_mbs was getting
called with a large number because of underflow.
Added a check for the same

Bug: 63125953
Test: run PoC on ASAN-enabled mpeg2dec before/after
Change-Id: I07f43c1745e38e800751997e97d44d2bab0615a8
/external/libmpeg2/decoder/impeg2d_pnb_pic.c
2df080153464bf57084d68ba3594e199bc140eb4 08-Jun-2017 Venkatarama Avadhani <venkatarama.avadhani@ittiam.com> Fixed Memory Overflow Errors

In function impeg2d_dec_p_b_slice, there was no check for num_mbs_left ==
0 after skip_mbs function call. Hence, even though it should have returned
as an error, it goes ahead to decode the frame and writes beyond the
buffer allocated for output. Put a check for the same.

Bug: 38207066
Test: before/after execution of PoC on angler/nyc-mr2-dev
Change-Id: If4b7bea51032bf2fe2edd03f64a68847aa4f6a00
/external/libmpeg2/decoder/impeg2d_pnb_pic.c
f217b853e7527552290bd047381338f934bccdd6 05-Apr-2017 Venkatarama Avadhani <venkatarama.avadhani@ittiam.com> Check Number of Skip MBs

Adding check to make sure the number of skip MBs do not exceed the total
number of MBs left to decode.

Bug: 34231163
Change-Id: I62ceffdafcbc0c6d580f6ae1b5b9ab0708a7134f
/external/libmpeg2/decoder/impeg2d_pnb_pic.c
58a6822d7140137ce957c6d2fc20bae1374186c1 23-Nov-2015 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Return error for wrong mb_type

If mb_type decoded returns an invalid type of MB, then return error

Bug: 26070014
Change-Id: I66abcad5de1352dd42d05b1a13bb4176153b133c
/external/libmpeg2/decoder/impeg2d_pnb_pic.c
b34913857dacd4b7f82715c48ae1de9c56ede9e0 16-Apr-2015 Venkatarama Avadhani <venkatarama.avadhani@ittiam.com> Fixed few issues seen in CTS tests

Buffer managers are reset to init state in reset()

Return picture type of picture being output instead of picture decoded

Read start code only if input bitstream has enough bytes

Bug: 22860270

Change-Id: Ic158d9dbff6d6b5295173b6ff2a4c6d6da66115f
/external/libmpeg2/decoder/impeg2d_pnb_pic.c
aed24eee7ddfc93f1436b0c1679431bd286879b4 11-Mar-2015 Venkatarama Avadhani <venkatarama.avadhani@ittiam.com> Initial version

Change-Id: I5a9473876e596e7461e91f971b0243f694f7e8fb
/external/libmpeg2/decoder/impeg2d_pnb_pic.c