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

[am] check if codePath is null before checking incremental

Browse Source 
BUG: 187653672
Test: builds
Change-Id: I12eb18e085660a20508a8407b2643ecbcddf1b54
This commit is contained in:
Songchun Fan
2021-05-10 10:10:34 -07:00
parent 88385cc1f0
commit efcd3258e9
2 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -7726,16 +7726,17 @@ public class ActivityManagerService extends IActivityManager.Stub
ApplicationErrorReport.CrashInfo crashInfo) {
float loadingProgress = 1;
IncrementalMetrics incrementalMetrics = null;
// Notify package manager service to possibly update package state
// Obtain Incremental information if available
if (r != null && r.info != null && r.info.packageName != null) {
final String codePath = r.info.getCodePath();
IncrementalStatesInfo incrementalStatesInfo =
mPackageManagerInt.getIncrementalStatesInfo(r.info.packageName, r.uid,
r.userId);
if (incrementalStatesInfo != null) {
loadingProgress = incrementalStatesInfo.getProgress();
}
if (IncrementalManager.isIncrementalPath(codePath)) {
final String codePath = r.info.getCodePath();
if (codePath != null && !codePath.isEmpty()
&& IncrementalManager.isIncrementalPath(codePath)) {
// Report in the main log about the incremental package
Slog.e(TAG, "App crashed on incremental package " + r.info.packageName
+ " which is " + ((int) (loadingProgress * 100)) + "% loaded.");

3
services/core/java/com/android/server/am/ProcessErrorStateRecord.java
View File

@@ -400,7 +400,8 @@ class ProcessErrorStateRecord {
loadingProgress = incrementalStatesInfo.getProgress();
}
final String codePath = mApp.info.getCodePath();
if (IncrementalManager.isIncrementalPath(codePath)) {
if (codePath != null && !codePath.isEmpty()
&& IncrementalManager.isIncrementalPath(codePath)) {
// Report in the main log that the incremental package is still loading
Slog.e(TAG, "App ANR on incremental package " + mApp.info.packageName
+ " which is " + ((int) (loadingProgress * 100)) + "% loaded.");