Skip to content

Commit 9a0b289

Browse files
author
Adam Lesinski
committed
Fix NPE in UsageStats for new User
Bug:16946585 Change-Id: Ibd0d674681f3f49087305cea33ce7292f5d2229d
1 parent 02b25e9 commit 9a0b289

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

services/usage/java/com/android/server/usage/UsageStatsService.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -311,11 +311,15 @@ public ParceledListSlice<UsageStats> queryUsageStats(int bucketType, long beginT
311311
final int userId = UserHandle.getCallingUserId();
312312
final long token = Binder.clearCallingIdentity();
313313
try {
314-
return new ParceledListSlice<>(UsageStatsService.this.queryUsageStats(
315-
userId, bucketType, beginTime, endTime));
314+
final List<UsageStats> results = UsageStatsService.this.queryUsageStats(
315+
userId, bucketType, beginTime, endTime);
316+
if (results != null) {
317+
return new ParceledListSlice<>(results);
318+
}
316319
} finally {
317320
Binder.restoreCallingIdentity(token);
318321
}
322+
return null;
319323
}
320324

321325
@Override

services/usage/java/com/android/server/usage/UserUsageStatsService.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ List<UsageStats> queryUsageStats(int bucketType, long beginTime, long endTime) {
181181

182182
final List<UsageStats> results = mDatabase.queryUsageStats(bucketType, beginTime, endTime);
183183
if (DEBUG) {
184-
Slog.d(TAG, mLogPrefix + "Results: " + results.size());
184+
Slog.d(TAG, mLogPrefix + "Results: " + (results == null ? 0 : results.size()));
185185
}
186186
return results;
187187
}

0 commit comments

Comments
 (0)