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

Merge "Prevent crash in AccountManager after profile removal."

Browse Source
This commit is contained in:
Dmitry Dementyev
2023-06-29 17:56:11 +00:00
committed by Gerrit Code Review
parent a680cb9cf8 486de90adb
commit 6cbae2d6d0
1 changed files with 15 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
services/core/java/com/android/server/accounts/AccountManagerService.java
View File

@@ -67,6 +67,7 @@ import android.content.pm.Signature;
import android.content.pm.SigningDetails.CertCapabilities; import android.content.pm.SigningDetails.CertCapabilities;
import android.content.pm.UserInfo; import android.content.pm.UserInfo;
import android.database.Cursor; import android.database.Cursor;
import android.database.sqlite.SQLiteCantOpenDatabaseException;
import android.database.sqlite.SQLiteFullException; import android.database.sqlite.SQLiteFullException;
import android.database.sqlite.SQLiteStatement; import android.database.sqlite.SQLiteStatement;
import android.os.Binder; import android.os.Binder;
@@ -1383,7 +1384,13 @@ public class AccountManagerService
private void purgeOldGrants(UserAccounts accounts) { private void purgeOldGrants(UserAccounts accounts) {
synchronized (accounts.dbLock) { synchronized (accounts.dbLock) {
synchronized (accounts.cacheLock) { synchronized (accounts.cacheLock) {
List<Integer> uids = accounts.accountsDb.findAllUidGrants(); List<Integer> uids;
try {
uids = accounts.accountsDb.findAllUidGrants();
} catch (SQLiteCantOpenDatabaseException e) {
Log.w(TAG, "Could not delete grants for user = " + accounts.userId);
return;
}
for (int uid : uids) { for (int uid : uids) {
final boolean packageExists = mPackageManager.getPackagesForUid(uid) != null; final boolean packageExists = mPackageManager.getPackagesForUid(uid) != null;
if (packageExists) { if (packageExists) {
@@ -1409,7 +1416,13 @@ public class AccountManagerService
mPackageManager.getPackageUidAsUser(packageName, accounts.userId); mPackageManager.getPackageUidAsUser(packageName, accounts.userId);
} catch (NameNotFoundException e) { } catch (NameNotFoundException e) {
// package does not exist - remove visibility values // package does not exist - remove visibility values
accounts.accountsDb.deleteAccountVisibilityForPackage(packageName); try {
accounts.accountsDb.deleteAccountVisibilityForPackage(packageName);
} catch (SQLiteCantOpenDatabaseException sqlException) {
Log.w(TAG, "Could not delete account visibility for user = "
+ accounts.userId, sqlException);
continue;
}
synchronized (accounts.dbLock) { synchronized (accounts.dbLock) {
synchronized (accounts.cacheLock) { synchronized (accounts.cacheLock) {
for (Account account : accounts.visibilityCache.keySet()) { for (Account account : accounts.visibilityCache.keySet()) {