From 06433bd9f354c27b9adcda2c07e05bcca6e2ad0d Mon Sep 17 00:00:00 2001 From: Dan Sandler Date: Wed, 25 Feb 2015 13:20:39 -0500 Subject: [PATCH] Avoid crashing on corrupted notifications. If a notification content view has an @id/icon or @id/right_icon, we need to make sure those ImageViews actually have a drawable inside. Bug: 19487078 Change-Id: I4ef6c4a10926823aa8680628097ed150d9e717d4 --- .../systemui/statusbar/NotificationTemplateViewWrapper.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/NotificationTemplateViewWrapper.java b/packages/SystemUI/src/com/android/systemui/statusbar/NotificationTemplateViewWrapper.java index 59b62e50c1e1f..57d162beb224f 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/NotificationTemplateViewWrapper.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/NotificationTemplateViewWrapper.java @@ -79,7 +79,8 @@ public class NotificationTemplateViewWrapper extends NotificationViewWrapper { // If the icon already has a color filter, we assume that we already forced the icon to be // white when we created the notification. - mIconForceGraysaleWhenDark = mIcon != null && mIcon.getDrawable().getColorFilter() != null; + final Drawable iconDrawable = mIcon != null ? mIcon.getDrawable() : null; + mIconForceGraysaleWhenDark = iconDrawable != null && iconDrawable.getColorFilter() != null; } private ImageView resolveIcon(ImageView largeIcon, ImageView rightIcon) {