Do not allow uninstalling while an app is pinned

If an app is pinned we want to avoid ways to unpin without entering a
set passcode. If the package of the base activity in the pinned activity
stack is uninstalled then the device exits pinning mode so we want to
restrict uninstalling this package.

Bug: 135604684
Test: Pin test app, test app tries to uninstall itself
      Pin test app, `adb uninstall`
      Pin test app, test app launches second test app, assert that
          second test app can be uninstalled but base test app can't

Change-Id: I32ee438e9dd9e245bed6e6a9f4efd0abbb70de1f
Merged-In: I32ee438e9dd9e245bed6e6a9f4efd0abbb70de1f
This commit is contained in:
Evan Severson
2020-10-13 13:15:41 -07:00
parent ef7605be2e
commit eeeab037df
5 changed files with 53 additions and 0 deletions

View File

@@ -1688,6 +1688,15 @@ public abstract class PackageManager {
* {@hide} */
public static final int DELETE_FAILED_USED_SHARED_LIBRARY = -6;
/**
* Deletion failed return code: this is passed to the
* {@link IPackageDeleteObserver} if the system failed to delete the package
* because there is an app pinned.
*
* @hide
*/
public static final int DELETE_FAILED_APP_PINNED = -7;
/**
* Return code that is passed to the {@link IPackageMoveObserver} when the
* package has been successfully moved by the system.
@@ -7545,6 +7554,7 @@ public abstract class PackageManager {
case DELETE_FAILED_OWNER_BLOCKED: return "DELETE_FAILED_OWNER_BLOCKED";
case DELETE_FAILED_ABORTED: return "DELETE_FAILED_ABORTED";
case DELETE_FAILED_USED_SHARED_LIBRARY: return "DELETE_FAILED_USED_SHARED_LIBRARY";
case DELETE_FAILED_APP_PINNED: return "DELETE_FAILED_APP_PINNED";
default: return Integer.toString(status);
}
}
@@ -7559,6 +7569,7 @@ public abstract class PackageManager {
case DELETE_FAILED_OWNER_BLOCKED: return PackageInstaller.STATUS_FAILURE_BLOCKED;
case DELETE_FAILED_ABORTED: return PackageInstaller.STATUS_FAILURE_ABORTED;
case DELETE_FAILED_USED_SHARED_LIBRARY: return PackageInstaller.STATUS_FAILURE_CONFLICT;
case DELETE_FAILED_APP_PINNED: return PackageInstaller.STATUS_FAILURE_BLOCKED;
default: return PackageInstaller.STATUS_FAILURE;
}
}

View File

@@ -18162,6 +18162,19 @@ public class PackageManagerService extends IPackageManager.Stub
final String packageName = versionedPackage.getPackageName();
final long versionCode = versionedPackage.getLongVersionCode();
final String internalPackageName;
try {
if (LocalServices.getService(ActivityTaskManagerInternal.class)
.isBaseOfLockedTask(packageName)) {
observer.onPackageDeleted(
packageName, PackageManager.DELETE_FAILED_APP_PINNED, null);
EventLog.writeEvent(0x534e4554, "127605586", -1, "");
return;
}
} catch (RemoteException e) {
e.rethrowFromSystemServer();
}
synchronized (mLock) {
// Normalize package name to handle renamed packages and static libs
internalPackageName = resolveInternalPackageNameLPr(packageName, versionCode);

View File

@@ -568,4 +568,10 @@ public abstract class ActivityTaskManagerInternal {
/** Set all associated companion app that belongs to an userId. */
public abstract void setCompanionAppPackages(int userId, Set<String> companionAppPackages);
/**
* @param packageName The package to check
* @return Whether the package is the base of any locked task
*/
public abstract boolean isBaseOfLockedTask(String packageName);
}

View File

@@ -7473,5 +7473,13 @@ public class ActivityTaskManagerService extends IActivityTaskManager.Stub {
mCompanionAppUidsMap.put(userId, result);
}
}
@Override
public boolean isBaseOfLockedTask(String packageName) {
synchronized (mGlobalLock) {
return getLockTaskController().isBaseOfLockedTask(packageName);
}
}
}
}

View File

@@ -870,6 +870,21 @@ public class LockTaskController {
return new Pair<>(flags1, flags2);
}
/**
* @param packageName The package to check
* @return Whether the package is the base of any locked task
*/
boolean isBaseOfLockedTask(String packageName) {
for (int i = 0; i < mLockTaskModeTasks.size(); i++) {
final Intent bi = mLockTaskModeTasks.get(i).getBaseIntent();
if (bi != null && packageName.equals(bi.getComponent()
.getPackageName())) {
return true;
}
}
return false;
}
/**
* Gets the cached value of LockTask feature flags for a specific user.
*/