summaryrefslogtreecommitdiffstats
path: root/src/com
diff options
context:
space:
mode:
authorStephen Bird <sbird@cyngn.com>2016-04-22 14:55:59 -0700
committerStephen Bird <sbird@cyngn.com>2016-04-22 16:47:20 -0700
commit6167f428ab08cfd0b7e7307ca5bbf4da8dee54dd (patch)
treec389858580c04f1e5a34d7f057cb6aeb4b8b3282 /src/com
parent416100f010119e244d8cd2aeeffe74821e3d1acc (diff)
downloadandroid_packages_apps_Dialer-6167f428ab08cfd0b7e7307ca5bbf4da8dee54dd.tar.gz
android_packages_apps_Dialer-6167f428ab08cfd0b7e7307ca5bbf4da8dee54dd.tar.bz2
android_packages_apps_Dialer-6167f428ab08cfd0b7e7307ca5bbf4da8dee54dd.zip
NPE: check that cn is null and return
We don't want to store null values in our sharedprefs, and we dont want to crash. Ticket: CD-569 Change-Id: I647bb9bb8861ed4205c11940b2efcda884e49a21
Diffstat (limited to 'src/com')
-rw-r--r--src/com/android/dialer/incall/InCallMetricsHelper.java4
1 files changed, 4 insertions, 0 deletions
diff --git a/src/com/android/dialer/incall/InCallMetricsHelper.java b/src/com/android/dialer/incall/InCallMetricsHelper.java
index 76733726b..2a24ae19a 100644
--- a/src/com/android/dialer/incall/InCallMetricsHelper.java
+++ b/src/com/android/dialer/incall/InCallMetricsHelper.java
@@ -320,6 +320,10 @@ public class InCallMetricsHelper {
*/
public static void increaseCountOfMetric(ComponentName cn, Events event, Categories cat,
Parameters param) {
+ if (cn == null) {
+ // this is only null if we do not have a sim card.
+ return;
+ }
HashMap<Parameters, Object> metricsData = getStoredEventParams(cn, cat, event);
metricsData.put(param, increaseCount(metricsData,param));
storeEvent(cn, cat, event, metricsData);