From 5518cf584b7759ddd0b326a9b9213d28d7d8c041 Mon Sep 17 00:00:00 2001 From: Jeff Sharkey Date: Fri, 24 Aug 2018 17:37:59 -0600 Subject: [PATCH] No permissions needed for static disk attributes. Detecting if a storage device has quota or resgid support isn't sensitive, so we're okay letting anyone ask about it. Bug: 112175169 Test: atest android.appsecurity.cts.StorageHostTest#testFullDisk Change-Id: I1e45eeade0d0d8ea242afca274b45a68972e6b57 --- .../java/com/android/server/usage/StorageStatsService.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/services/usage/java/com/android/server/usage/StorageStatsService.java b/services/usage/java/com/android/server/usage/StorageStatsService.java index 61d6b7def981e..d12eda75ff328 100644 --- a/services/usage/java/com/android/server/usage/StorageStatsService.java +++ b/services/usage/java/com/android/server/usage/StorageStatsService.java @@ -155,8 +155,6 @@ public class StorageStatsService extends IStorageStatsManager.Stub { @Override public boolean isQuotaSupported(String volumeUuid, String callingPackage) { - enforcePermission(Binder.getCallingUid(), callingPackage); - try { return mInstaller.isQuotaSupported(volumeUuid); } catch (InstallerException e) { @@ -166,8 +164,6 @@ public class StorageStatsService extends IStorageStatsManager.Stub { @Override public boolean isReservedSupported(String volumeUuid, String callingPackage) { - enforcePermission(Binder.getCallingUid(), callingPackage); - if (volumeUuid == StorageManager.UUID_PRIVATE_INTERNAL) { return SystemProperties.getBoolean(StorageManager.PROP_HAS_RESERVED, false); } else {