Skip to content

Commit 0cd867c

Browse files
author
Amith Yamasani
committed
Fix deadlock in removing a user
Cleanup PackageManager settings after the package directories are removed. Change-Id: I264bc95ea21da653a234d7c81df5b25d4de7aa9e
1 parent 7e877fa commit 0cd867c

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

services/java/com/android/server/pm/UserManagerService.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -523,6 +523,10 @@ public boolean removeUser(int userHandle) {
523523
synchronized (mUsers) {
524524
result = removeUserLocked(userHandle);
525525
}
526+
527+
// Cleanup package manager settings
528+
mPm.cleanUpUser(userHandle);
529+
526530
// Let other services shutdown any activity
527531
Intent addedIntent = new Intent(Intent.ACTION_USER_REMOVED);
528532
addedIntent.putExtra(Intent.EXTRA_USER_HANDLE, userHandle);
@@ -555,8 +559,6 @@ private boolean removeUserLocked(int userHandle) {
555559
return false;
556560
}
557561

558-
mPm.cleanUpUser(userHandle);
559-
560562
// Remove this user from the list
561563
mUsers.remove(userHandle);
562564
// Remove user file

0 commit comments

Comments
 (0)