From 93f7d0ffdf10b7d5b4a44864db24f6cbfac925c1 Mon Sep 17 00:00:00 2001 From: miod <> Date: Tue, 7 Oct 2014 04:58:50 +0000 Subject: EC_KEY_set_group() does an EC_GROUP_dup() of its argument, so we don't need to do it in ec_copy_parameters() prior to invoking EC_KEY_set_group(). ok doug@ jsing@ --- src/lib/libcrypto/ec/ec_ameth.c | 10 ++-------- src/lib/libssl/src/crypto/ec/ec_ameth.c | 10 ++-------- 2 files changed, 4 insertions(+), 16 deletions(-) diff --git a/src/lib/libcrypto/ec/ec_ameth.c b/src/lib/libcrypto/ec/ec_ameth.c index 6b4dfdec46..b4ae1c0623 100644 --- a/src/lib/libcrypto/ec/ec_ameth.c +++ b/src/lib/libcrypto/ec/ec_ameth.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ec_ameth.c,v 1.13 2014/07/13 15:47:51 logan Exp $ */ +/* $OpenBSD: ec_ameth.c,v 1.14 2014/10/07 04:58:50 miod Exp $ */ /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL * project 2006. */ @@ -393,13 +393,7 @@ ec_missing_parameters(const EVP_PKEY * pkey) static int ec_copy_parameters(EVP_PKEY * to, const EVP_PKEY * from) { - EC_GROUP *group = EC_GROUP_dup(EC_KEY_get0_group(from->pkey.ec)); - if (group == NULL) - return 0; - if (EC_KEY_set_group(to->pkey.ec, group) == 0) - return 0; - EC_GROUP_free(group); - return 1; + return EC_KEY_set_group(to->pkey.ec, EC_KEY_get0_group(from->pkey.ec)); } static int diff --git a/src/lib/libssl/src/crypto/ec/ec_ameth.c b/src/lib/libssl/src/crypto/ec/ec_ameth.c index 6b4dfdec46..b4ae1c0623 100644 --- a/src/lib/libssl/src/crypto/ec/ec_ameth.c +++ b/src/lib/libssl/src/crypto/ec/ec_ameth.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ec_ameth.c,v 1.13 2014/07/13 15:47:51 logan Exp $ */ +/* $OpenBSD: ec_ameth.c,v 1.14 2014/10/07 04:58:50 miod Exp $ */ /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL * project 2006. */ @@ -393,13 +393,7 @@ ec_missing_parameters(const EVP_PKEY * pkey) static int ec_copy_parameters(EVP_PKEY * to, const EVP_PKEY * from) { - EC_GROUP *group = EC_GROUP_dup(EC_KEY_get0_group(from->pkey.ec)); - if (group == NULL) - return 0; - if (EC_KEY_set_group(to->pkey.ec, group) == 0) - return 0; - EC_GROUP_free(group); - return 1; + return EC_KEY_set_group(to->pkey.ec, EC_KEY_get0_group(from->pkey.ec)); } static int -- cgit v1.2.3-55-g6feb