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

Commit 547789d2 authored by Eric Laurent's avatar Eric Laurent
Browse files

audioflinger: fix wake lock acquisition check 

Commit e14a5d6d introduced a bug in ThreadBase::acquireWakeLock() where
the wake lock acquisition return code is not initialized before
being checked causing the wake lock token to be sometines not set
and potential failure to release the wake lock later.

Bug: 10985160.
Change-Id: Iffd40e773ae3bcfec3c148a844d5dbebbf474eaf
parent 7bab0248

services/audioflinger/Threads.cpp

+2 −2
Original line number Diff line number Diff line
@@ -499,13 +499,13 @@ void AudioFlinger::ThreadBase::acquireWakeLock_l(int uid) 
        sp<IBinder> binder = new BBinder();

        status_t status;

        if (uid >= 0) {

            mPowerManager->acquireWakeLockWithUid(POWERMANAGER_PARTIAL_WAKE_LOCK,

            status = mPowerManager->acquireWakeLockWithUid(POWERMANAGER_PARTIAL_WAKE_LOCK,

                    binder,

                    String16(mName),

                    String16("media"),

                    uid);

        } else {

            mPowerManager->acquireWakeLock(POWERMANAGER_PARTIAL_WAKE_LOCK,

            status = mPowerManager->acquireWakeLock(POWERMANAGER_PARTIAL_WAKE_LOCK,

                    binder,

                    String16(mName),

                    String16("media"));