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

Merge "Changed INTERACT_ACROSS_PROFILES appop to be set per UID" into rvc-qpr-dev

Browse Source
This commit is contained in:
Kholoud Mohamed
2021-09-08 10:56:21 +00:00
committed by Android (Google) Code Review
parent 7c93ad5848 f5e0e80312
commit b4a530d83b
1 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
services/core/java/com/android/server/pm/CrossProfileAppsServiceImpl.java
View File

@@ -482,16 +482,16 @@ public class CrossProfileAppsServiceImpl extends ICrossProfileApps.Stub {
// this particular app-op to be modified without the broader app-op permissions. // this particular app-op to be modified without the broader app-op permissions.
mInjector.withCleanCallingIdentity(() -> mInjector.withCleanCallingIdentity(() ->
mInjector.getAppOpsManager() mInjector.getAppOpsManager()
.setMode(OP_INTERACT_ACROSS_PROFILES, uid, packageName, newMode)); .setUidMode(OP_INTERACT_ACROSS_PROFILES, uid, newMode));
} else { } else {
mInjector.getAppOpsManager() mInjector.getAppOpsManager()
.setMode(OP_INTERACT_ACROSS_PROFILES, uid, packageName, newMode); .setUidMode(OP_INTERACT_ACROSS_PROFILES, uid, newMode);
} }
// Kill the UID before sending the broadcast to ensure that apps can be informed when // Kill the UID before sending the broadcast to ensure that apps can be informed when
// their app-op has been revoked. // their app-op has been revoked.
maybeKillUid(packageName, uid, hadPermission); maybeKillUid(packageName, uid, hadPermission);
sendCanInteractAcrossProfilesChangedBroadcast(packageName, uid, UserHandle.of(profileId)); sendCanInteractAcrossProfilesChangedBroadcast(packageName, UserHandle.of(profileId));
maybeLogSetInteractAcrossProfilesAppOp(packageName, newMode, logMetrics, uid); maybeLogSetInteractAcrossProfilesAppOp(packageName, newMode, logMetrics);
} }
/** /**
@@ -509,7 +509,7 @@ public class CrossProfileAppsServiceImpl extends ICrossProfileApps.Stub {
} }
private void maybeLogSetInteractAcrossProfilesAppOp( private void maybeLogSetInteractAcrossProfilesAppOp(
String packageName, @Mode int newMode, boolean logMetrics, int uid) { String packageName, @Mode int newMode, boolean logMetrics) {
if (!logMetrics) { if (!logMetrics) {
return; return;
} }
@@ -517,7 +517,7 @@ public class CrossProfileAppsServiceImpl extends ICrossProfileApps.Stub {
.createEvent(DevicePolicyEnums.SET_INTERACT_ACROSS_PROFILES_APP_OP) .createEvent(DevicePolicyEnums.SET_INTERACT_ACROSS_PROFILES_APP_OP)
.setStrings(packageName) .setStrings(packageName)
.setInt(newMode) .setInt(newMode)
.setBoolean(appDeclaresCrossProfileAttribute(uid)) .setBoolean(appDeclaresCrossProfileAttribute(packageName))
.write(); .write();
} }
@@ -533,10 +533,10 @@ public class CrossProfileAppsServiceImpl extends ICrossProfileApps.Stub {
} }
private void sendCanInteractAcrossProfilesChangedBroadcast( private void sendCanInteractAcrossProfilesChangedBroadcast(
String packageName, int uid, UserHandle userHandle) { String packageName, UserHandle userHandle) {
final Intent intent = final Intent intent =
new Intent(ACTION_CAN_INTERACT_ACROSS_PROFILES_CHANGED).setPackage(packageName); new Intent(ACTION_CAN_INTERACT_ACROSS_PROFILES_CHANGED).setPackage(packageName);
if (appDeclaresCrossProfileAttribute(uid)) { if (appDeclaresCrossProfileAttribute(packageName)) {
intent.addFlags( intent.addFlags(
Intent.FLAG_RECEIVER_INCLUDE_BACKGROUND | Intent.FLAG_RECEIVER_FOREGROUND); Intent.FLAG_RECEIVER_INCLUDE_BACKGROUND | Intent.FLAG_RECEIVER_FOREGROUND);
} else { } else {
@@ -553,8 +553,8 @@ public class CrossProfileAppsServiceImpl extends ICrossProfileApps.Stub {
.queryBroadcastReceiversAsUser(intent, /* flags= */ 0, userHandle); .queryBroadcastReceiversAsUser(intent, /* flags= */ 0, userHandle);
} }
private boolean appDeclaresCrossProfileAttribute(int uid) { private boolean appDeclaresCrossProfileAttribute(String packageName) {
return mInjector.getPackageManagerInternal().getPackage(uid).isCrossProfile(); return mInjector.getPackageManagerInternal().getPackage(packageName).isCrossProfile();
} }
@Override @Override