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

Merge "Fix UserController to not throw in stopUser"

Browse Source
This commit is contained in:
Ethan Lee
2022-12-06 17:45:26 +00:00
committed by Android (Google) Code Review
parent 7d056a186c eb90e9679d
commit 7c872db149
2 changed files with 35 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
services/core/java/com/android/server/am/UserController.java
View File

@@ -949,9 +949,8 @@ class UserController implements Handler.Callback {
int stopUser(final int userId, final boolean force, boolean allowDelayedLocking,
final IStopUserCallback stopUserCallback, KeyEvictedCallback keyEvictedCallback) {
checkCallingPermission(INTERACT_ACROSS_USERS_FULL, "stopUser");
if (userId < 0 || userId == UserHandle.USER_SYSTEM) {
throw new IllegalArgumentException("Can't stop system user " + userId);
}
Preconditions.checkArgument(userId >= 0, "Invalid user id %d", userId);
enforceShellRestriction(UserManager.DISALLOW_DEBUGGING_FEATURES, userId);
synchronized (mLock) {
return stopUsersLU(userId, force, allowDelayedLocking, stopUserCallback,

33
services/tests/servicestests/src/com/android/server/am/UserControllerTest.java
View File

@@ -638,6 +638,39 @@ public class UserControllerTest {
/* keyEvictedCallback= */ mKeyEvictedCallback, /* expectLocking= */ true);
}
@Test
public void testStopUser_invalidUser() {
int userId = -1;
assertThrows(IllegalArgumentException.class,
() -> mUserController.stopUser(userId, /* force= */ true,
/* allowDelayedLocking= */ true, /* stopUserCallback= */ null,
/* keyEvictedCallback= */ null));
}
@Test
public void testStopUser_systemUser() {
int userId = UserHandle.USER_SYSTEM;
int r = mUserController.stopUser(userId, /* force= */ true,
/* allowDelayedLocking= */ true, /* stopUserCallback= */ null,
/* keyEvictedCallback= */ null);
assertThat(r).isEqualTo(ActivityManager.USER_OP_ERROR_IS_SYSTEM);
}
@Test
public void testStopUser_currentUser() {
setUpUser(TEST_USER_ID1, /* flags= */ 0);
mUserController.startUser(TEST_USER_ID1, /* foreground= */ true);
int r = mUserController.stopUser(TEST_USER_ID1, /* force= */ true,
/* allowDelayedLocking= */ true, /* stopUserCallback= */ null,
/* keyEvictedCallback= */ null);
assertThat(r).isEqualTo(ActivityManager.USER_OP_IS_CURRENT);
}
/**
* Test conditional delayed locking with mDelayUserDataLocking true.
*/