Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "maybePruneOldTraces: bail if modification times change." am: 9144eaf12f am: 72d2a3f7cc am: 69de46ba86

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1712554

Change-Id: I783858d1215dae38814d67dc9b7c88233a9c6d91
This commit is contained in:
Elliott Hughes
2021-05-21 17:20:20 +00:00
committed by Automerger Merge Worker
parent 62aa20e48b 69de46ba86
commit f83bdfe9c2
1 changed files with 11 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
services/core/java/com/android/server/am/ActivityManagerService.java
View File

@@ -3287,13 +3287,19 @@ public class ActivityManagerService extends IActivityManager.Stub
final int max = SystemProperties.getInt("tombstoned.max_anr_count", 64); final int max = SystemProperties.getInt("tombstoned.max_anr_count", 64);
final long now = System.currentTimeMillis(); final long now = System.currentTimeMillis();
Arrays.sort(files, Comparator.comparingLong(File::lastModified).reversed()); try {
for (int i = 0; i < files.length; ++i) { Arrays.sort(files, Comparator.comparingLong(File::lastModified).reversed());
if (i > max || (now - files[i].lastModified()) > DAY_IN_MILLIS) { for (int i = 0; i < files.length; ++i) {
if (!files[i].delete()) { if (i > max || (now - files[i].lastModified()) > DAY_IN_MILLIS) {
Slog.w(TAG, "Unable to prune stale trace file: " + files[i]); if (!files[i].delete()) {
Slog.w(TAG, "Unable to prune stale trace file: " + files[i]);
}
} }
} }
} catch (IllegalArgumentException e) {
// The modification times changed while we were sorting. Bail...
// https://issuetracker.google.com/169836837
Slog.w(TAG, "tombstone modification times changed while sorting; not pruning", e);
} }
} }