diff options
author | Herbert Xu <herbert@gondor.apana.org.au> | 2008-07-07 08:23:56 -0400 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2008-07-10 08:35:17 -0400 |
commit | ca786dc738f4f583b57b1bba7a335b5e8233f4b0 (patch) | |
tree | e2e6178fac1d9b3ac2b557cac76977e15f7d5d2c /crypto/digest.c | |
parent | caee16883a235b1b042282276859e7d5901fad21 (diff) |
crypto: hash - Fixed digest size check
The digest size check on hash algorithms is incorrect. It's
perfectly valid for hash algorithms to have a digest length
longer than their block size. For example crc32c has a block
size of 1 and a digest size of 4. Rather than having it lie
about its block size, this patch fixes the checks to do what
they really should which is to bound the digest size so that
code placing the digest on the stack continue to work.
HMAC however still needs to check this as it's only defined
for such algorithms.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto/digest.c')
-rw-r--r-- | crypto/digest.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/digest.c b/crypto/digest.c index 025c9aea24ed..d63d5d96feec 100644 --- a/crypto/digest.c +++ b/crypto/digest.c | |||
@@ -141,7 +141,7 @@ int crypto_init_digest_ops(struct crypto_tfm *tfm) | |||
141 | struct hash_tfm *ops = &tfm->crt_hash; | 141 | struct hash_tfm *ops = &tfm->crt_hash; |
142 | struct digest_alg *dalg = &tfm->__crt_alg->cra_digest; | 142 | struct digest_alg *dalg = &tfm->__crt_alg->cra_digest; |
143 | 143 | ||
144 | if (dalg->dia_digestsize > crypto_tfm_alg_blocksize(tfm)) | 144 | if (dalg->dia_digestsize > PAGE_SIZE / 8) |
145 | return -EINVAL; | 145 | return -EINVAL; |
146 | 146 | ||
147 | ops->init = init; | 147 | ops->init = init; |