Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6b6b1889 authored by Harish Mahendrakar's avatar Harish Mahendrakar
Browse files

C2SoftHevcDec: Initialize mStride after first header decode 

After first header decode, decoder stride was set to ALIGN128(width),
but mStride that tracks the stride in the plugin wasn't set to this.

When allocated buffer has a stride that is different from
ALIGN128(width), this would lead to wrong stride used in the decoder.

Bug: 238984399
Test: atest CtsMediaV2TestCases -- --module-arg \
CtsMediaV2TestCases:instrumentation-arg:\
codec-prefix:=c2.android.hevc.decoder

Change-Id: I06886cc718a1b8b9fa247b248b23adb2feb80f90
parent 6906fb05

media/codec2/components/hevc/C2SoftHevcDec.cpp

+2 −1
Original line number Diff line number Diff line
@@ -917,7 +917,8 @@ void C2SoftHevcDec::process( 
        if (0 < ps_decode_op->u4_pic_wd && 0 < ps_decode_op->u4_pic_ht) {

            if (mHeaderDecoded == false) {

                mHeaderDecoded = true;

                setParams(ALIGN128(ps_decode_op->u4_pic_wd), IVD_DECODE_FRAME);

                mStride = ALIGN128(ps_decode_op->u4_pic_wd);

                setParams(mStride, IVD_DECODE_FRAME);

            }

            if (ps_decode_op->u4_pic_wd != mWidth ||  ps_decode_op->u4_pic_ht != mHeight) {

                mWidth = ps_decode_op->u4_pic_wd;