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

Commit a542f2c5 authored by Harish Mahendrakar's avatar Harish Mahendrakar Committed by Cherrypicker Worker
Browse files

SoftVideoDecodeOMXComponent: validate OMX params for dynamic HDR 

Bug: 273935108
Bug: 281065553
(cherry picked from https://partner-android-review.googlesource.com/q/commit:b2c67bdcf57149a5e19a04466205266dc543fd86)
Merged-In: I707745594a9196d8d85d4c4bb498eba3c6198b42
Change-Id: I707745594a9196d8d85d4c4bb498eba3c6198b42
parent 4a3ec616

media/libstagefright/omx/SoftVideoDecoderOMXComponent.cpp

+8 −0
Original line number Diff line number Diff line
@@ -616,6 +616,10 @@ OMX_ERRORTYPE SoftVideoDecoderOMXComponent::getConfig( 
                DescribeHDR10PlusInfoParams* outParams =

                        (DescribeHDR10PlusInfoParams *)params;



                if (!isValidOMXParam(outParams)) {

                    return OMX_ErrorBadParameter;

                }



                outParams->nParamSizeUsed = info->size();



                // If the buffer provided by the client does not have enough
@@ -694,6 +698,10 @@ OMX_ERRORTYPE SoftVideoDecoderOMXComponent::internalSetConfig( 
            const DescribeHDR10PlusInfoParams* inParams =

                    (DescribeHDR10PlusInfoParams *)params;



            if (!isValidOMXParam(inParams)) {

                return OMX_ErrorBadParameter;

            }



            if (*frameConfig) {

                // This is a request to append to the current frame config set.

                // For now, we only support kDescribeHdr10PlusInfoIndex, which