summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOndrej Mosnáček <omosnacek@gmail.com>2016-09-23 10:47:32 +0200
committerHerbert Xu <herbert@gondor.apana.org.au>2016-10-02 22:31:52 +0800
commit50d2e6dc1f83db0563c7d6603967bf9585ce934b (patch)
tree67e18e631fd217f787b39a821046b77356022e6f
parent5ecf8ef9103cb018cbd82b6eace529ff4c5b5c66 (diff)
downloadop-kernel-dev-50d2e6dc1f83db0563c7d6603967bf9585ce934b.zip
op-kernel-dev-50d2e6dc1f83db0563c7d6603967bf9585ce934b.tar.gz
crypto: gcm - Fix IV buffer size in crypto_gcm_setkey
The cipher block size for GCM is 16 bytes, and thus the CTR transform used in crypto_gcm_setkey() will also expect a 16-byte IV. However, the code currently reserves only 8 bytes for the IV, causing an out-of-bounds access in the CTR transform. This patch fixes the issue by setting the size of the IV buffer to 16 bytes. Fixes: 84c911523020 ("[CRYPTO] gcm: Add support for async ciphers") Signed-off-by: Ondrej Mosnacek <omosnacek@gmail.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
-rw-r--r--crypto/gcm.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/gcm.c b/crypto/gcm.c
index 70a892e8..f624ac9 100644
--- a/crypto/gcm.c
+++ b/crypto/gcm.c
@@ -117,7 +117,7 @@ static int crypto_gcm_setkey(struct crypto_aead *aead, const u8 *key,
struct crypto_skcipher *ctr = ctx->ctr;
struct {
be128 hash;
- u8 iv[8];
+ u8 iv[16];
struct crypto_gcm_setkey_result result;
OpenPOWER on IntegriCloud