diff options
author | Rich Salz <rsalz@openssl.org> | 2016-11-15 18:54:28 -0500 |
---|---|---|
committer | Rich Salz <rsalz@openssl.org> | 2016-11-15 19:03:18 -0500 |
commit | d18afb5bf29dc3b81b5f7a9eda2abde35041a441 (patch) | |
tree | 9c66c5792423c82d4710445ca21352042e8a0f42 /crypto/rsa/rsa_gen.c | |
parent | 18ad46297dcd44c1ac6c9f7f2f966e19c4d45e91 (diff) |
Check return value of some BN functions.
Factorise multiple bn_get_top(group->field) calls
Add missing checks on some conditional BN_copy return value
Add missing checks on some BN_copy return value
Add missing checks on a few bn_wexpand return value
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/1626)
(cherry picked from commit 78e09b53a40729f5e99829ccc733b592bd22fea1)
Diffstat (limited to 'crypto/rsa/rsa_gen.c')
-rw-r--r-- | crypto/rsa/rsa_gen.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/crypto/rsa/rsa_gen.c b/crypto/rsa/rsa_gen.c index 5c6b6192e6..0d1d56bfe7 100644 --- a/crypto/rsa/rsa_gen.c +++ b/crypto/rsa/rsa_gen.c @@ -75,7 +75,8 @@ static int rsa_builtin_keygen(RSA *rsa, int bits, BIGNUM *e_value, if (!rsa->iqmp && ((rsa->iqmp = BN_secure_new()) == NULL)) goto err; - BN_copy(rsa->e, e_value); + if (BN_copy(rsa->e, e_value) == NULL) + goto err; /* generate p and q */ for (;;) { |