Snap for 10919181 from abbd1b917ef56ce12d351f51f3b636852b8b825d to udc-qpr1-release (8d233d41) · Commits · e / os / android_frameworks_base

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

+20 −9

Original line number	Diff line number	Diff line
		@@ -285,6 +285,8 @@ public class UserManagerService extends IUserManager.Stub {

		private static final int USER_VERSION = 11;

		private static final int MAX_USER_STRING_LENGTH = 500;

		private static final long EPOCH_PLUS_30_YEARS = 30L * 365 * 24 * 60 * 60 * 1000L; // ms

		static final int WRITE_USER_MSG = 1;
		@@ -4258,15 +4260,17 @@ public class UserManagerService extends IUserManager.Stub {
		// Write seed data
		if (userData.persistSeedData) {
		if (userData.seedAccountName != null) {
		serializer.attribute(null, ATTR_SEED_ACCOUNT_NAME, userData.seedAccountName);
		serializer.attribute(null, ATTR_SEED_ACCOUNT_NAME,
		truncateString(userData.seedAccountName));
		}
		if (userData.seedAccountType != null) {
		serializer.attribute(null, ATTR_SEED_ACCOUNT_TYPE, userData.seedAccountType);
		serializer.attribute(null, ATTR_SEED_ACCOUNT_TYPE,
		truncateString(userData.seedAccountType));
		}
		}
		if (userInfo.name != null) {
		serializer.startTag(null, TAG_NAME);
		serializer.text(userInfo.name);
		serializer.text(truncateString(userInfo.name));
		serializer.endTag(null, TAG_NAME);
		}
		synchronized (mRestrictionsLock) {
		@@ -4315,6 +4319,13 @@ public class UserManagerService extends IUserManager.Stub {
		serializer.endDocument();
		}

		private String truncateString(String original) {
		if (original == null \|\| original.length() <= MAX_USER_STRING_LENGTH) {
		return original;
		}
		return original.substring(0, MAX_USER_STRING_LENGTH);
		}

		/*
		* Writes the user list file in this format:
		*
		@@ -4750,7 +4761,7 @@ public class UserManagerService extends IUserManager.Stub {
		@UserIdInt int parentId, boolean preCreate, @Nullable String[] disallowedPackages,
		@NonNull TimingsTraceAndSlog t, @Nullable Object token)
		throws UserManager.CheckedUserOperationException {

		String truncatedName = truncateString(name);
		final UserTypeDetails userTypeDetails = mUserTypes.get(userType);
		if (userTypeDetails == null) {
		throwCheckedUserOperationException(
		@@ -4785,8 +4796,8 @@ public class UserManagerService extends IUserManager.Stub {

		// Try to use a pre-created user (if available).
		if (!preCreate && parentId < 0 && isUserTypeEligibleForPreCreation(userTypeDetails)) {
		final UserInfo preCreatedUser = convertPreCreatedUserIfPossible(userType, flags, name,
		token);
		final UserInfo preCreatedUser = convertPreCreatedUserIfPossible(userType, flags,
		truncatedName, token);
		if (preCreatedUser != null) {
		return preCreatedUser;
		}
		@@ -4873,7 +4884,7 @@ public class UserManagerService extends IUserManager.Stub {
		flags \|= UserInfo.FLAG_EPHEMERAL_ON_CREATE;
		}

		userInfo = new UserInfo(userId, name, null, flags, userType);
		userInfo = new UserInfo(userId, truncatedName, null, flags, userType);
		userInfo.serialNumber = mNextSerialNumber++;
		userInfo.creationTime = getCreationTime();
		userInfo.partial = true;
		@@ -6357,8 +6368,8 @@ public class UserManagerService extends IUserManager.Stub {
		Slog.e(LOG_TAG, "No such user for settings seed data u=" + userId);
		return;
		}
		userData.seedAccountName = accountName;
		userData.seedAccountType = accountType;
		userData.seedAccountName = truncateString(accountName);
		userData.seedAccountType = truncateString(accountType);
		userData.seedAccountOptions = accountOptions;
		userData.persistSeedData = persist;
		}

services/tests/mockingservicestests/src/com/android/server/pm/UserManagerServiceTest.java

+21 −0

Original line number	Diff line number	Diff line
		@@ -15,9 +15,12 @@
		*/
		package com.android.server.pm;

		import static android.os.UserManager.USER_TYPE_FULL_SECONDARY;

		import static com.android.dx.mockito.inline.extended.ExtendedMockito.doNothing;
		import static com.android.dx.mockito.inline.extended.ExtendedMockito.doReturn;

		import static com.google.common.truth.Truth.assertThat;
		import static com.google.common.truth.Truth.assertWithMessage;

		import static org.junit.Assert.assertThrows;
		@@ -402,6 +405,24 @@ public final class UserManagerServiceTest {
		.isEqualTo(USER_ID);
		}

		@Test
		public void testCreateUserWithLongName_TruncatesName() {
		UserInfo user = mUms.createUserWithThrow(generateLongString(), USER_TYPE_FULL_SECONDARY, 0);
		assertThat(user.name.length()).isEqualTo(500);
		UserInfo user1 = mUms.createUserWithThrow("Test", USER_TYPE_FULL_SECONDARY, 0);
		assertThat(user1.name.length()).isEqualTo(4);
		}

		private String generateLongString() {
		String partialString = "Test Name Test Name Test Name Test Name Test Name Test Name Test "
		+ "Name Test Name Test Name Test Name "; //String of length 100
		StringBuilder resultString = new StringBuilder();
		for (int i = 0; i < 660; i++) {
		resultString.append(partialString);
		}
		return resultString.toString();
		}

		private void mockCurrentUser(@UserIdInt int userId) {
		mockGetLocalService(ActivityManagerInternal.class, mActivityManagerInternal);