diff options
author | Scott Lobdell <slobdell@google.com> | 2021-04-10 00:27:34 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2021-04-10 00:27:34 +0000 |
commit | 01da99e3e562d93c902419316b2f6aa464777e16 (patch) | |
tree | 2fe6d242f8218ab669e2a98f757a85005ff88088 /keystore/java/android/security/LegacyVpnProfileStore.java | |
parent | 80bf29781b44a4c22692a3a1a1e60a371471c25a (diff) | |
parent | 6560a5b465795d6922ccd0c83693fa04be2910a6 (diff) |
Merge changes from topic "SP1A.210407.002" into s-keystone-qcom-dev
* changes:
Adapt to new ranking logic from upstream.
Revert "Initial data pipeline for Smartspace media recommendations data in media carousel."
Merge SP1A.210407.002
Diffstat (limited to 'keystore/java/android/security/LegacyVpnProfileStore.java')
-rw-r--r-- | keystore/java/android/security/LegacyVpnProfileStore.java | 37 |
1 files changed, 9 insertions, 28 deletions
diff --git a/keystore/java/android/security/LegacyVpnProfileStore.java b/keystore/java/android/security/LegacyVpnProfileStore.java index 41cfb2707fcf..1d2738e71311 100644 --- a/keystore/java/android/security/LegacyVpnProfileStore.java +++ b/keystore/java/android/security/LegacyVpnProfileStore.java @@ -19,7 +19,6 @@ package android.security; import android.annotation.NonNull; import android.os.ServiceManager; import android.os.ServiceSpecificException; -import android.security.keystore.AndroidKeyStoreProvider; import android.security.vpnprofilestore.IVpnProfileStore; import android.util.Log; @@ -53,13 +52,8 @@ public class LegacyVpnProfileStore { */ public static boolean put(@NonNull String alias, @NonNull byte[] profile) { try { - if (AndroidKeyStoreProvider.isKeystore2Enabled()) { - getService().put(alias, profile); - return true; - } else { - return KeyStore.getInstance().put( - alias, profile, KeyStore.UID_SELF, 0); - } + getService().put(alias, profile); + return true; } catch (Exception e) { Log.e(TAG, "Failed to put vpn profile.", e); return false; @@ -77,11 +71,7 @@ public class LegacyVpnProfileStore { */ public static byte[] get(@NonNull String alias) { try { - if (AndroidKeyStoreProvider.isKeystore2Enabled()) { - return getService().get(alias); - } else { - return KeyStore.getInstance().get(alias, true /* suppressKeyNotFoundWarning */); - } + return getService().get(alias); } catch (ServiceSpecificException e) { if (e.errorCode != PROFILE_NOT_FOUND) { Log.e(TAG, "Failed to get vpn profile.", e); @@ -100,12 +90,8 @@ public class LegacyVpnProfileStore { */ public static boolean remove(@NonNull String alias) { try { - if (AndroidKeyStoreProvider.isKeystore2Enabled()) { - getService().remove(alias); - return true; - } else { - return KeyStore.getInstance().delete(alias); - } + getService().remove(alias); + return true; } catch (ServiceSpecificException e) { if (e.errorCode != PROFILE_NOT_FOUND) { Log.e(TAG, "Failed to remove vpn profile.", e); @@ -124,16 +110,11 @@ public class LegacyVpnProfileStore { */ public static @NonNull String[] list(@NonNull String prefix) { try { - if (AndroidKeyStoreProvider.isKeystore2Enabled()) { - final String[] aliases = getService().list(prefix); - for (int i = 0; i < aliases.length; ++i) { - aliases[i] = aliases[i].substring(prefix.length()); - } - return aliases; - } else { - final String[] result = KeyStore.getInstance().list(prefix); - return result != null ? result : new String[0]; + final String[] aliases = getService().list(prefix); + for (int i = 0; i < aliases.length; ++i) { + aliases[i] = aliases[i].substring(prefix.length()); } + return aliases; } catch (Exception e) { Log.e(TAG, "Failed to list vpn profiles.", e); } |