Skip to content

Commit 9878bb9

Browse files
Christopher TateThe Android Automerger
authored andcommitted
Kill the real/isolated uid group, not the ApplicationInfo uid
This is a direct reimplementation in L of fixes applied to N in these two commits: 8dc8d37 e8741d2 Bug 19285814 Change-Id: I59bcc8f1d41c426e9da635bea9ad1d7c6756d5aa Resolve merge conflict when cp'ing ag/941553 to mnc-mr1-release branch
1 parent 613f63b commit 9878bb9

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

services/core/java/com/android/server/am/ActivityManagerService.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2814,7 +2814,7 @@ final void removeLruProcessLocked(ProcessRecord app) {
28142814
if (!app.killed) {
28152815
Slog.wtfStack(TAG, "Removing process that hasn't been killed: " + app);
28162816
Process.killProcessQuiet(app.pid);
2817-
killProcessGroup(app.info.uid, app.pid);
2817+
killProcessGroup(app.uid, app.pid);
28182818
}
28192819
if (lrui <= mLruProcessActivityStart) {
28202820
mLruProcessActivityStart--;
@@ -3189,7 +3189,7 @@ final ProcessRecord startProcessLocked(String processName, ApplicationInfo info,
31893189
// clean it up now.
31903190
if (DEBUG_PROCESSES || DEBUG_CLEANUP) Slog.v(TAG_PROCESSES, "App died: " + app);
31913191
checkTime(startTime, "startProcess: bad proc running, killing");
3192-
killProcessGroup(app.info.uid, app.pid);
3192+
killProcessGroup(app.uid, app.pid);
31933193
handleAppDiedLocked(app, true, true);
31943194
checkTime(startTime, "startProcess: done killing old proc");
31953195
}
@@ -4705,7 +4705,7 @@ final void appDiedLocked(ProcessRecord app, int pid, IApplicationThread thread,
47054705
if (!fromBinderDied) {
47064706
Process.killProcessQuiet(pid);
47074707
}
4708-
killProcessGroup(app.info.uid, pid);
4708+
killProcessGroup(app.uid, pid);
47094709
app.killed = true;
47104710
}
47114711

services/core/java/com/android/server/am/ProcessRecord.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -548,7 +548,7 @@ void kill(String reason, boolean noisy) {
548548
}
549549
EventLog.writeEvent(EventLogTags.AM_KILL, userId, pid, processName, setAdj, reason);
550550
Process.killProcessQuiet(pid);
551-
Process.killProcessGroup(info.uid, pid);
551+
Process.killProcessGroup(uid, pid);
552552
if (!persistent) {
553553
killed = true;
554554
killedByAm = true;

0 commit comments

Comments
 (0)