]> git.itanic.dy.fi Git - linux-stable/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)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 15 Dec 2016 16:50:35 +0000 (08:50 -0800)
commitbdb5ed2040f54bc4b2cdaf2f313588daee694b82
tree4d65edad0f2de79a811b494523c31ee6760c6e83
parentbfef274e4dae76cdee275b5985c85758e346e825
crypto: marvell - Don't corrupt state of an STD req for re-stepped ahash

commit 9e5f7a149e00d211177f6de8be427ebc72a1c363 upstream.

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 2786cee8e50b ("crypto: marvell - Move SRAM I/O op...")
Signed-off-by: Romain Perier <romain.perier@free-electrons.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/crypto/marvell/hash.c