Merge "DO NOT MERGE: Disable ENABLE_DYNAMIC_PERMISSIONS without helper method." into rvc-qpr-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
03b8867166
@@ -115,7 +115,7 @@ public class UriGrantsManagerService extends IUriGrantsManager.Stub {
|
||||
private static final String TAG = "UriGrantsManagerService";
|
||||
// Maximum number of persisted Uri grants a package is allowed
|
||||
private static final int MAX_PERSISTED_URI_GRANTS = 512;
|
||||
private static final boolean ENABLE_DYNAMIC_PERMISSIONS = true;
|
||||
private static final boolean ENABLE_DYNAMIC_PERMISSIONS = false;
|
||||
|
||||
private final Object mLock = new Object();
|
||||
private final H mH;
|
||||
@@ -991,9 +991,7 @@ public class UriGrantsManagerService extends IUriGrantsManager.Stub {
|
||||
// If this provider says that grants are always required, we need to
|
||||
// consult it directly to determine if the UID has permission
|
||||
final boolean forceMet;
|
||||
if (ENABLE_DYNAMIC_PERMISSIONS
|
||||
&& pi.forceUriPermissions
|
||||
&& isDynamicPermissionEnabledInMP()) {
|
||||
if (ENABLE_DYNAMIC_PERMISSIONS && pi.forceUriPermissions) {
|
||||
final int providerUserId = UserHandle.getUserId(pi.applicationInfo.uid);
|
||||
final int clientUserId = UserHandle.getUserId(uid);
|
||||
if (providerUserId == clientUserId) {
|
||||
@@ -1011,15 +1009,6 @@ public class UriGrantsManagerService extends IUriGrantsManager.Stub {
|
||||
return readMet && writeMet && forceMet;
|
||||
}
|
||||
|
||||
/**
|
||||
* Returns true if the available MediaProvider version contains the changes that enable dynamic
|
||||
* permission.
|
||||
*/
|
||||
private boolean isDynamicPermissionEnabledInMP() {
|
||||
// TODO(b/159995598) Check MediaProvider version.
|
||||
return false;
|
||||
}
|
||||
|
||||
@GuardedBy("mLock")
|
||||
private void removeUriPermissionIfNeededLocked(UriPermission perm) {
|
||||
if (perm.modeFlags != 0) {
|
||||
|
||||
Reference in New Issue
Block a user