summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNarayan Kamath <narayan@google.com>2014-12-10 10:00:14 +0000
committerNarayan Kamath <narayan@google.com>2014-12-10 10:01:44 +0000
commitc46f75ef1d479f7b606ab04283b1b0f84849476a (patch)
tree523e665dd1117dadd7d60fbc0ccb5a2a4574d989
parentfb5c932dbbe2e7fc3ed4f0af71dc8ac76f22df57 (diff)
downloadbase-c46f75ef1d479f7b606ab04283b1b0f84849476a.tar.gz
Avoid tearing down process groups twice.
The second removal always triggers a warning. Change-Id: Ieafa8374dbffdaff0d576de004a688cae6f8ac18
-rwxr-xr-xservices/core/java/com/android/server/am/ActivityManagerService.java8
1 files changed, 5 insertions, 3 deletions
diff --git a/services/core/java/com/android/server/am/ActivityManagerService.java b/services/core/java/com/android/server/am/ActivityManagerService.java
index 79f2e7c1c0f4..e3f7fb3c9120 100755
--- a/services/core/java/com/android/server/am/ActivityManagerService.java
+++ b/services/core/java/com/android/server/am/ActivityManagerService.java
@@ -4874,9 +4874,11 @@ public final class ActivityManagerService extends ActivityManagerNative
stats.noteProcessDiedLocked(app.info.uid, pid);
}
- Process.killProcessQuiet(pid);
- Process.killProcessGroup(app.info.uid, pid);
- app.killed = true;
+ if (!app.killed) {
+ Process.killProcessQuiet(pid);
+ Process.killProcessGroup(app.info.uid, pid);
+ app.killed = true;
+ }
// Clean up already done if the process has been re-started.
if (app.pid == pid && app.thread != null &&