commit 9a0b289f060e68e42d3db0b843d55cf920f3da12
author Adam Lesinski <adamlesinski@google.com> Mon Aug 11 15:03:32 2014 -0700
committer Adam Lesinski <adamlesinski@google.com> Tue Aug 12 20:25:37 2014 +0000
tree d4859fad956007459e0263aa9045b870ade4609c
parent 02b25e9ba6b5c15c3af08bf74ad70519fcbcd8ce

Fix NPE in UsageStats for new User

Bug:16946585

Change-Id: Ibd0d674681f3f49087305cea33ce7292f5d2229d

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

diff --git a/services/usage/java/com/android/server/usage/UsageStatsService.java b/services/usage/java/com/android/server/usage/UsageStatsService.java
index c38391a..82e837d 100644
--- a/services/usage/java/com/android/server/usage/UsageStatsService.java
+++ b/services/usage/java/com/android/server/usage/UsageStatsService.java
@@ -311,11 +311,15 @@
             final int userId = UserHandle.getCallingUserId();
             final long token = Binder.clearCallingIdentity();
             try {
-                return new ParceledListSlice<>(UsageStatsService.this.queryUsageStats(
-                        userId, bucketType, beginTime, endTime));
+                final List<UsageStats> results = UsageStatsService.this.queryUsageStats(
+                        userId, bucketType, beginTime, endTime);
+                if (results != null) {
+                    return new ParceledListSlice<>(results);
+                }
             } finally {
                 Binder.restoreCallingIdentity(token);
             }
+            return null;
         }
 
         @Override

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

diff --git a/services/usage/java/com/android/server/usage/UserUsageStatsService.java b/services/usage/java/com/android/server/usage/UserUsageStatsService.java
index 2dfd0f6..e489fb3 100644
--- a/services/usage/java/com/android/server/usage/UserUsageStatsService.java
+++ b/services/usage/java/com/android/server/usage/UserUsageStatsService.java
@@ -181,7 +181,7 @@
 
         final List<UsageStats> results = mDatabase.queryUsageStats(bucketType, beginTime, endTime);
         if (DEBUG) {
-            Slog.d(TAG, mLogPrefix + "Results: " + results.size());
+            Slog.d(TAG, mLogPrefix + "Results: " + (results == null ? 0 : results.size()));
         }
         return results;
     }