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

Commit c7455a0b authored by Pawin Vongmasa's avatar Pawin Vongmasa
Browse files

Check the return value from allocateSecureBuffer 

Test: Small CtsMediaTestCases

Test: com.google.android.media.gts.WidevineCodecStressTests in
GtsMediaTestCases

Bug: 38452958
Change-Id: Iba53c3fc144a3ee7d1f54b62ab3836b1a3a46dc6
parent 873b2bc8

media/libmedia/omx/1.0/WOmxNode.cpp

+2 −1
Original line number Diff line number Diff line
@@ -350,7 +350,8 @@ Return<void> TWOmxNode::allocateSecureBuffer( 
            &buffer,

            &bufferData,

            &nativeHandle));

    _hidl_cb(status, buffer, nativeHandle->handle());

    _hidl_cb(status, buffer, nativeHandle == nullptr ?

            nullptr : nativeHandle->handle());

    return Void();

}

media/libstagefright/omx/1.0/WOmxNode.cpp

+2 −1
Original line number Diff line number Diff line
@@ -353,7 +353,8 @@ Return<void> TWOmxNode::allocateSecureBuffer( 
            &buffer,

            &bufferData,

            &nativeHandle));

    _hidl_cb(status, buffer, nativeHandle->handle());

    _hidl_cb(status, buffer, nativeHandle == nullptr ?

            nullptr : nativeHandle->handle());

    return Void();

}