Commit eedcf47e authored Dec 31, 2017 by Adrian DC Committed by Michael Bestas Dec 12, 2020

stagefright: ACodec: Resolve empty vendor parameters usage



 * On devices where vendorKeys is empty, the status was left
    as kExtensionsUnchecked and would block getVendorParameters
    on a wrong loop of empty vendor keys

 * Instead, set the status to kExtensionsNone to validate
    the check and the empty vendor keys, matching the old
    behaviour without vendor keys in getPortFormat

Change-Id: Ic3bdc9454c6af8e5d41ab901d4156ccb62281974
Signed-off-by: Adrian DC <radian.dc@gmail.com>

parent f8cc94a9

media/libstagefright/ACodec.cpp

+1 −0

Original line number	Diff line number	Diff line
		@@ -7916,6 +7916,7 @@ status_t ACodec::setVendorParameters(const sp<AMessage> &params) {
		// don't bother component if we don't have vendor extensions as they may not have implemented
		// the android vendor extension support, which will lead to unnecessary OMX failure logs.
		if (vendorKeys.empty()) {
		mVendorExtensionsStatus = kExtensionsNone;
		return OK;
		}