Commit 441f99c9 authored by Romain Izard's avatar Romain Izard Committed by Herbert Xu

crypto: ccm - preserve the IV buffer

The IV buffer used during CCM operations is used twice, during both the
hashing step and the ciphering step.

When using a hardware accelerator that updates the contents of the IV
buffer at the end of ciphering operations, the value will be modified.
In the decryption case, the subsequent setup of the hashing algorithm
will interpret the updated IV instead of the original value, which can
lead to out-of-bounds writes.

Reuse the idata buffer, only used in the hashing step, to preserve the
IV's value during the ciphering step in the decryption case.
Signed-off-by: default avatarRomain Izard <romain.izard.pro@gmail.com>
Reviewed-by: default avatarTudor Ambarus <tudor.ambarus@microchip.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent d041b557
...@@ -363,7 +363,7 @@ static int crypto_ccm_decrypt(struct aead_request *req) ...@@ -363,7 +363,7 @@ static int crypto_ccm_decrypt(struct aead_request *req)
unsigned int cryptlen = req->cryptlen; unsigned int cryptlen = req->cryptlen;
u8 *authtag = pctx->auth_tag; u8 *authtag = pctx->auth_tag;
u8 *odata = pctx->odata; u8 *odata = pctx->odata;
u8 *iv = req->iv; u8 *iv = pctx->idata;
int err; int err;
cryptlen -= authsize; cryptlen -= authsize;
...@@ -379,6 +379,8 @@ static int crypto_ccm_decrypt(struct aead_request *req) ...@@ -379,6 +379,8 @@ static int crypto_ccm_decrypt(struct aead_request *req)
if (req->src != req->dst) if (req->src != req->dst)
dst = pctx->dst; dst = pctx->dst;
memcpy(iv, req->iv, 16);
skcipher_request_set_tfm(skreq, ctx->ctr); skcipher_request_set_tfm(skreq, ctx->ctr);
skcipher_request_set_callback(skreq, pctx->flags, skcipher_request_set_callback(skreq, pctx->flags,
crypto_ccm_decrypt_done, req); crypto_ccm_decrypt_done, req);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment