diff options
| author | 2017-05-05 15:26:13 -0600 | |
|---|---|---|
| committer | 2017-05-05 15:26:25 -0600 | |
| commit | 2abd66c4ffdb7905128b1ca245d4ccb97cbda1c8 (patch) | |
| tree | 70d0080c5a63db021d96c416a0a94b49052abd8b /services/usage/java | |
| parent | e44125cd3a5fc25306285bdfcacc00459e01307b (diff) | |
Defeat @Nullable String[] with empty list.
Test: builds, boots
Bug: 37577123
Change-Id: I6bf1af565398c30aecc0939a23846c0f85c13ffe
Diffstat (limited to 'services/usage/java')
| -rw-r--r-- | services/usage/java/com/android/server/usage/StorageStatsService.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/services/usage/java/com/android/server/usage/StorageStatsService.java b/services/usage/java/com/android/server/usage/StorageStatsService.java index e13665bed275..d9abc1595fc6 100644 --- a/services/usage/java/com/android/server/usage/StorageStatsService.java +++ b/services/usage/java/com/android/server/usage/StorageStatsService.java @@ -16,6 +16,8 @@ package com.android.server.usage; +import static com.android.internal.util.ArrayUtils.defeatNullable; + import android.app.AppOpsManager; import android.app.usage.ExternalStorageStats; import android.app.usage.IStorageStatsManager; @@ -232,7 +234,7 @@ public class StorageStatsService extends IStorageStatsManager.Stub { enforcePermission(Binder.getCallingUid(), callingPackage); } - if (mPackage.getPackagesForUid(appInfo.uid).length == 1) { + if (defeatNullable(mPackage.getPackagesForUid(appInfo.uid)).length == 1) { // Only one package inside UID means we can fast-path return queryStatsForUid(volumeUuid, appInfo.uid, callingPackage); } else { @@ -276,7 +278,7 @@ public class StorageStatsService extends IStorageStatsManager.Stub { enforcePermission(Binder.getCallingUid(), callingPackage); } - final String[] packageNames = mPackage.getPackagesForUid(uid); + final String[] packageNames = defeatNullable(mPackage.getPackagesForUid(uid)); final long[] ceDataInodes = new long[packageNames.length]; String[] codePaths = new String[0]; |