Commit 73af07de authored by Herbert Xu's avatar Herbert Xu Committed by Linus Torvalds

[CRYPTO] hmac: Fix error truncation by unlikely()

The error return values are truncated by unlikely so we need to
save it first.  Thanks to Kyle Moffett for spotting this.
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 79da342c
...@@ -92,13 +92,17 @@ static int hmac_init(struct hash_desc *pdesc) ...@@ -92,13 +92,17 @@ static int hmac_init(struct hash_desc *pdesc)
struct hmac_ctx *ctx = align_ptr(ipad + bs * 2 + ds, sizeof(void *)); struct hmac_ctx *ctx = align_ptr(ipad + bs * 2 + ds, sizeof(void *));
struct hash_desc desc; struct hash_desc desc;
struct scatterlist tmp; struct scatterlist tmp;
int err;
desc.tfm = ctx->child; desc.tfm = ctx->child;
desc.flags = pdesc->flags & CRYPTO_TFM_REQ_MAY_SLEEP; desc.flags = pdesc->flags & CRYPTO_TFM_REQ_MAY_SLEEP;
sg_set_buf(&tmp, ipad, bs); sg_set_buf(&tmp, ipad, bs);
return unlikely(crypto_hash_init(&desc)) ?: err = crypto_hash_init(&desc);
crypto_hash_update(&desc, &tmp, bs); if (unlikely(err))
return err;
return crypto_hash_update(&desc, &tmp, bs);
} }
static int hmac_update(struct hash_desc *pdesc, static int hmac_update(struct hash_desc *pdesc,
...@@ -123,13 +127,17 @@ static int hmac_final(struct hash_desc *pdesc, u8 *out) ...@@ -123,13 +127,17 @@ static int hmac_final(struct hash_desc *pdesc, u8 *out)
struct hmac_ctx *ctx = align_ptr(digest + ds, sizeof(void *)); struct hmac_ctx *ctx = align_ptr(digest + ds, sizeof(void *));
struct hash_desc desc; struct hash_desc desc;
struct scatterlist tmp; struct scatterlist tmp;
int err;
desc.tfm = ctx->child; desc.tfm = ctx->child;
desc.flags = pdesc->flags & CRYPTO_TFM_REQ_MAY_SLEEP; desc.flags = pdesc->flags & CRYPTO_TFM_REQ_MAY_SLEEP;
sg_set_buf(&tmp, opad, bs + ds); sg_set_buf(&tmp, opad, bs + ds);
return unlikely(crypto_hash_final(&desc, digest)) ?: err = crypto_hash_final(&desc, digest);
crypto_hash_digest(&desc, &tmp, bs + ds, out); if (unlikely(err))
return err;
return crypto_hash_digest(&desc, &tmp, bs + ds, out);
} }
static int hmac_digest(struct hash_desc *pdesc, struct scatterlist *sg, static int hmac_digest(struct hash_desc *pdesc, struct scatterlist *sg,
...@@ -145,6 +153,7 @@ static int hmac_digest(struct hash_desc *pdesc, struct scatterlist *sg, ...@@ -145,6 +153,7 @@ static int hmac_digest(struct hash_desc *pdesc, struct scatterlist *sg,
struct hash_desc desc; struct hash_desc desc;
struct scatterlist sg1[2]; struct scatterlist sg1[2];
struct scatterlist sg2[1]; struct scatterlist sg2[1];
int err;
desc.tfm = ctx->child; desc.tfm = ctx->child;
desc.flags = pdesc->flags & CRYPTO_TFM_REQ_MAY_SLEEP; desc.flags = pdesc->flags & CRYPTO_TFM_REQ_MAY_SLEEP;
...@@ -154,8 +163,11 @@ static int hmac_digest(struct hash_desc *pdesc, struct scatterlist *sg, ...@@ -154,8 +163,11 @@ static int hmac_digest(struct hash_desc *pdesc, struct scatterlist *sg,
sg1[1].length = 0; sg1[1].length = 0;
sg_set_buf(sg2, opad, bs + ds); sg_set_buf(sg2, opad, bs + ds);
return unlikely(crypto_hash_digest(&desc, sg1, nbytes + bs, digest)) ?: err = crypto_hash_digest(&desc, sg1, nbytes + bs, digest);
crypto_hash_digest(&desc, sg2, bs + ds, out); if (unlikely(err))
return err;
return crypto_hash_digest(&desc, sg2, bs + ds, out);
} }
static int hmac_init_tfm(struct crypto_tfm *tfm) static int hmac_init_tfm(struct crypto_tfm *tfm)
......
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