diff options
author | Yehuda Sadeh <yehuda@hq.newdream.net> | 2009-03-27 13:03:51 +0800 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2009-03-27 13:03:51 +0800 |
commit | 5d8f02acff790f62a476754252fbbc6f5ce4250d (patch) | |
tree | 96c91264af588c9ab8025005180041bb502889d1 /crypto/shash.c | |
parent | 6c89089ed788cdb56b63a15ffd57ec7d3a530b20 (diff) | |
download | linux-crypto-5d8f02acff790f62a476754252fbbc6f5ce4250d.tar.gz linux-crypto-5d8f02acff790f62a476754252fbbc6f5ce4250d.zip |
crypto: shash - Fix unaligned calculation with short length
When the total length is shorter than the calculated number of unaligned bytes, the call to shash->update breaks. For example, calling crc32c on unaligned buffer with length of 1 can result in a system crash.
Signed-off-by: Yehuda Sadeh <yehuda@hq.newdream.net>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto/shash.c')
-rw-r--r-- | crypto/shash.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/crypto/shash.c b/crypto/shash.c index 7a659733..2ccc8b00 100644 --- a/crypto/shash.c +++ b/crypto/shash.c @@ -77,6 +77,9 @@ static int shash_update_unaligned(struct shash_desc *desc, const u8 *data, u8 buf[shash_align_buffer_size(unaligned_len, alignmask)] __attribute__ ((aligned)); + if (unaligned_len > len) + unaligned_len = len; + memcpy(buf, data, unaligned_len); return shash->update(desc, buf, unaligned_len) ?: |