diff options
| author | Szilveszter Ördög <slipszi@gmail.com> | 2010-03-03 08:03:23 +0800 |
|---|---|---|
| committer | Herbert Xu <herbert@gondor.apana.org.au> | 2010-03-03 08:03:23 +0800 |
| commit | ba91796957fbf00d5e0030974b89a00935e1e863 (patch) | |
| tree | 0a5707fdd62b9e8fe2e491c57f6d5fcb40e43111 /crypto/ahash.c | |
| parent | 80878e51c9804ae17cf6f383db3fb599ddc9750c (diff) | |
| download | linux-crypto-ba91796957fbf00d5e0030974b89a00935e1e863.tar.gz linux-crypto-ba91796957fbf00d5e0030974b89a00935e1e863.zip | |
crypto: hash - Fix handling of unaligned buffers
The correct way to calculate the start of the aligned part of an
unaligned buffer is:
offset = ALIGN(offset, alignmask + 1);
However, crypto_hash_walk_done() has:
offset += alignmask - 1;
offset = ALIGN(offset, alignmask + 1);
which actually skips a whole block unless offset % (alignmask + 1) == 1.
This patch fixes the problem.
Signed-off-by: Szilveszter Ördög <slipszi@gmail.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto/ahash.c')
| -rw-r--r-- | crypto/ahash.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/crypto/ahash.c b/crypto/ahash.c index 33a4ff45..b8c59b88 100644 --- a/crypto/ahash.c +++ b/crypto/ahash.c @@ -78,7 +78,6 @@ int crypto_hash_walk_done(struct crypto_hash_walk *walk, int err) walk->data -= walk->offset; if (nbytes && walk->offset & alignmask && !err) { - walk->offset += alignmask - 1; walk->offset = ALIGN(walk->offset, alignmask + 1); walk->data += walk->offset; |