diff options
author | Herbert Xu <herbert@gondor.apana.org.au> | 2016-01-26 11:16:37 -0500 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2016-01-27 07:25:13 -0500 |
commit | 00420a65fa2beb3206090ead86942484df2275f3 (patch) | |
tree | 887480bf63c072d111d1e277432efd1fa3153d69 /crypto | |
parent | 625b4a982788fefe1361daee2ea160192cd80572 (diff) |
crypto: shash - Fix has_key setting
The has_key logic is wrong for shash algorithms as they always
have a setkey function. So we should instead be testing against
shash_no_setkey.
Fixes: a5596d633278 ("crypto: hash - Add crypto_ahash_has_setkey")
Cc: stable@vger.kernel.org
Reported-by: Stephan Mueller <smueller@chronox.de>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Stephan Mueller <smueller@chronox.de>
Diffstat (limited to 'crypto')
-rw-r--r-- | crypto/shash.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/crypto/shash.c b/crypto/shash.c index 88a27de79848..359754591653 100644 --- a/crypto/shash.c +++ b/crypto/shash.c | |||
@@ -354,11 +354,10 @@ int crypto_init_shash_ops_async(struct crypto_tfm *tfm) | |||
354 | crt->final = shash_async_final; | 354 | crt->final = shash_async_final; |
355 | crt->finup = shash_async_finup; | 355 | crt->finup = shash_async_finup; |
356 | crt->digest = shash_async_digest; | 356 | crt->digest = shash_async_digest; |
357 | crt->setkey = shash_async_setkey; | ||
358 | |||
359 | crt->has_setkey = alg->setkey != shash_no_setkey; | ||
357 | 360 | ||
358 | if (alg->setkey) { | ||
359 | crt->setkey = shash_async_setkey; | ||
360 | crt->has_setkey = true; | ||
361 | } | ||
362 | if (alg->export) | 361 | if (alg->export) |
363 | crt->export = shash_async_export; | 362 | crt->export = shash_async_export; |
364 | if (alg->import) | 363 | if (alg->import) |