crypto: rsa - add a check for allocation failure

[ Upstream commit d872ca165cb67112f2841ef9c37d51ef7e63d1e4 ]

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: 6637e11e4a ("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>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Dan Carpenter 2023-10-30 12:02:59 +03:00 committed by Greg Kroah-Hartman
parent 089ff0eeb7
commit 2831f4d3bf
1 changed files with 2 additions and 0 deletions

View File

@ -220,6 +220,8 @@ static int rsa_check_exponent_fips(MPI e)
} }
e_max = mpi_alloc(0); e_max = mpi_alloc(0);
if (!e_max)
return -ENOMEM;
mpi_set_bit(e_max, 256); mpi_set_bit(e_max, 256);
if (mpi_cmp(e, e_max) >= 0) { if (mpi_cmp(e, e_max) >= 0) {