]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
3.18-stable patches
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 23 Feb 2018 16:52:33 +0000 (17:52 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 23 Feb 2018 16:52:33 +0000 (17:52 +0100)
added patches:
crypto-s5p-sss-fix-kernel-oops-in-aes-ecb-mode.patch

queue-3.18/crypto-s5p-sss-fix-kernel-oops-in-aes-ecb-mode.patch [new file with mode: 0644]
queue-3.18/series

diff --git a/queue-3.18/crypto-s5p-sss-fix-kernel-oops-in-aes-ecb-mode.patch b/queue-3.18/crypto-s5p-sss-fix-kernel-oops-in-aes-ecb-mode.patch
new file mode 100644 (file)
index 0000000..dc60f31
--- /dev/null
@@ -0,0 +1,60 @@
+From c927b080c67e3e97193c81fc1d27f4251bf4e036 Mon Sep 17 00:00:00 2001
+From: Kamil Konieczny <k.konieczny@partner.samsung.com>
+Date: Wed, 7 Feb 2018 16:52:09 +0100
+Subject: crypto: s5p-sss - Fix kernel Oops in AES-ECB mode
+
+From: Kamil Konieczny <k.konieczny@partner.samsung.com>
+
+commit c927b080c67e3e97193c81fc1d27f4251bf4e036 upstream.
+
+In AES-ECB mode crypt is done with key only, so any use of IV
+can cause kernel Oops. Use IV only in AES-CBC and AES-CTR.
+
+Signed-off-by: Kamil Konieczny <k.konieczny@partner.samsung.com>
+Reported-by: Anand Moon <linux.amoon@gmail.com>
+Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
+Tested-by: Anand Moon <linux.amoon@gmail.com>
+Cc: stable@vger.kernel.org # can be applied after commit 8f9702aad138
+Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+
+---
+ drivers/crypto/s5p-sss.c |   13 +++++++++----
+ 1 file changed, 9 insertions(+), 4 deletions(-)
+
+--- a/drivers/crypto/s5p-sss.c
++++ b/drivers/crypto/s5p-sss.c
+@@ -419,16 +419,21 @@ static void s5p_aes_crypt_start(struct s
+       uint32_t                    aes_control;
+       int                         err;
+       unsigned long               flags;
++      u8 *iv;
+       aes_control = SSS_AES_KEY_CHANGE_MODE;
+       if (mode & FLAGS_AES_DECRYPT)
+               aes_control |= SSS_AES_MODE_DECRYPT;
+-      if ((mode & FLAGS_AES_MODE_MASK) == FLAGS_AES_CBC)
++      if ((mode & FLAGS_AES_MODE_MASK) == FLAGS_AES_CBC) {
+               aes_control |= SSS_AES_CHAIN_MODE_CBC;
+-      else if ((mode & FLAGS_AES_MODE_MASK) == FLAGS_AES_CTR)
++              iv = req->info;
++      } else if ((mode & FLAGS_AES_MODE_MASK) == FLAGS_AES_CTR) {
+               aes_control |= SSS_AES_CHAIN_MODE_CTR;
+-
++              iv = req->info;
++      } else {
++              iv = NULL; /* AES_ECB */
++      }
+       if (dev->ctx->keylen == AES_KEYSIZE_192)
+               aes_control |= SSS_AES_KEY_SIZE_192;
+       else if (dev->ctx->keylen == AES_KEYSIZE_256)
+@@ -458,7 +463,7 @@ static void s5p_aes_crypt_start(struct s
+               goto outdata_error;
+       SSS_AES_WRITE(dev, AES_CONTROL, aes_control);
+-      s5p_set_aes(dev, dev->ctx->aes_key, req->info, dev->ctx->keylen);
++      s5p_set_aes(dev, dev->ctx->aes_key, iv, dev->ctx->keylen);
+       s5p_set_dma_indata(dev,  req->src);
+       s5p_set_dma_outdata(dev, req->dst);
index 14a471f9b27938c5dc6d36e783034aaa2f3add44..6d5f8d60c3a812bcacc9c6856c6e4010d7b638cf 100644 (file)
@@ -55,3 +55,4 @@ x86-mm-kmmio-fix-mmiotrace-for-page-unaligned-addresses.patch
 xen-xen_acpi_processor-is-dom0-only.patch
 hippi-fix-a-fix-a-possible-sleep-in-atomic-bug-in-rr_close.patch
 kvm-async_pf-fix-df-due-to-inject-page-not-present-and-page-ready-exceptions-simultaneously.patch
+crypto-s5p-sss-fix-kernel-oops-in-aes-ecb-mode.patch