Decoder: Fixed memory overflow in shared display mode.

The factor multiplication should happen only at the source,
not at the destination.

Bug: 71375536
Test: manual
Change-Id: Ib5f00b87150a0533880346fac5464b0b1a802c36
This commit is contained in:
Ritu Baldwa 2017-12-27 17:45:30 +05:30 committed by Ray Essick
parent 7ea47d575d
commit c3b026a87d

View file

@ -1267,12 +1267,15 @@ WORD32 ih264d_get_next_display_field(dec_struct_t * ps_dec,
buf = ps_dec->disp_bufs[i].buf[1];
buf += ps_dec->disp_bufs[i].u4_ofst[1];
pv_disp_op->s_disp_frm_buf.pv_u_buf = buf
+ pic_buf->u2_crop_offset_uv;
+ (pic_buf->u2_crop_offset_uv
/ YUV420SP_FACTOR);
buf = ps_dec->disp_bufs[i].buf[2];
buf += ps_dec->disp_bufs[i].u4_ofst[2];
pv_disp_op->s_disp_frm_buf.pv_v_buf = buf
+ pic_buf->u2_crop_offset_uv;
+ (pic_buf->u2_crop_offset_uv
/ YUV420SP_FACTOR);
}
}
}