Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 776c0908 authored by Selim Cinek's avatar Selim Cinek Committed by The Android Automerger
Browse files

Fixed a crash in the notification group manager 

When 2 notifications were posted as the summary of the same
group, then the system would eventually crash.

Bug: 23676310
Change-Id: Ia8f95e624f3f43d1b55169dd8102e3c89427dc76
parent 982f2357

packages/SystemUI/src/com/android/systemui/statusbar/phone/NotificationGroupManager.java

+7 −0
Original line number Diff line number Diff line
@@ -78,6 +78,13 @@ public class NotificationGroupManager { 
        Notification notif = sbn.getNotification();

        String groupKey = sbn.getGroupKey();

        final NotificationGroup group = mGroupMap.get(groupKey);

        if (group == null) {

            // When an app posts 2 different notifications as summary of the same group, then a

            // cancellation of the first notification removes this group.

            // This situation is not supported and we will not allow such notifications anymore in

            // the close future. See b/23676310 for reference.

            return;

        }

        if (notif.isGroupSummary()) {

            group.summary = null;

        } else {