drgreen/android_frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Don't crash when component enable/disable broadcasts race with uninstall" into klp-dev

Browse Source
This commit is contained in:
Christopher Tate
2013-10-09 22:18:41 +00:00
committed by Android (Google) Code Review
parent 24305da7d6 06e5fed139
commit d606be2562
1 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
services/java/com/android/server/NotificationManagerService.java
View File

@ -1167,12 +1167,20 @@ public class NotificationManagerService extends INotificationManager.Stub
} }
if (packageChanged) { if (packageChanged) {
// We cancel notifications for packages which have just been disabled // We cancel notifications for packages which have just been disabled
try {
final int enabled = mContext.getPackageManager() final int enabled = mContext.getPackageManager()
.getApplicationEnabledSetting(pkgName); .getApplicationEnabledSetting(pkgName);
if (enabled == PackageManager.COMPONENT_ENABLED_STATE_ENABLED if (enabled == PackageManager.COMPONENT_ENABLED_STATE_ENABLED
|| enabled == PackageManager.COMPONENT_ENABLED_STATE_DEFAULT) { || enabled == PackageManager.COMPONENT_ENABLED_STATE_DEFAULT) {
cancelNotifications = false; cancelNotifications = false;
} }
} catch (IllegalArgumentException e) {
// Package doesn't exist; probably racing with uninstall.
// cancelNotifications is already true, so nothing to do here.
if (DBG) {
Slog.i(TAG, "Exception trying to look up app enabled setting", e);
}
}
} }
pkgList = new String[]{pkgName}; pkgList = new String[]{pkgName};
} }