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

Commit e2c1f9b7 authored by Wang Le's avatar Wang Le Committed by Amith Yamasani
Browse files

Fix the issue that App crash on startForeground 

Install single APP in any user space except Owner user, the APP
got crash when call startForeground in Service Component with
Notification which without SmallIcon.

External bug:https://code.google.com/p/android/issues/detail?id=209043


Internal bug: 28602381

Change-Id: I88a3b8afc213cafe0f280f9b44d6acd2ea0df207
Signed-off-by: default avatarWang Le <wangle@xiaomi.com>
parent f28b2341

services/core/java/com/android/server/am/ServiceRecord.java

+3 −3
Original line number Diff line number Diff line
@@ -441,7 +441,7 @@ final class ServiceRecord extends Binder { 
                    Notification localForegroundNoti = _foregroundNoti;

                    try {

                        if (localForegroundNoti.getSmallIcon() == null) {

                            // It is not correct for the caller to supply a notification

                            // It is not correct for the caller to not supply a notification

                            // icon, but this used to be able to slip through, so for

                            // those dirty apps we will create a notification clearly

                            // blaming the app.
@@ -458,8 +458,8 @@ final class ServiceRecord extends Binder { 
                            }

                            Context ctx = null;

                            try {

                                ctx = ams.mContext.createPackageContext(

                                        appInfo.packageName, 0);

                                ctx = ams.mContext.createPackageContextAsUser(

                                        appInfo.packageName, 0, new UserHandle(userId));



                                Notification.Builder notiBuilder = new Notification.Builder(ctx);