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

Commit 5ad4ec37 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Android (Google) Code Review
Browse files

Merge "Do not re-initialize synthetic password" into rvc-dev

parents f1c03846 2d51788b

services/core/java/com/android/server/locksettings/LockSettingsService.java

+6 −1
Original line number Diff line number Diff line
@@ -113,6 +113,7 @@ import com.android.internal.messages.nano.SystemMessageProto.SystemMessage; 
import com.android.internal.notification.SystemNotificationChannels;

import com.android.internal.util.DumpUtils;

import com.android.internal.util.IndentingPrintWriter;

import com.android.internal.util.Preconditions;

import com.android.internal.widget.ICheckCredentialProgressCallback;

import com.android.internal.widget.ILockSettings;

import com.android.internal.widget.LockPatternUtils;
@@ -2618,6 +2619,10 @@ public class LockSettingsService extends ILockSettings.Stub { 
    protected AuthenticationToken initializeSyntheticPasswordLocked(byte[] credentialHash,

            LockscreenCredential credential, int userId) {

        Slog.i(TAG, "Initialize SyntheticPassword for user: " + userId);

        Preconditions.checkState(

                getSyntheticPasswordHandleLocked(userId) == SyntheticPasswordManager.DEFAULT_HANDLE,

                "Cannot reinitialize SP");



        final AuthenticationToken auth = mSpManager.newSyntheticPasswordAndSid(

                getGateKeeperService(), credentialHash, credential, userId);

        onAuthTokenKnownForUser(userId, auth);
@@ -2678,7 +2683,7 @@ public class LockSettingsService extends ILockSettings.Stub { 


    @VisibleForTesting

    protected boolean shouldMigrateToSyntheticPasswordLocked(int userId) {

        return true;

        return getSyntheticPasswordHandleLocked(userId) == SyntheticPasswordManager.DEFAULT_HANDLE;

    }



    private VerifyCredentialResponse spBasedDoVerifyCredential(LockscreenCredential userCredential,

services/tests/servicestests/src/com/android/server/locksettings/SyntheticPasswordTests.java

+16 −2
Original line number Diff line number Diff line
@@ -519,10 +519,24 @@ public class SyntheticPasswordTests extends BaseLockSettingsServiceTests { 
        LockscreenCredential password = newPassword("password");

        initializeCredentialUnderSP(password, PRIMARY_USER_ID);

        assertTrue(mService.setLockCredential(password, password, PRIMARY_USER_ID));

        assertNoOrphanedFilesLeft(PRIMARY_USER_ID);

    }



    @Test

    public void testAddingEscrowToken_NoOrphanedFilesLeft() throws Exception {

        final byte[] token = "some-high-entropy-secure-token".getBytes();

        for (int i = 0; i < 16; i++) {

            long handle = mLocalService.addEscrowToken(token, PRIMARY_USER_ID, null);

            assertTrue(mLocalService.isEscrowTokenActive(handle, PRIMARY_USER_ID));

            mLocalService.removeEscrowToken(handle, PRIMARY_USER_ID);

        }

        assertNoOrphanedFilesLeft(PRIMARY_USER_ID);

    }



    private void assertNoOrphanedFilesLeft(int userId) {

        String handleString = String.format("%016x",

                mService.getSyntheticPasswordHandleLocked(PRIMARY_USER_ID));

        File directory = mStorage.getSyntheticPasswordDirectoryForUser(PRIMARY_USER_ID);

                mService.getSyntheticPasswordHandleLocked(userId));

        File directory = mStorage.getSyntheticPasswordDirectoryForUser(userId);

        for (File file : directory.listFiles()) {

            String[] parts = file.getName().split("\\.");

            if (!parts[0].equals(handleString) && !parts[0].equals("0000000000000000")) {