summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@linaro.org>2023-10-30 12:02:59 +0300
committerHerbert Xu <herbert@gondor.apana.org.au>2023-11-17 19:16:29 +0800
commitd872ca165cb67112f2841ef9c37d51ef7e63d1e4 (patch)
tree4362d1d267146d179960b36097da7aaf4f6d035f
parentfea845fd79b534948a707511591c059d293668c1 (diff)
downloadlinux-stable-d872ca165cb67112f2841ef9c37d51ef7e63d1e4.tar.gz
linux-stable-d872ca165cb67112f2841ef9c37d51ef7e63d1e4.tar.bz2
linux-stable-d872ca165cb67112f2841ef9c37d51ef7e63d1e4.zip
crypto: rsa - add a check for allocation failure
Static checkers insist that the mpi_alloc() allocation can fail so add a check to prevent a NULL dereference. Small allocations like this can't actually fail in current kernels, but adding a check is very simple and makes the static checkers happy. Fixes: 6637e11e4ad2 ("crypto: rsa - allow only odd e and restrict value in FIPS mode") Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
-rw-r--r--crypto/rsa.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/crypto/rsa.c b/crypto/rsa.c
index c79613cdce6e..b9cd11fb7d36 100644
--- a/crypto/rsa.c
+++ b/crypto/rsa.c
@@ -220,6 +220,8 @@ static int rsa_check_exponent_fips(MPI e)
}
e_max = mpi_alloc(0);
+ if (!e_max)
+ return -ENOMEM;
mpi_set_bit(e_max, 256);
if (mpi_cmp(e, e_max) >= 0) {