]> git.baikalelectronics.ru Git - kernel.git/commit
crypto: marvell - Don't corrupt state of an STD req for re-stepped ahash
authorRomain Perier <romain.perier@free-electrons.com>
Mon, 5 Dec 2016 08:56:39 +0000 (09:56 +0100)
committerHerbert Xu <herbert@gondor.apana.org.au>
Wed, 7 Dec 2016 11:55:17 +0000 (19:55 +0800)
commit7425cccf44c14221d0b465a78846a2f353c18537
tree59e300f413afc7deb48395bd86b98806d92f16b9
parent75d3eca8d9d3e9e4c05c2501354f7efdac171a0a
crypto: marvell - Don't corrupt state of an STD req for re-stepped ahash

mv_cesa_hash_std_step() copies the creq->state into the SRAM at each
step, but this is only required on the first one. By doing that, we
overwrite the engine state, and get erroneous results when the crypto
request is split in several chunks to fit in the internal SRAM.

This commit changes the function to copy the state only on the first
step.

Fixes: commit 97fe2e3c6cbf ("crypto: marvell - Move SRAM I/O op...")
Signed-off-by: Romain Perier <romain.perier@free-electrons.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/marvell/hash.c