From c3b026a87d7da17ca5196e1973137b8691e60bde Mon Sep 17 00:00:00 2001 From: Ritu Baldwa Date: Wed, 27 Dec 2017 17:45:30 +0530 Subject: [PATCH] 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 --- decoder/ih264d_utils.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/decoder/ih264d_utils.c b/decoder/ih264d_utils.c index 170bc97..d60db67 100644 --- a/decoder/ih264d_utils.c +++ b/decoder/ih264d_utils.c @@ -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); + } } }