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

Merge "Fix deleting legacy key blobs"

Browse Source 
am: c24a4b5f44

Change-Id: I7003a474706ce4ad018e9568db6fda448ee378f7
This commit is contained in:
Janis Danisevskis
2018-11-21 14:49:11 -08:00
committed by android-build-merger
parent 7d2ae7480f c24a4b5f44
commit b3c61fac5f
2 changed files with 13 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
keystore/java/android/security/Credentials.java
View File

@@ -282,8 +282,11 @@ public class Credentials {
* Returns {@code true} if the entry no longer exists.
*/
public static boolean deleteUserKeyTypeForAlias(KeyStore keystore, String alias, int uid) {
return keystore.delete(Credentials.USER_PRIVATE_KEY + alias, uid) ||
keystore.delete(Credentials.USER_SECRET_KEY + alias, uid);
int ret = keystore.delete2(Credentials.USER_PRIVATE_KEY + alias, uid);
if (ret == KeyStore.KEY_NOT_FOUND) {
return keystore.delete(Credentials.USER_SECRET_KEY + alias, uid);
}
return ret == KeyStore.NO_ERROR;
}
/**

12
keystore/java/android/security/KeyStore.java
View File

@@ -255,16 +255,20 @@ public class KeyStore {
}
}
public boolean delete(String key, int uid) {
int delete2(String key, int uid) {
try {
int ret = mBinder.del(key, uid);
return (ret == NO_ERROR || ret == KEY_NOT_FOUND);
return mBinder.del(key, uid);
} catch (RemoteException e) {
Log.w(TAG, "Cannot connect to keystore", e);
return false;
return SYSTEM_ERROR;
}
}
public boolean delete(String key, int uid) {
int ret = delete2(key, uid);
return ret == NO_ERROR || ret == KEY_NOT_FOUND;
}
@UnsupportedAppUsage
public boolean delete(String key) {
return delete(key, UID_SELF);