Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ea97f890 authored by Dmitry Dementyev's avatar Dmitry Dementyev Committed by Android (Google) Code Review
Browse files

Merge "Add more NonNull annotations to RecoveryController API" into pi-dev

parents a68dfc3c 0bbaf189
Loading
Loading
Loading
Loading
+4 −4
Original line number Diff line number Diff line
@@ -496,7 +496,7 @@ public class RecoveryController {
     *     screen is required to generate recoverable keys.
     */
    @RequiresPermission(android.Manifest.permission.RECOVER_KEYSTORE)
    public Key generateKey(@NonNull String alias) throws InternalRecoveryServiceException,
    public @NonNull Key generateKey(@NonNull String alias) throws InternalRecoveryServiceException,
            LockScreenRequiredException {
        try {
            String grantAlias = mBinder.generateKey(alias);
@@ -527,7 +527,7 @@ public class RecoveryController {
     *
     */
    @RequiresPermission(android.Manifest.permission.RECOVER_KEYSTORE)
    public Key importKey(@NonNull String alias, byte[] keyBytes)
    public @NonNull Key importKey(@NonNull String alias, @NonNull byte[] keyBytes)
            throws InternalRecoveryServiceException, LockScreenRequiredException {
        try {
            String grantAlias = mBinder.importKey(alias, keyBytes);
@@ -575,7 +575,7 @@ public class RecoveryController {
    /**
     * Returns the key with the given {@code grantAlias}.
     */
    Key getKeyFromGrant(String grantAlias) throws UnrecoverableKeyException {
    @NonNull Key getKeyFromGrant(@NonNull String grantAlias) throws UnrecoverableKeyException {
        return AndroidKeyStoreProvider.loadAndroidKeyStoreKeyFromKeystore(
                mKeyStore,
                grantAlias,
@@ -611,7 +611,7 @@ public class RecoveryController {
    }

    @RequiresPermission(android.Manifest.permission.RECOVER_KEYSTORE)
    public Map<String, X509Certificate> getRootCertificates() {
    public @NonNull Map<String, X509Certificate> getRootCertificates() {
        return TrustedRootCertificates.getRootCertificates();
    }

+1 −1
Original line number Diff line number Diff line
@@ -257,7 +257,7 @@ public class RecoverySession implements AutoCloseable {
    }

    /** Given a map from alias to grant alias, returns a map from alias to a {@link Key} handle. */
    private @NonNull Map<String, Key> getKeysFromGrants(Map<String, String> grantAliases)
    private @NonNull Map<String, Key> getKeysFromGrants(@NonNull Map<String, String> grantAliases)
            throws InternalRecoveryServiceException {
        ArrayMap<String, Key> keysByAlias = new ArrayMap<>(grantAliases.size());
        for (String alias : grantAliases.keySet()) {