From 2a68761bc9a9d0d16403ecfb2189eb71d3a2697b Mon Sep 17 00:00:00 2001 From: Olivier Gaillard Date: Fri, 24 Feb 2023 16:49:23 +0000 Subject: [PATCH] Do not collect system daemons dumps for non-system apps as this is not likely to be useful. Change-Id: Ia22b6f36402509cd851861f00b136a2bc660c94f Bug: 254634348 Test: unit tests --- .../java/com/android/server/am/ProcessErrorStateRecord.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/services/core/java/com/android/server/am/ProcessErrorStateRecord.java b/services/core/java/com/android/server/am/ProcessErrorStateRecord.java index 58a47d7b1dc46..83caf0f6048a1 100644 --- a/services/core/java/com/android/server/am/ProcessErrorStateRecord.java +++ b/services/core/java/com/android/server/am/ProcessErrorStateRecord.java @@ -459,7 +459,10 @@ class ProcessErrorStateRecord { // don't dump native PIDs for background ANRs unless // it is the process of interest String[] nativeProcs = null; - if (isSilentAnr || onlyDumpSelf) { + boolean isSystemApp = mApp.info.isSystemApp() || mApp.info.isSystemExt(); + // Do not collect system daemons dumps as this is not likely to be useful + // for non-system apps. + if (!isSystemApp || isSilentAnr || onlyDumpSelf) { for (int i = 0; i < NATIVE_STACKS_OF_INTEREST.length; i++) { if (NATIVE_STACKS_OF_INTEREST[i].equals(mApp.processName)) { nativeProcs = new String[] { mApp.processName };