diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2023-10-30 12:02:59 +0300 |
|---|---|---|
| committer | Herbert Xu <herbert@gondor.apana.org.au> | 2023-11-17 19:16:29 +0800 |
| commit | d872ca165cb67112f2841ef9c37d51ef7e63d1e4 (patch) | |
| tree | 4362d1d267146d179960b36097da7aaf4f6d035f /crypto/rsa.c | |
| parent | fea845fd79b534948a707511591c059d293668c1 (diff) | |
| download | lwn-d872ca165cb67112f2841ef9c37d51ef7e63d1e4.tar.gz lwn-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>
Diffstat (limited to 'crypto/rsa.c')
| -rw-r--r-- | crypto/rsa.c | 2 |
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) { |