diff options
| author | doug <> | 2015-07-20 21:56:47 +0000 |
|---|---|---|
| committer | doug <> | 2015-07-20 21:56:47 +0000 |
| commit | 5d357bac46bea38ee1ee013733a720859014071a (patch) | |
| tree | 4b5e590c0c19a7c05e63409e17e02092de6b825b /src/usr.bin/openssl/ecparam.c | |
| parent | 4110bba1b380615e51f07e2247b3b3d4bc6a589a (diff) | |
| download | openbsd-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 '')
| -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) |
