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

Commit 09a90ab5 authored by Jeff Tinker's avatar Jeff Tinker
Browse files

Fix missing check of hidl return status 

closeAllSessions() was calling into the hidl interface closeSession
directly, and neglected to check the return status which resulted in
an abort. Instead, call the DrmHal::closeSession() method which handles
the hidl return status correctly.

Test: gts media tests

bug:72400509
Change-Id: I444be998702aeaca2905ddf560a5138d5775a316
parent 87b46795

drm/libmediadrm/DrmHal.cpp

+3 −3
Original line number Diff line number Diff line
@@ -197,9 +197,9 @@ DrmHal::DrmHal() 


void DrmHal::closeOpenSessions() {

    if (mPlugin != NULL) {

        for (size_t i = 0; i < mOpenSessions.size(); i++) {

            mPlugin->closeSession(toHidlVec(mOpenSessions[i]));

            DrmSessionManager::Instance()->removeSession(mOpenSessions[i]);

        auto openSessions = mOpenSessions;

        for (size_t i = 0; i < openSessions.size(); i++) {

            closeSession(openSessions[i]);

        }

    }

    mOpenSessions.clear();