diff options
author | Hamsalekha S <hamsalekha.s@ittiam.com> | 2017-08-09 13:41:39 +0530 |
---|---|---|
committer | mse1969 <mse1969@posteo.de> | 2018-01-10 22:35:32 +0100 |
commit | 17c125b6515ffe2f4f685d2a025085d9868cb9b7 (patch) | |
tree | da670a7264f551ddceb3bb2da3a7b584332e7038 | |
parent | fa0b6fb711db3fa15ef59e7a88a98161bcb36d02 (diff) | |
download | android_external_libavc-17c125b6515ffe2f4f685d2a025085d9868cb9b7.tar.gz android_external_libavc-17c125b6515ffe2f4f685d2a025085d9868cb9b7.tar.bz2 android_external_libavc-17c125b6515ffe2f4f685d2a025085d9868cb9b7.zip |
Decoder: Detect change of mbaff flag in SPS
Change in Mbaff flag needs re-initialization of NMB group
and other variables in decoder context.
Bug: 64380237
Test: ran poc on ASAN before/after
Change-Id: I0fc65e4dfc3cc2c15528ec52da1782ecec61feab
(cherry picked from commit d524ba03101c0c662c9d365d7357536b42a0265e)
CVE-2017-13204
-rw-r--r-- | decoder/ih264d_parse_headers.c | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/decoder/ih264d_parse_headers.c b/decoder/ih264d_parse_headers.c index d8c37a6..2314003 100644 --- a/decoder/ih264d_parse_headers.c +++ b/decoder/ih264d_parse_headers.c @@ -479,7 +479,7 @@ WORD32 ih264d_parse_sps(dec_struct_t *ps_dec, dec_bit_stream_t *ps_bitstrm) { UWORD8 i; dec_seq_params_t *ps_seq = NULL; - UWORD8 u1_profile_idc, u1_level_idc, u1_seq_parameter_set_id; + UWORD8 u1_profile_idc, u1_level_idc, u1_seq_parameter_set_id, u1_mb_aff_flag = 0; UWORD16 i2_max_frm_num; UWORD32 *pu4_bitstrm_buf = ps_bitstrm->pu4_buffer; UWORD32 *pu4_bitstrm_ofst = &ps_bitstrm->u4_ofst; @@ -796,9 +796,19 @@ WORD32 ih264d_parse_sps(dec_struct_t *ps_dec, dec_bit_stream_t *ps_bitstrm) COPYTHECONTEXT("SPS: frame_mbs_only_flag", u1_frm); if(!u1_frm) + u1_mb_aff_flag = ih264d_get_bit_h264(ps_bitstrm); + + if((ps_dec->i4_header_decoded & 1) + && (ps_seq->u1_mb_aff_flag != u1_mb_aff_flag)) + { + ps_dec->u1_res_changed = 1; + return IVD_RES_CHANGED; + } + + if(!u1_frm) { u2_pic_ht <<= 1; - ps_seq->u1_mb_aff_flag = ih264d_get_bit_h264(ps_bitstrm); + ps_seq->u1_mb_aff_flag = u1_mb_aff_flag; COPYTHECONTEXT("SPS: mb_adaptive_frame_field_flag", ps_seq->u1_mb_aff_flag); |