diff options
author | doug <> | 2015-07-20 21:56:47 +0000 |
---|---|---|
committer | doug <> | 2015-07-20 21:56:47 +0000 |
commit | 770c1a1e3dd5a12c14b8bda6f18802758f37f0ab (patch) | |
tree | 4b5e590c0c19a7c05e63409e17e02092de6b825b /src | |
parent | c866eef593eb0b6a68ae4c9cfbbb2cff0aa6b167 (diff) | |
download | openbsd-770c1a1e3dd5a12c14b8bda6f18802758f37f0ab.tar.gz openbsd-770c1a1e3dd5a12c14b8bda6f18802758f37f0ab.tar.bz2 openbsd-770c1a1e3dd5a12c14b8bda6f18802758f37f0ab.zip |
No need to recheck for NULL in openssl(1) ecparam.
Fixes Coverity issue 78802.
ok bcook@
Diffstat (limited to 'src')
-rw-r--r-- | src/usr.bin/openssl/ecparam.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/usr.bin/openssl/ecparam.c b/src/usr.bin/openssl/ecparam.c index 361d2350fa..1410d89f4e 100644 --- a/src/usr.bin/openssl/ecparam.c +++ b/src/usr.bin/openssl/ecparam.c | |||
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: ecparam.c,v 1.10 2015/07/20 03:02:46 doug Exp $ */ | 1 | /* $OpenBSD: ecparam.c,v 1.11 2015/07/20 21:56:47 doug Exp $ */ |
2 | /* | 2 | /* |
3 | * Written by Nils Larsch for the OpenSSL project. | 3 | * Written by Nils Larsch for the OpenSSL project. |
4 | */ | 4 | */ |
@@ -451,10 +451,6 @@ ecparam_main(int argc, char **argv) | |||
451 | if (!EC_GROUP_get_cofactor(group, ec_cofactor, NULL)) | 451 | if (!EC_GROUP_get_cofactor(group, ec_cofactor, NULL)) |
452 | goto end; | 452 | goto end; |
453 | 453 | ||
454 | if (!ec_p || !ec_a || !ec_b || !ec_gen || | ||
455 | !ec_order || !ec_cofactor) | ||
456 | goto end; | ||
457 | |||
458 | len = BN_num_bits(ec_order); | 454 | len = BN_num_bits(ec_order); |
459 | 455 | ||
460 | if ((tmp_len = (size_t) BN_num_bytes(ec_p)) > buf_len) | 456 | if ((tmp_len = (size_t) BN_num_bytes(ec_p)) > buf_len) |