Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Don't kill apps for OP_REQUEST_INSTALL_PACKAGES on targetSdk < 26." into rvc-dev am: 91d3f2a4fc

Browse Source 
Change-Id: Ie441e6e7418507fd5c649a4a1e63543b34f56382
This commit is contained in:
Martijn Coenen
2020-05-25 19:10:52 +00:00
committed by Automerger Merge Worker
parent 2f34b5f9fb 91d3f2a4fc
commit e4836a412b
1 changed files with 19 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
services/core/java/com/android/server/StorageManagerService.java
View File

@@ -83,6 +83,7 @@ import android.content.res.ObbInfo;
import android.database.ContentObserver; import android.database.ContentObserver;
import android.net.Uri; import android.net.Uri;
import android.os.Binder; import android.os.Binder;
import android.os.Build;
import android.os.DropBoxManager; import android.os.DropBoxManager;
import android.os.Environment; import android.os.Environment;
import android.os.Handler; import android.os.Handler;
@@ -4662,6 +4663,10 @@ class StorageManagerService extends IStorageManager.Stub
} }
public void onAppOpsChanged(int code, int uid, @Nullable String packageName, int mode) { public void onAppOpsChanged(int code, int uid, @Nullable String packageName, int mode) {
if (packageName == null) {
// This happens :(
return;
}
final long token = Binder.clearCallingIdentity(); final long token = Binder.clearCallingIdentity();
try { try {
if (mIsFuseEnabled) { if (mIsFuseEnabled) {
@@ -4669,7 +4674,20 @@ class StorageManagerService extends IStorageManager.Stub
switch(code) { switch(code) {
case OP_REQUEST_INSTALL_PACKAGES: case OP_REQUEST_INSTALL_PACKAGES:
// Always kill regardless of op change, to remount apps /storage // Always kill regardless of op change, to remount apps /storage
try {
ApplicationInfo ai = mIPackageManager.getApplicationInfo(
packageName,
0, UserHandle.getUserId(uid));
if (ai.targetSdkVersion >= Build.VERSION_CODES.O) {
killAppForOpChange(code, uid, packageName); killAppForOpChange(code, uid, packageName);
} else {
// Apps targeting <26 didn't need this app op to install
// packages - they only need the manifest permission, instead.
// So, there's also no need to kill them.
}
} catch (RemoteException e) {
// Ignore, this is an in-process call
}
return; return;
case OP_MANAGE_EXTERNAL_STORAGE: case OP_MANAGE_EXTERNAL_STORAGE:
if (mode != MODE_ALLOWED) { if (mode != MODE_ALLOWED) {