From 85bdb47bcb483ea48cbbca445bfa6fe35b7c9c12 Mon Sep 17 00:00:00 2001 From: Song Pan Date: Tue, 25 Feb 2020 13:46:20 +0000 Subject: [PATCH] Catch all exception from statusReceiver.sendIntent. Any exception here (e.g., NPE) will crash the entire system process. Bug: 150217444 Test: atest AppIntegrityManagerServiceImplTest Change-Id: I86eaa0bf7cb9d39fb23ee6aa71e80547bb95cc7e (cherry picked from commit 1787f310655e36f11efd424f1d847e83208d64d8) --- .../server/integrity/AppIntegrityManagerServiceImpl.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/services/core/java/com/android/server/integrity/AppIntegrityManagerServiceImpl.java b/services/core/java/com/android/server/integrity/AppIntegrityManagerServiceImpl.java index 63054cf0c5168..fd8e159bf7912 100644 --- a/services/core/java/com/android/server/integrity/AppIntegrityManagerServiceImpl.java +++ b/services/core/java/com/android/server/integrity/AppIntegrityManagerServiceImpl.java @@ -40,7 +40,6 @@ import android.content.integrity.Rule; import android.content.pm.PackageInfo; import android.content.pm.PackageManager; import android.content.pm.PackageManagerInternal; -import android.content.pm.PackageParser; import android.content.pm.PackageUserState; import android.content.pm.ParceledListSlice; import android.content.pm.Signature; @@ -202,7 +201,7 @@ public class AppIntegrityManagerServiceImpl extends IAppIntegrityManager.Stub { intent, /* onFinished= */ null, /* handler= */ null); - } catch (IntentSender.SendIntentException e) { + } catch (Exception e) { Slog.e(TAG, "Error sending status feedback.", e); } });