diff options
author | Alex Cope <alexcope@google.com> | 2016-11-14 11:02:54 -0800 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2016-11-17 23:34:59 +0800 |
commit | 75aa0a7cafe951538c7cb7c5ed457a3371ec5bcd (patch) | |
tree | 9cbf746474322cb3dfab8ab135b01145f72508a3 | |
parent | b516b88a7946c92d418c8ad2673b94ea568d1cf5 (diff) | |
download | talos-op-linux-75aa0a7cafe951538c7cb7c5ed457a3371ec5bcd.tar.gz talos-op-linux-75aa0a7cafe951538c7cb7c5ed457a3371ec5bcd.zip |
crypto: gf128mul - Zero memory when freeing multiplication table
GF(2^128) multiplication tables are typically used for secret
information, so it's a good idea to zero them on free.
Signed-off-by: Alex Cope <alexcope@google.com>
Signed-off-by: Eric Biggers <ebiggers@google.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
-rw-r--r-- | crypto/gf128mul.c | 4 | ||||
-rw-r--r-- | include/crypto/gf128mul.h | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/crypto/gf128mul.c b/crypto/gf128mul.c index 57c85ddbd852..72015fee533d 100644 --- a/crypto/gf128mul.c +++ b/crypto/gf128mul.c @@ -310,8 +310,8 @@ void gf128mul_free_64k(struct gf128mul_64k *t) int i; for (i = 0; i < 16; i++) - kfree(t->t[i]); - kfree(t); + kzfree(t->t[i]); + kzfree(t); } EXPORT_SYMBOL(gf128mul_free_64k); diff --git a/include/crypto/gf128mul.h b/include/crypto/gf128mul.h index b611aa99f9b4..592d47e565a8 100644 --- a/include/crypto/gf128mul.h +++ b/include/crypto/gf128mul.h @@ -177,7 +177,7 @@ void gf128mul_4k_bbe(be128 *a, struct gf128mul_4k *t); static inline void gf128mul_free_4k(struct gf128mul_4k *t) { - kfree(t); + kzfree(t); } |