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

Commit acb731f9 authored by Jonathan Scott's avatar Jonathan Scott
Browse files

[RESTRICT AUTOMERGE] Use userId instead of USER_CURRENT in shouldLockKeyguard. 

USER_CURRENT is not an acceptable input to isSecure(). The test has also
been updated as the use of any() for the mock hid this bug.

Test: atest LockTaskControllerTest
Bug: 176801033
Change-Id: I8aa736bb7b8b60937ad4319e6127c356be5e9b55
parent 6c1e9153

services/core/java/com/android/server/wm/LockTaskController.java

+5 −5
Original line number Diff line number Diff line
@@ -512,7 +512,7 @@ public class LockTaskController { 
            setStatusBarState(mLockTaskModeState, userId);

            setKeyguardState(mLockTaskModeState, userId);

            if (oldLockTaskModeState == LOCK_TASK_MODE_PINNED) {

                lockKeyguardIfNeeded();

                lockKeyguardIfNeeded(userId);

            }

            if (getDevicePolicyManager() != null) {

                getDevicePolicyManager().notifyLockTaskModeChanged(false, null, userId);
@@ -824,15 +824,15 @@ public class LockTaskController { 
     * Helper method for locking the device immediately. This may be necessary when the device

     * leaves the pinned mode.

     */

    private void lockKeyguardIfNeeded() {

        if (shouldLockKeyguard()) {

    private void lockKeyguardIfNeeded(int userId) {

        if (shouldLockKeyguard(userId)) {

            mWindowManager.lockNow(null);

            mWindowManager.dismissKeyguard(null /* callback */, null /* message */);

            getLockPatternUtils().requireCredentialEntry(USER_ALL);

        }

    }



    private boolean shouldLockKeyguard() {

    private boolean shouldLockKeyguard(int userId) {

        // This functionality should be kept consistent with

        // com.android.settings.security.ScreenPinningSettings (see b/127605586)

        try {
@@ -842,7 +842,7 @@ public class LockTaskController { 
        } catch (Settings.SettingNotFoundException e) {

            // Log to SafetyNet for b/127605586

            android.util.EventLog.writeEvent(0x534e4554, "127605586", -1, "");

            return getLockPatternUtils().isSecure(USER_CURRENT);

            return getLockPatternUtils().isSecure(userId);

        }

    }

services/tests/wmtests/src/com/android/server/wm/LockTaskControllerTest.java

+1 −1
Original line number Diff line number Diff line
@@ -450,7 +450,7 @@ public class LockTaskControllerTest { 
        Settings.Secure.clearProviderForTest();



        // AND a password is set

        when(mLockPatternUtils.isSecure(anyInt()))

        when(mLockPatternUtils.isSecure(TEST_USER_ID))

                .thenReturn(true);



        // AND there is a task record