12345678910111213141516171819202122232425262728293031323334 |
- From 1ad0f1603a6b2afb62a1c065409aaa4e43ca7627 Mon Sep 17 00:00:00 2001
- From: Eric Biggers <[email protected]>
- Date: Wed, 14 Nov 2018 12:19:39 -0800
- Subject: [PATCH 03/15] crypto: drop mask=CRYPTO_ALG_ASYNC from 'cipher' tfm
- allocations
- 'cipher' algorithms (single block ciphers) are always synchronous, so
- passing CRYPTO_ALG_ASYNC in the mask to crypto_alloc_cipher() has no
- effect. Many users therefore already don't pass it, but some still do.
- This inconsistency can cause confusion, especially since the way the
- 'mask' argument works is somewhat counterintuitive.
- Thus, just remove the unneeded CRYPTO_ALG_ASYNC flags.
- This patch shouldn't change any actual behavior.
- Signed-off-by: Eric Biggers <[email protected]>
- Signed-off-by: Herbert Xu <[email protected]>
- ---
- drivers/crypto/amcc/crypto4xx_alg.c | 3 +--
- 1 file changed, 1 insertion(+), 2 deletions(-)
- --- a/drivers/crypto/amcc/crypto4xx_alg.c
- +++ b/drivers/crypto/amcc/crypto4xx_alg.c
- @@ -540,8 +540,7 @@ static int crypto4xx_compute_gcm_hash_ke
- uint8_t src[16] = { 0 };
- int rc = 0;
-
- - aes_tfm = crypto_alloc_cipher("aes", 0, CRYPTO_ALG_ASYNC |
- - CRYPTO_ALG_NEED_FALLBACK);
- + aes_tfm = crypto_alloc_cipher("aes", 0, CRYPTO_ALG_NEED_FALLBACK);
- if (IS_ERR(aes_tfm)) {
- rc = PTR_ERR(aes_tfm);
- pr_warn("could not load aes cipher driver: %d\n", rc);
|