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

Merge "Ignore package mode for MANAGE_EXTERNAL_STORAGE" into rvc-dev

Browse Source
This commit is contained in:
Shafik Nassar
2020-02-28 15:30:58 +00:00
committed by Android (Google) Code Review
parent 8cf381064f 3ea80350ef
commit ab2132d460
1 changed files with 15 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
services/core/java/com/android/server/appop/AppOpsService.java
View File

@@ -4060,9 +4060,11 @@ public class AppOpsService extends IAppOpsService.Stub {
private void readOp(XmlPullParser parser, @NonNull UidState uidState, private void readOp(XmlPullParser parser, @NonNull UidState uidState,
@NonNull String pkgName, boolean isPrivileged) throws NumberFormatException, @NonNull String pkgName, boolean isPrivileged) throws NumberFormatException,
XmlPullParserException, IOException { XmlPullParserException, IOException {
Op op = new Op(uidState, pkgName, int opCode = Integer.parseInt(parser.getAttributeValue(null, "n"));
Integer.parseInt(parser.getAttributeValue(null, "n")), if (isIgnoredAppOp(opCode)) {
uidState.uid); return;
}
Op op = new Op(uidState, pkgName, opCode, uidState.uid);
final int mode = XmlUtils.readIntAttribute(parser, "m", final int mode = XmlUtils.readIntAttribute(parser, "m",
AppOpsManager.opToDefaultMode(op.op)); AppOpsManager.opToDefaultMode(op.op));
@@ -4096,6 +4098,16 @@ public class AppOpsService extends IAppOpsService.Stub {
ops.put(op.op, op); ops.put(op.op, op);
} }
//TODO(b/149995538): Remove once this has reached all affected devices
private static boolean isIgnoredAppOp(int op) {
switch (op) {
case AppOpsManager.OP_MANAGE_EXTERNAL_STORAGE:
return true;
default:
return false;
}
}
void writeState() { void writeState() {
synchronized (mFile) { synchronized (mFile) {
FileOutputStream stream; FileOutputStream stream;