summaryrefslogtreecommitdiff
path: root/src/usr.bin/openssl/ecparam.c
diff options
context:
space:
mode:
authordoug <>2015-07-20 21:56:47 +0000
committerdoug <>2015-07-20 21:56:47 +0000
commit5d357bac46bea38ee1ee013733a720859014071a (patch)
tree4b5e590c0c19a7c05e63409e17e02092de6b825b /src/usr.bin/openssl/ecparam.c
parent4110bba1b380615e51f07e2247b3b3d4bc6a589a (diff)
downloadopenbsd-5d357bac46bea38ee1ee013733a720859014071a.tar.gz
openbsd-5d357bac46bea38ee1ee013733a720859014071a.tar.bz2
openbsd-5d357bac46bea38ee1ee013733a720859014071a.zip
No need to recheck for NULL in openssl(1) ecparam.
Fixes Coverity issue 78802. ok bcook@
Diffstat (limited to 'src/usr.bin/openssl/ecparam.c')
-rw-r--r--src/usr.bin/openssl/ecparam.c6
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)