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

Commit 8558ec7d authored by Bryce Lee's avatar Bryce Lee
Browse files

Don't resume activity if user is locked and encryption is unsupported 

Previously we were not checking if to see if this condition was met
when determining the top running activity. As a result we could get
in a loop where we force stop the process since the app doesn't
support encryption yet try to start it again due to the
ActivityRecord being on top.

This changelist addresses the issue by checking this condition when
determining whether it is okay to show.

Change-Id: I868f97ce8b3ea4220019b8570407e33f27ea88bc
Fixes: 64259425
Test: go/wm-smoke
parent c3e7f875

core/java/android/content/pm/ApplicationInfo.java

+5 −0
Original line number Diff line number Diff line
@@ -1508,6 +1508,11 @@ public class ApplicationInfo extends PackageItemInfo implements Parcelable { 
        return (privateFlags & ApplicationInfo.PRIVATE_FLAG_PARTIALLY_DIRECT_BOOT_AWARE) != 0;

    }



    /** @hide */

    public boolean isEncryptionAware() {

        return isDirectBootAware() || isPartiallyDirectBootAware();

    }



    /**

     * @hide

     */

services/core/java/com/android/server/am/ActivityRecord.java

+8 −0
Original line number Diff line number Diff line
@@ -146,6 +146,7 @@ import android.os.RemoteException; 
import android.os.SystemClock;

import android.os.Trace;

import android.os.UserHandle;

import android.os.storage.StorageManager;

import android.service.voice.IVoiceInteractionSession;

import android.util.EventLog;

import android.util.Log;
@@ -2013,6 +2014,13 @@ final class ActivityRecord extends ConfigurationContainer implements AppWindowCo 


    /** Checks whether the activity should be shown for current user. */

    public boolean okToShowLocked() {

        // We cannot show activities when the device is locked and the application is not

        // encryption aware.

        if (!StorageManager.isUserKeyUnlocked(userId)

                && !info.applicationInfo.isEncryptionAware()) {

            return false;

        }



        return (info.flags & FLAG_SHOW_FOR_ALL_USERS) != 0

                || (mStackSupervisor.isCurrentProfileLocked(userId)

                && service.mUserController.isUserRunningLocked(userId, 0 /* flags */));

services/core/java/com/android/server/pm/PackageManagerService.java

+1 −2
Original line number Diff line number Diff line
@@ -3884,8 +3884,7 @@ public class PackageManagerService extends IPackageManager.Stub 
                throw new SecurityException("Package " + packageName + " is currently frozen!");

            }













            if (!userKeyUnlocked && !(ps.pkg.applicationInfo.isDirectBootAware()













                    || ps.pkg.applicationInfo.isPartiallyDirectBootAware())) {













            if (!userKeyUnlocked && !ps.pkg.applicationInfo.isEncryptionAware()) {















                throw new SecurityException("Package " + packageName + " is not encryption aware!");















            }















        }