>From 335b39e0c55a1dba13cda3e8222947f2cb4120ed Mon Sep 17 00:00:00 2001
>From: Sergey Mironov <ier...@gmail.com>
>Date: Thu, 12 Nov 2009 13:10:05 +0300
>Subject: [PATCH 2/2] aes_s390: access fallback.cip cipher fallback mode
>
>|The fallback code in cipher mode touch the union fallback.blk instead
>|of fallback.cip. This is wrong because we use the cipher and not the
>|blockcipher. This did not show any side effects yet because both types /
>|structs contain the same element right now.
>
>Signed-off-by: Sergey Mironov <ier...@gmail.com>

Looks good.

>---
> arch/s390/crypto/aes_s390.c |    8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/arch/s390/crypto/aes_s390.c b/arch/s390/crypto/aes_s390.c
>index e33f32b..6f0f8b9 100644
>--- a/arch/s390/crypto/aes_s390.c
>+++ b/arch/s390/crypto/aes_s390.c
>@@ -75,14 +75,14 @@ static int setkey_fallback_cip(struct crypto_tfm
>*tfm, const u8 *in_key,
>       struct s390_aes_ctx *sctx = crypto_tfm_ctx(tfm);
>       int ret;
>
>-      sctx->fallback.blk->base.crt_flags &= ~CRYPTO_TFM_REQ_MASK;
>-      sctx->fallback.blk->base.crt_flags |= (tfm->crt_flags &
>+      sctx->fallback.cip->base.crt_flags &= ~CRYPTO_TFM_REQ_MASK;
>+      sctx->fallback.cip->base.crt_flags |= (tfm->crt_flags &
>                       CRYPTO_TFM_REQ_MASK);
>
>       ret = crypto_cipher_setkey(sctx->fallback.cip, in_key, key_len);
>       if (ret) {
>               tfm->crt_flags &= ~CRYPTO_TFM_RES_MASK;
>-              tfm->crt_flags |= (sctx->fallback.blk->base.crt_flags &
>+              tfm->crt_flags |= (sctx->fallback.cip->base.crt_flags &
>                               CRYPTO_TFM_RES_MASK);
>       }
>       return ret;
>@@ -170,7 +170,7 @@ static int fallback_init_cip(struct crypto_tfm *tfm)
>
>       if (IS_ERR(sctx->fallback.cip)) {
>               printk(KERN_ERR "Error allocating fallback algo %s\n", name);
>-              return PTR_ERR(sctx->fallback.blk);
>+              return PTR_ERR(sctx->fallback.cip);
>       }
>
>       return 0;
>-- 
>1.6.4.4
>--
>To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
>the body of a message to majord...@vger.kernel.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to