diff options
| author |   doug <doug@openbsd.org> | 2015-07-20 21:56:47 +0000 |
|---|---|---|
| committer | doug <doug@openbsd.org> | 2015-07-20 21:56:47 +0000 |
| commit | 7d48b8ed0d09099b44e66e8a35cb816a3a1c01df (patch) | |
| tree | e49aca54e63f5ef042a71d2ab8d3329b44700a93 | |
| parent | Don't try to run ECDH if ecdh_checks fails in openssl(1) speed. (diff) | |
| download | wireguard-openbsd-7d48b8ed0d09099b44e66e8a35cb816a3a1c01df.tar.xz wireguard-openbsd-7d48b8ed0d09099b44e66e8a35cb816a3a1c01df.zip | |
No need to recheck for NULL in openssl(1) ecparam.
Fixes Coverity issue 78802.
ok bcook@
| -rw-r--r-- | usr.bin/openssl/ecparam.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/usr.bin/openssl/ecparam.c b/usr.bin/openssl/ecparam.c index 361d2350faa..1410d89f4e0 100644 --- a/usr.bin/openssl/ecparam.c +++ b/usr.bin/openssl/ecparam.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ecparam.c,v 1.10 2015/07/20 03:02:46 doug Exp $ */ +/* $OpenBSD: ecparam.c,v 1.11 2015/07/20 21:56:47 doug Exp $ */ /* * Written by Nils Larsch for the OpenSSL project. */ @@ -451,10 +451,6 @@ ecparam_main(int argc, char **argv) if (!EC_GROUP_get_cofactor(group, ec_cofactor, NULL)) goto end; - if (!ec_p || !ec_a || !ec_b || !ec_gen || - !ec_order || !ec_cofactor) - goto end; - len = BN_num_bits(ec_order); if ((tmp_len = (size_t) BN_num_bytes(ec_p)) > buf_len) |