diff options
| author | TreeHugger Robot <treehugger-gerrit@google.com> | 2017-04-18 23:32:02 +0000 |
|---|---|---|
| committer | Android (Google) Code Review <android-gerrit@google.com> | 2017-04-18 23:32:09 +0000 |
| commit | ddf1d16504d60be6deb8d130e920b0709aa331b5 (patch) | |
| tree | 74ea26f22d21f462a50d410f04da31158fe12c5f /core/java/android | |
| parent | 944782fac265834242db046a255364ca7dedfd5e (diff) | |
| parent | 572c5df5783bfbe2828180fcbabd97bea4c9188e (diff) | |
Merge "guard against nulls in LogMaker" into oc-dev
Diffstat (limited to 'core/java/android')
| -rw-r--r-- | core/java/android/metrics/LogMaker.java | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/core/java/android/metrics/LogMaker.java b/core/java/android/metrics/LogMaker.java index a76a8a0e3da2..3c6baa76558a 100644 --- a/core/java/android/metrics/LogMaker.java +++ b/core/java/android/metrics/LogMaker.java @@ -54,7 +54,11 @@ public class LogMaker { /* Deserialize from the eventlog */ public LogMaker(Object[] items) { - deserialize(items); + if (items != null) { + deserialize(items); + } else { + setCategory(MetricsEvent.VIEW_UNKNOWN); + } } /** @param category to replace the existing setting. */ @@ -373,13 +377,13 @@ public class LogMaker { */ public void deserialize(Object[] items) { int i = 0; - while (i < items.length) { + while (items != null && i < items.length) { Object key = items[i++]; Object value = i < items.length ? items[i++] : null; if (key instanceof Integer) { entries.put((Integer) key, value); } else { - Log.i(TAG, "Invalid key " + key.toString()); + Log.i(TAG, "Invalid key " + (key == null ? "null" : key.toString())); } } } |
