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

Commit 2da77695 authored by Automerger Merge Worker's avatar Automerger Merge Worker
Browse files

Merge "Do not allow -1 duration in setUserAuthenticationParameters" into rvc-dev am: 0ffc3165 

Change-Id: I921ff74f052a08477f9e553f228e50f2160c1ef7
parents 5dbf7b13 0ffc3165

api/current.txt

+2 −2
Original line number Diff line number Diff line
@@ -42787,7 +42787,7 @@ package android.security.keystore { 
    method @NonNull public android.security.keystore.KeyGenParameterSpec.Builder setRandomizedEncryptionRequired(boolean);

    method @NonNull public android.security.keystore.KeyGenParameterSpec.Builder setSignaturePaddings(java.lang.String...);

    method @NonNull public android.security.keystore.KeyGenParameterSpec.Builder setUnlockedDeviceRequired(boolean);

    method @NonNull public android.security.keystore.KeyGenParameterSpec.Builder setUserAuthenticationParameters(@IntRange(from=0xffffffff) int, int);

    method @NonNull public android.security.keystore.KeyGenParameterSpec.Builder setUserAuthenticationParameters(@IntRange(from=0) int, int);

    method @NonNull public android.security.keystore.KeyGenParameterSpec.Builder setUserAuthenticationRequired(boolean);

    method @NonNull public android.security.keystore.KeyGenParameterSpec.Builder setUserAuthenticationValidWhileOnBody(boolean);

    method @Deprecated @NonNull public android.security.keystore.KeyGenParameterSpec.Builder setUserAuthenticationValidityDurationSeconds(@IntRange(from=0xffffffff) int);
@@ -42905,7 +42905,7 @@ package android.security.keystore { 
    method @NonNull public android.security.keystore.KeyProtection.Builder setRandomizedEncryptionRequired(boolean);

    method @NonNull public android.security.keystore.KeyProtection.Builder setSignaturePaddings(java.lang.String...);

    method @NonNull public android.security.keystore.KeyProtection.Builder setUnlockedDeviceRequired(boolean);

    method @NonNull public android.security.keystore.KeyProtection.Builder setUserAuthenticationParameters(@IntRange(from=0xffffffff) int, int);

    method @NonNull public android.security.keystore.KeyProtection.Builder setUserAuthenticationParameters(@IntRange(from=0) int, int);

    method @NonNull public android.security.keystore.KeyProtection.Builder setUserAuthenticationRequired(boolean);

    method @NonNull public android.security.keystore.KeyProtection.Builder setUserAuthenticationValidWhileOnBody(boolean);

    method @Deprecated @NonNull public android.security.keystore.KeyProtection.Builder setUserAuthenticationValidityDurationSeconds(@IntRange(from=0xffffffff) int);

keystore/java/android/security/keystore/KeyGenParameterSpec.java

+4 −7
Original line number Diff line number Diff line
@@ -1264,8 +1264,7 @@ public final class KeyGenParameterSpec implements AlgorithmParameterSpec, UserAu 
         * successfully.

         *

         * @param timeout duration in seconds or {@code 0} if user authentication must take place

         *        for every use of the key. {@code -1} is also accepted for legacy purposes. It is

         *        functionally the same as {@code 0}.

         *        for every use of the key.

         * @param type set of authentication types which can authorize use of the key. See

         *        {@link KeyProperties}.{@code AUTH} flags.

         *
@@ -1275,12 +1274,10 @@ public final class KeyGenParameterSpec implements AlgorithmParameterSpec, UserAu 
         * @see KeyguardManager

         */

        @NonNull

        public Builder setUserAuthenticationParameters(@IntRange(from = -1) int timeout,

        public Builder setUserAuthenticationParameters(@IntRange(from = 0) int timeout,

                                                       @KeyProperties.AuthEnum int type) {

            if (timeout < -1) {

                throw new IllegalArgumentException("timeout must be -1 or larger");

            } else if (timeout == -1) {

                timeout = 0;

            if (timeout < 0) {

                throw new IllegalArgumentException("timeout must be 0 or larger");

            }

            mUserAuthenticationValidityDurationSeconds = timeout;

            mUserAuthenticationType = type;

keystore/java/android/security/keystore/KeyProtection.java

+4 −7
Original line number Diff line number Diff line
@@ -894,8 +894,7 @@ public final class KeyProtection implements ProtectionParameter, UserAuthArgs { 
         * successfully.

         *

         * @param timeout duration in seconds or {@code 0} if user authentication must take place

         *        for every use of the key. {@code -1} is also accepted for legacy purposes. It is

         *        functionally the same as {@code 0}.

         *        for every use of the key.

         * @param type set of authentication types which can authorize use of the key. See

         *        {@link KeyProperties}.{@code AUTH} flags.

         *
@@ -905,12 +904,10 @@ public final class KeyProtection implements ProtectionParameter, UserAuthArgs { 
         * @see KeyguardManager

         */

        @NonNull

        public Builder setUserAuthenticationParameters(@IntRange(from = -1) int timeout,

        public Builder setUserAuthenticationParameters(@IntRange(from = 0) int timeout,

                                                       @KeyProperties.AuthEnum int type) {

            if (timeout < -1) {

                throw new IllegalArgumentException("timeout must be -1 or larger");

            } else if (timeout == -1) {

                timeout = 0;

            if (timeout < 0) {

                throw new IllegalArgumentException("timeout must be 0 or larger");

            }

            mUserAuthenticationValidityDurationSeconds = timeout;

            mUserAuthenticationType = type;