2017-04-06 08:16:33

by Herbert Xu

Subject: [PATCH 4/4] crypto: api - Extend algorithm name limit to 128 bytes

With the new explicit IV generators, we may now exceed the 64-byte
length limit on the algorithm name, e.g., with

echainiv(authencesn(hmac(sha256-generic),cbc(des3_ede-generic)))

This patch extends the length limit to 128 bytes.

Reported-by: Alexander Sverdlin <[email protected]>
Signed-off-by: Herbert Xu <[email protected]>
---

include/linux/crypto.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/crypto.h b/include/linux/crypto.h
index c0b0cf3..84da997 100644
--- a/include/linux/crypto.h
+++ b/include/linux/crypto.h
@@ -123,7 +123,7 @@
/*
* Miscellaneous stuff.
*/
-#define CRYPTO_MAX_ALG_NAME 64
+#define CRYPTO_MAX_ALG_NAME 128

/*
* The macro CRYPTO_MINALIGN_ATTR (along with the void * type in the actual

2017-04-06 15:11:51

by Alexander Sverdlin

Subject: Re: [PATCH 4/4] crypto: api - Extend algorithm name limit to 128 bytes

On 06/04/17 10:16, Herbert Xu wrote:
> With the new explicit IV generators, we may now exceed the 64-byte
> length limit on the algorithm name, e.g., with
>
> echainiv(authencesn(hmac(sha256-generic),cbc(des3_ede-generic)))
>
> This patch extends the length limit to 128 bytes.
>
> Reported-by: Alexander Sverdlin <[email protected]>
> Signed-off-by: Herbert Xu <[email protected]>

Acked-by: Alexander Sverdlin <[email protected]>
Tested-by: Alexander Sverdlin <[email protected]>

> ---
>
> include/linux/crypto.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/crypto.h b/include/linux/crypto.h
> index c0b0cf3..84da997 100644
> --- a/include/linux/crypto.h
> +++ b/include/linux/crypto.h
> @@ -123,7 +123,7 @@
> /*
> * Miscellaneous stuff.
> */
> -#define CRYPTO_MAX_ALG_NAME 64
> +#define CRYPTO_MAX_ALG_NAME 128
>
> /*
> * The macro CRYPTO_MINALIGN_ATTR (along with the void * type in the actual
>

--
Best regards,
Alexander Sverdlin.