generic-library/openssl
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

free NULL cleanup.

Browse Source 
This gets EC_GROUP_clear_free EC_GROUP_free, EC_KEY_free,
EC_POINT_clear_free, EC_POINT_free

Reviewed-by: Kurt Roeckx <kurt@openssl.org>
This commit is contained in:
Rich Salz 2015-03-25 18:35:24 -04:00
parent 17dd65e6e1
commit 8fdc3734c0
30 changed files with 103 additions and 186 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
apps/ec.c
View File

@ -345,7 +345,6 @@ int MAIN(int argc, char **argv)
end: end:
BIO_free(in); BIO_free(in);
BIO_free_all(out); BIO_free_all(out);
if (eckey)
EC_KEY_free(eckey); EC_KEY_free(eckey);
if (passin) if (passin)
OPENSSL_free(passin); OPENSSL_free(passin);

1
apps/ecparam.c
View File

@ -623,7 +623,6 @@ int MAIN(int argc, char **argv)
OPENSSL_free(buffer); OPENSSL_free(buffer);
BIO_free(in); BIO_free(in);
BIO_free_all(out); BIO_free_all(out);
if (group != NULL)
EC_GROUP_free(group); EC_GROUP_free(group);
apps_shutdown(); apps_shutdown();
OPENSSL_EXIT(ret); OPENSSL_EXIT(ret);

3
apps/speed.c
View File

@ -2457,12 +2457,9 @@ int MAIN(int argc, char **argv)
#ifndef OPENSSL_NO_EC #ifndef OPENSSL_NO_EC
for (i = 0; i < EC_NUM; i++) for (i = 0; i < EC_NUM; i++)
if (ecdsa[i] != NULL)
EC_KEY_free(ecdsa[i]); EC_KEY_free(ecdsa[i]);
for (i = 0; i < EC_NUM; i++) { for (i = 0; i < EC_NUM; i++) {
if (ecdh_a[i] != NULL)
EC_KEY_free(ecdh_a[i]); EC_KEY_free(ecdh_a[i]);
if (ecdh_b[i] != NULL)
EC_KEY_free(ecdh_b[i]); EC_KEY_free(ecdh_b[i]);
} }
#endif #endif

2
crypto/ec/ec2_mult.c
View File

@ -437,9 +437,7 @@ int ec_GF2m_simple_mul(const EC_GROUP *group, EC_POINT *r,
ret = 1; ret = 1;
err: err:
if (p)
EC_POINT_free(p); EC_POINT_free(p);
if (acc)
EC_POINT_free(acc); EC_POINT_free(acc);
if (new_ctx != NULL) if (new_ctx != NULL)
BN_CTX_free(new_ctx); BN_CTX_free(new_ctx);

4
crypto/ec/ec_ameth.c
View File

@ -176,7 +176,6 @@ static EC_KEY *eckey_type2param(int ptype, void *pval)
return eckey; return eckey;
ecerr: ecerr:
if (eckey)
EC_KEY_free(eckey); EC_KEY_free(eckey);
return NULL; return NULL;
} }
@ -210,7 +209,6 @@ static int eckey_pub_decode(EVP_PKEY *pkey, X509_PUBKEY *pubkey)
return 1; return 1;
ecerr: ecerr:
if (eckey)
EC_KEY_free(eckey); EC_KEY_free(eckey);
return 0; return 0;
} }
@ -292,7 +290,6 @@ static int eckey_priv_decode(EVP_PKEY *pkey, PKCS8_PRIV_KEY_INFO *p8)
ecliberr: ecliberr:
ECerr(EC_F_ECKEY_PRIV_DECODE, ERR_R_EC_LIB); ECerr(EC_F_ECKEY_PRIV_DECODE, ERR_R_EC_LIB);
ecerr: ecerr:
if (eckey)
EC_KEY_free(eckey); EC_KEY_free(eckey);
return 0; return 0;
} }
@ -706,7 +703,6 @@ static int ecdh_cms_set_peerkey(EVP_PKEY_CTX *pctx,
if (EVP_PKEY_derive_set_peer(pctx, pkpeer) > 0) if (EVP_PKEY_derive_set_peer(pctx, pkpeer) > 0)
rv = 1; rv = 1;
err: err:
if (ecpeer)
EC_KEY_free(ecpeer); EC_KEY_free(ecpeer);
if (pkpeer) if (pkpeer)
EVP_PKEY_free(pkpeer); EVP_PKEY_free(pkpeer);

13
crypto/ec/ec_asn1.c
View File

@ -910,8 +910,8 @@ static EC_GROUP *ec_asn1_parameters2group(const ECPARAMETERS *params)
ok = 1; ok = 1;
err:if (!ok) { err:
if (ret) if (!ok) {
EC_GROUP_clear_free(ret); EC_GROUP_clear_free(ret);
ret = NULL; ret = NULL;
} }
@ -922,7 +922,6 @@ static EC_GROUP *ec_asn1_parameters2group(const ECPARAMETERS *params)
BN_free(a); BN_free(a);
if (b) if (b)
BN_free(b); BN_free(b);
if (point)
EC_POINT_free(point); EC_POINT_free(point);
return (ret); return (ret);
} }
@ -982,10 +981,10 @@ EC_GROUP *d2i_ECPKParameters(EC_GROUP **a, const unsigned char **in, long len)
return NULL; return NULL;
} }
if (a && *a) if (a) {
EC_GROUP_clear_free(*a); EC_GROUP_clear_free(*a);
if (a)
*a = group; *a = group;
}
ECPKPARAMETERS_free(params); ECPKPARAMETERS_free(params);
return (group); return (group);
@ -1030,7 +1029,6 @@ EC_KEY *d2i_ECPrivateKey(EC_KEY **a, const unsigned char **in, long len)
ret = *a; ret = *a;
if (priv_key->parameters) { if (priv_key->parameters) {
if (ret->group)
EC_GROUP_clear_free(ret->group); EC_GROUP_clear_free(ret->group);
ret->group = ec_asn1_pkparameters2group(priv_key->parameters); ret->group = ec_asn1_pkparameters2group(priv_key->parameters);
} }
@ -1055,7 +1053,6 @@ EC_KEY *d2i_ECPrivateKey(EC_KEY **a, const unsigned char **in, long len)
goto err; goto err;
} }
if (ret->pub_key)
EC_POINT_clear_free(ret->pub_key); EC_POINT_clear_free(ret->pub_key);
ret->pub_key = EC_POINT_new(ret->group); ret->pub_key = EC_POINT_new(ret->group);
if (ret->pub_key == NULL) { if (ret->pub_key == NULL) {
@ -1098,7 +1095,7 @@ EC_KEY *d2i_ECPrivateKey(EC_KEY **a, const unsigned char **in, long len)
ok = 1; ok = 1;
err: err:
if (!ok) { if (!ok) {
if (ret && (a == NULL || *a != ret)) if (a == NULL || *a != ret)
EC_KEY_free(ret); EC_KEY_free(ret);
ret = NULL; ret = NULL;
} }

1
crypto/ec/ec_check.c
View File

@ -114,7 +114,6 @@ int EC_GROUP_check(const EC_GROUP *group, BN_CTX *ctx)
BN_CTX_end(ctx); BN_CTX_end(ctx);
if (new_ctx != NULL) if (new_ctx != NULL)
BN_CTX_free(new_ctx); BN_CTX_free(new_ctx);
if (point)
EC_POINT_free(point); EC_POINT_free(point);
return ret; return ret;
} }

1
crypto/ec/ec_curve.c
View File

@ -3114,7 +3114,6 @@ static EC_GROUP *ec_group_new_from_data(const ec_list_element curve)
EC_GROUP_free(group); EC_GROUP_free(group);
group = NULL; group = NULL;
} }
if (P)
EC_POINT_free(P); EC_POINT_free(P);
if (ctx) if (ctx)
BN_CTX_free(ctx); BN_CTX_free(ctx);

10
crypto/ec/ec_key.c
View File

@ -120,9 +120,7 @@ void EC_KEY_free(EC_KEY *r)
} }
#endif #endif
if (r->group != NULL)
EC_GROUP_free(r->group); EC_GROUP_free(r->group);
if (r->pub_key != NULL)
EC_POINT_free(r->pub_key); EC_POINT_free(r->pub_key);
if (r->priv_key != NULL) if (r->priv_key != NULL)
BN_clear_free(r->priv_key); BN_clear_free(r->priv_key);
@ -146,7 +144,6 @@ EC_KEY *EC_KEY_copy(EC_KEY *dest, const EC_KEY *src)
if (src->group) { if (src->group) {
const EC_METHOD *meth = EC_GROUP_method_of(src->group); const EC_METHOD *meth = EC_GROUP_method_of(src->group);
/* clear the old group */ /* clear the old group */
if (dest->group)
EC_GROUP_free(dest->group); EC_GROUP_free(dest->group);
dest->group = EC_GROUP_new(meth); dest->group = EC_GROUP_new(meth);
if (dest->group == NULL) if (dest->group == NULL)
@ -156,7 +153,6 @@ EC_KEY *EC_KEY_copy(EC_KEY *dest, const EC_KEY *src)
} }
/* copy the public key */ /* copy the public key */
if (src->pub_key && src->group) { if (src->pub_key && src->group) {
if (dest->pub_key)
EC_POINT_free(dest->pub_key); EC_POINT_free(dest->pub_key);
dest->pub_key = EC_POINT_new(src->group); dest->pub_key = EC_POINT_new(src->group);
if (dest->pub_key == NULL) if (dest->pub_key == NULL)
@ -274,7 +270,7 @@ int EC_KEY_generate_key(EC_KEY *eckey)
err: err:
if (order) if (order)
BN_free(order); BN_free(order);
if (pub_key != NULL && eckey->pub_key == NULL) if (eckey->pub_key == NULL)
EC_POINT_free(pub_key); EC_POINT_free(pub_key);
if (priv_key != NULL && eckey->priv_key == NULL) if (priv_key != NULL && eckey->priv_key == NULL)
BN_free(priv_key); BN_free(priv_key);
@ -347,7 +343,6 @@ int EC_KEY_check_key(const EC_KEY *eckey)
err: err:
if (ctx != NULL) if (ctx != NULL)
BN_CTX_free(ctx); BN_CTX_free(ctx);
if (point != NULL)
EC_POINT_free(point); EC_POINT_free(point);
return (ok); return (ok);
} }
@ -422,7 +417,6 @@ int EC_KEY_set_public_key_affine_coordinates(EC_KEY *key, BIGNUM *x,
err: err:
if (ctx) if (ctx)
BN_CTX_free(ctx); BN_CTX_free(ctx);
if (point)
EC_POINT_free(point); EC_POINT_free(point);
return ok; return ok;
@ -435,7 +429,6 @@ const EC_GROUP *EC_KEY_get0_group(const EC_KEY *key)
int EC_KEY_set_group(EC_KEY *key, const EC_GROUP *group) int EC_KEY_set_group(EC_KEY *key, const EC_GROUP *group)
{ {
if (key->group != NULL)
EC_GROUP_free(key->group); EC_GROUP_free(key->group);
key->group = EC_GROUP_dup(group); key->group = EC_GROUP_dup(group);
return (key->group == NULL) ? 0 : 1; return (key->group == NULL) ? 0 : 1;
@ -461,7 +454,6 @@ const EC_POINT *EC_KEY_get0_public_key(const EC_KEY *key)
int EC_KEY_set_public_key(EC_KEY *key, const EC_POINT *pub_key) int EC_KEY_set_public_key(EC_KEY *key, const EC_POINT *pub_key)
{ {
if (key->pub_key != NULL)
EC_POINT_free(key->pub_key); EC_POINT_free(key->pub_key);
key->pub_key = EC_POINT_dup(pub_key, key->group); key->pub_key = EC_POINT_dup(pub_key, key->group);
return (key->pub_key == NULL) ? 0 : 1; return (key->pub_key == NULL) ? 0 : 1;

9
crypto/ec/ec_lib.c
View File

@ -138,7 +138,6 @@ void EC_GROUP_free(EC_GROUP *group)
if (group->mont_data) if (group->mont_data)
BN_MONT_CTX_free(group->mont_data); BN_MONT_CTX_free(group->mont_data);
if (group->generator != NULL)
EC_POINT_free(group->generator); EC_POINT_free(group->generator);
BN_free(group->order); BN_free(group->order);
BN_free(group->cofactor); BN_free(group->cofactor);
@ -164,7 +163,6 @@ void EC_GROUP_clear_free(EC_GROUP *group)
if (group->mont_data) if (group->mont_data)
BN_MONT_CTX_free(group->mont_data); BN_MONT_CTX_free(group->mont_data);
if (group->generator != NULL)
EC_POINT_clear_free(group->generator); EC_POINT_clear_free(group->generator);
BN_clear_free(group->order); BN_clear_free(group->order);
BN_clear_free(group->cofactor); BN_clear_free(group->cofactor);
@ -232,11 +230,9 @@ int EC_GROUP_copy(EC_GROUP *dest, const EC_GROUP *src)
return 0; return 0;
} else { } else {
/* src->generator == NULL */ /* src->generator == NULL */
if (dest->generator != NULL) {
EC_POINT_clear_free(dest->generator); EC_POINT_clear_free(dest->generator);
dest->generator = NULL; dest->generator = NULL;
} }
}
if (!BN_copy(dest->order, src->order)) if (!BN_copy(dest->order, src->order))
return 0; return 0;
@ -283,10 +279,9 @@ EC_GROUP *EC_GROUP_dup(const EC_GROUP *a)
err: err:
if (!ok) { if (!ok) {
if (t)
EC_GROUP_free(t); EC_GROUP_free(t);
return NULL; return NULL;
} else }
return t; return t;
} }
@ -790,7 +785,7 @@ EC_POINT *EC_POINT_dup(const EC_POINT *a, const EC_GROUP *group)
if (!r) { if (!r) {
EC_POINT_free(t); EC_POINT_free(t);
return NULL; return NULL;
} else }
return t; return t;
} }

3
crypto/ec/ec_mult.c
View File

@ -556,7 +556,6 @@ int ec_wNAF_mul(const EC_GROUP *group, EC_POINT *r, const BIGNUM *scalar,
err: err:
if (new_ctx != NULL) if (new_ctx != NULL)
BN_CTX_free(new_ctx); BN_CTX_free(new_ctx);
if (tmp != NULL)
EC_POINT_free(tmp); EC_POINT_free(tmp);
if (wsize != NULL) if (wsize != NULL)
OPENSSL_free(wsize); OPENSSL_free(wsize);
@ -759,9 +758,7 @@ int ec_wNAF_precompute_mult(EC_GROUP *group, BN_CTX *ctx)
EC_POINT_free(*p); EC_POINT_free(*p);
OPENSSL_free(points); OPENSSL_free(points);
} }
if (tmp_point)
EC_POINT_free(tmp_point); EC_POINT_free(tmp_point);
if (base)
EC_POINT_free(base); EC_POINT_free(base);
return ret; return ret;
} }

5
crypto/ec/ec_pmeth.c
View File

@ -146,9 +146,7 @@ static void pkey_ec_cleanup(EVP_PKEY_CTX *ctx)
{ {
EC_PKEY_CTX *dctx = ctx->data; EC_PKEY_CTX *dctx = ctx->data;
if (dctx) { if (dctx) {
if (dctx->gen_group)
EC_GROUP_free(dctx->gen_group); EC_GROUP_free(dctx->gen_group);
if (dctx->co_key)
EC_KEY_free(dctx->co_key); EC_KEY_free(dctx->co_key);
if (dctx->kdf_ukm) if (dctx->kdf_ukm)
OPENSSL_free(dctx->kdf_ukm); OPENSSL_free(dctx->kdf_ukm);
@ -289,7 +287,6 @@ static int pkey_ec_ctrl(EVP_PKEY_CTX *ctx, int type, int p1, void *p2)
ECerr(EC_F_PKEY_EC_CTRL, EC_R_INVALID_CURVE); ECerr(EC_F_PKEY_EC_CTRL, EC_R_INVALID_CURVE);
return 0; return 0;
} }
if (dctx->gen_group)
EC_GROUP_free(dctx->gen_group); EC_GROUP_free(dctx->gen_group);
dctx->gen_group = group; dctx->gen_group = group;
return 1; return 1;
@ -331,7 +328,7 @@ static int pkey_ec_ctrl(EVP_PKEY_CTX *ctx, int type, int p1, void *p2)
EC_KEY_set_flags(dctx->co_key, EC_FLAG_COFACTOR_ECDH); EC_KEY_set_flags(dctx->co_key, EC_FLAG_COFACTOR_ECDH);
else else
EC_KEY_clear_flags(dctx->co_key, EC_FLAG_COFACTOR_ECDH); EC_KEY_clear_flags(dctx->co_key, EC_FLAG_COFACTOR_ECDH);
} else if (dctx->co_key) { } else {
EC_KEY_free(dctx->co_key); EC_KEY_free(dctx->co_key);
dctx->co_key = NULL; dctx->co_key = NULL;
} }

2
crypto/ec/ec_print.c
View File

@ -110,7 +110,7 @@ EC_POINT *EC_POINT_bn2point(const EC_GROUP *group,
ret = point; ret = point;
if (!EC_POINT_oct2point(group, ret, buf, buf_len, ctx)) { if (!EC_POINT_oct2point(group, ret, buf, buf_len, ctx)) {
if (point == NULL) if (ret != point)
EC_POINT_clear_free(ret); EC_POINT_clear_free(ret);
OPENSSL_free(buf); OPENSSL_free(buf);
return NULL; return NULL;

2
crypto/ec/ecp_nistp224.c
View File

@ -1602,7 +1602,6 @@ int ec_GFp_nistp224_points_mul(const EC_GROUP *group, EC_POINT *r,
err: err:
BN_CTX_end(ctx); BN_CTX_end(ctx);
if (generator != NULL)
EC_POINT_free(generator); EC_POINT_free(generator);
if (new_ctx != NULL) if (new_ctx != NULL)
BN_CTX_free(new_ctx); BN_CTX_free(new_ctx);
@ -1739,7 +1738,6 @@ int ec_GFp_nistp224_precompute_mult(EC_GROUP *group, BN_CTX *ctx)
pre = NULL; pre = NULL;
err: err:
BN_CTX_end(ctx); BN_CTX_end(ctx);
if (generator != NULL)
EC_POINT_free(generator); EC_POINT_free(generator);
if (new_ctx != NULL) if (new_ctx != NULL)
BN_CTX_free(new_ctx); BN_CTX_free(new_ctx);

2
crypto/ec/ecp_nistp256.c
View File

@ -2193,7 +2193,6 @@ int ec_GFp_nistp256_points_mul(const EC_GROUP *group, EC_POINT *r,
err: err:
BN_CTX_end(ctx); BN_CTX_end(ctx);
if (generator != NULL)
EC_POINT_free(generator); EC_POINT_free(generator);
if (new_ctx != NULL) if (new_ctx != NULL)
BN_CTX_free(new_ctx); BN_CTX_free(new_ctx);
@ -2340,7 +2339,6 @@ int ec_GFp_nistp256_precompute_mult(EC_GROUP *group, BN_CTX *ctx)
pre = NULL; pre = NULL;
err: err:
BN_CTX_end(ctx); BN_CTX_end(ctx);
if (generator != NULL)
EC_POINT_free(generator); EC_POINT_free(generator);
if (new_ctx != NULL) if (new_ctx != NULL)
BN_CTX_free(new_ctx); BN_CTX_free(new_ctx);

2
crypto/ec/ecp_nistp521.c
View File

@ -2006,7 +2006,6 @@ int ec_GFp_nistp521_points_mul(const EC_GROUP *group, EC_POINT *r,
err: err:
BN_CTX_end(ctx); BN_CTX_end(ctx);
if (generator != NULL)
EC_POINT_free(generator); EC_POINT_free(generator);
if (new_ctx != NULL) if (new_ctx != NULL)
BN_CTX_free(new_ctx); BN_CTX_free(new_ctx);
@ -2123,7 +2122,6 @@ int ec_GFp_nistp521_precompute_mult(EC_GROUP *group, BN_CTX *ctx)
pre = NULL; pre = NULL;
err: err:
BN_CTX_end(ctx); BN_CTX_end(ctx);
if (generator != NULL)
EC_POINT_free(generator); EC_POINT_free(generator);
if (new_ctx != NULL) if (new_ctx != NULL)
BN_CTX_free(new_ctx); BN_CTX_free(new_ctx);

2
crypto/ec/ecp_nistz256.c
View File

@ -874,9 +874,7 @@ static int ecp_nistz256_mult_precompute(EC_GROUP *group, BN_CTX *ctx)
ecp_nistz256_pre_comp_free(pre_comp); ecp_nistz256_pre_comp_free(pre_comp);
if (precomp_storage) if (precomp_storage)
OPENSSL_free(precomp_storage); OPENSSL_free(precomp_storage);
if (P)
EC_POINT_free(P); EC_POINT_free(P);
if (T)
EC_POINT_free(T); EC_POINT_free(T);
return ret; return ret;
} }

16
crypto/ec/ectest.c
View File

@ -864,17 +864,11 @@ static void prime_field_tests(void)
BN_free(y); BN_free(y);
BN_free(z); BN_free(z);
if (P_160)
EC_GROUP_free(P_160); EC_GROUP_free(P_160);
if (P_192)
EC_GROUP_free(P_192); EC_GROUP_free(P_192);
if (P_224)
EC_GROUP_free(P_224); EC_GROUP_free(P_224);
if (P_256)
EC_GROUP_free(P_256); EC_GROUP_free(P_256);
if (P_384)
EC_GROUP_free(P_384); EC_GROUP_free(P_384);
if (P_521)
EC_GROUP_free(P_521); EC_GROUP_free(P_521);
} }
@ -1346,25 +1340,15 @@ static void char2_field_tests(void)
BN_free(z); BN_free(z);
BN_free(cof); BN_free(cof);
if (C2_K163)
EC_GROUP_free(C2_K163); EC_GROUP_free(C2_K163);
if (C2_B163)
EC_GROUP_free(C2_B163); EC_GROUP_free(C2_B163);
if (C2_K233)
EC_GROUP_free(C2_K233); EC_GROUP_free(C2_K233);
if (C2_B233)
EC_GROUP_free(C2_B233); EC_GROUP_free(C2_B233);
if (C2_K283)
EC_GROUP_free(C2_K283); EC_GROUP_free(C2_K283);
if (C2_B283)
EC_GROUP_free(C2_B283); EC_GROUP_free(C2_B283);
if (C2_K409)
EC_GROUP_free(C2_K409); EC_GROUP_free(C2_K409);
if (C2_B409)
EC_GROUP_free(C2_B409); EC_GROUP_free(C2_B409);
if (C2_K571)
EC_GROUP_free(C2_K571); EC_GROUP_free(C2_K571);
if (C2_B571)
EC_GROUP_free(C2_B571); EC_GROUP_free(C2_B571);
} }

6
crypto/ecdh/ecdhtest.c
View File

@ -290,9 +290,7 @@ static int test_ecdh_curve(int nid, const char *text, BN_CTX *ctx, BIO *out)
BN_free(x_b); BN_free(x_b);
if (y_b) if (y_b)
BN_free(y_b); BN_free(y_b);
if (b)
EC_KEY_free(b); EC_KEY_free(b);
if (a)
EC_KEY_free(a); EC_KEY_free(a);
return (ret); return (ret);
} }
@ -394,11 +392,9 @@ static EC_KEY *mk_eckey(int nid, const unsigned char *p, size_t plen)
err: err:
if (priv) if (priv)
BN_clear_free(priv); BN_clear_free(priv);
if (pub)
EC_POINT_free(pub); EC_POINT_free(pub);
if (ok) if (ok)
return k; return k;
else if (k)
EC_KEY_free(k); EC_KEY_free(k);
return NULL; return NULL;
} }
@ -440,9 +436,7 @@ static int ecdh_kat(BIO *out, const char *cname, int nid,
goto err; goto err;
rv = 1; rv = 1;
err: err:
if (key1)
EC_KEY_free(key1); EC_KEY_free(key1);
if (key2)
EC_KEY_free(key2); EC_KEY_free(key2);
if (Ztmp) if (Ztmp)
OPENSSL_free(Ztmp); OPENSSL_free(Ztmp);

1
crypto/ecdh/ech_ossl.c
View File

@ -202,7 +202,6 @@ static int ecdh_compute_key(void *out, size_t outlen, const EC_POINT *pub_key,
} }
err: err:
if (tmp)
EC_POINT_free(tmp); EC_POINT_free(tmp);
if (ctx) if (ctx)
BN_CTX_end(ctx); BN_CTX_end(ctx);

6
crypto/ecdsa/ecdsatest.c
View File

@ -240,7 +240,6 @@ int x9_62_test_internal(BIO *out, int nid, const char *r_in, const char *s_in)
x962_int_err: x962_int_err:
if (!ret) if (!ret)
BIO_printf(out, " failed\n"); BIO_printf(out, " failed\n");
if (key)
EC_KEY_free(key); EC_KEY_free(key);
if (signature) if (signature)
ECDSA_SIG_free(signature); ECDSA_SIG_free(signature);
@ -354,9 +353,8 @@ int test_builtin(BIO *out)
goto builtin_err; goto builtin_err;
EC_GROUP_free(group); EC_GROUP_free(group);
degree = EC_GROUP_get_degree(EC_KEY_get0_group(eckey)); degree = EC_GROUP_get_degree(EC_KEY_get0_group(eckey));
if (degree < 160) if (degree < 160) {
/* drop the curve */ /* drop the curve */
{
EC_KEY_free(eckey); EC_KEY_free(eckey);
eckey = NULL; eckey = NULL;
continue; continue;
@ -506,9 +504,7 @@ int test_builtin(BIO *out)
ret = 1; ret = 1;
builtin_err: builtin_err:
if (eckey)
EC_KEY_free(eckey); EC_KEY_free(eckey);
if (wrong_eckey)
EC_KEY_free(wrong_eckey); EC_KEY_free(wrong_eckey);
if (ecdsa_sig) if (ecdsa_sig)
ECDSA_SIG_free(ecdsa_sig); ECDSA_SIG_free(ecdsa_sig);

2
crypto/ecdsa/ecs_ossl.c
View File

@ -242,7 +242,6 @@ static int ecdsa_sign_setup(EC_KEY *eckey, BN_CTX *ctx_in,
BN_CTX_free(ctx); BN_CTX_free(ctx);
if (order != NULL) if (order != NULL)
BN_free(order); BN_free(order);
if (tmp_point != NULL)
EC_POINT_free(tmp_point); EC_POINT_free(tmp_point);
if (X) if (X)
BN_clear_free(X); BN_clear_free(X);
@ -475,7 +474,6 @@ static int ecdsa_do_verify(const unsigned char *dgst, int dgst_len,
err: err:
BN_CTX_end(ctx); BN_CTX_end(ctx);
BN_CTX_free(ctx); BN_CTX_free(ctx);
if (point)
EC_POINT_free(point); EC_POINT_free(point);
return ret; return ret;
} }

2
doc/crypto/EC_GROUP_new.pod
View File

@ -75,8 +75,10 @@ In order to construct a builtin curve use the function EC_GROUP_new_by_curve_nam
be constructed. be constructed.
EC_GROUP_free frees the memory associated with the EC_GROUP. EC_GROUP_free frees the memory associated with the EC_GROUP.
If B<group> is NULL nothing is done.
EC_GROUP_clear_free destroys any sensitive data held within the EC_GROUP and then frees its memory. EC_GROUP_clear_free destroys any sensitive data held within the EC_GROUP and then frees its memory.
If B<group> is NULL nothing is done.
=head1 RETURN VALUES =head1 RETURN VALUES

1
doc/crypto/EC_KEY_new.pod
View File

@ -47,6 +47,7 @@ EC_GROUP_new_by_curve_name.
Calling EC_KEY_free decrements the reference count for the EC_KEY object, and if it has dropped to zero then frees the memory associated Calling EC_KEY_free decrements the reference count for the EC_KEY object, and if it has dropped to zero then frees the memory associated
with it. with it.
If B<key> is NULL nothing is done.
EC_KEY_copy copies the contents of the EC_KEY in B<src> into B<dest>. EC_KEY_copy copies the contents of the EC_KEY in B<src> into B<dest>.

2
doc/crypto/EC_POINT_new.pod
View File

@ -53,8 +53,10 @@ An EC_POINT represents a point on a curve. A new point is constructed by calling
object that the point relates to. object that the point relates to.
EC_POINT_free frees the memory associated with the EC_POINT. EC_POINT_free frees the memory associated with the EC_POINT.
if B<point> is NULL nothing is done.
EC_POINT_clear_free destroys any sensitive data held within the EC_POINT and then frees its memory. EC_POINT_clear_free destroys any sensitive data held within the EC_POINT and then frees its memory.
if B<point> is NULL nothing is done.
EC_POINT_copy copies the point B<src> into B<dst>. Both B<src> and B<dst> must use the same EC_METHOD. EC_POINT_copy copies the point B<src> into B<dst>. Both B<src> and B<dst> must use the same EC_METHOD.

2
engines/ccgost/gost_ameth.c
View File

@ -284,9 +284,7 @@ static void pkey_free_gost94(EVP_PKEY *key)
static void pkey_free_gost01(EVP_PKEY *key) static void pkey_free_gost01(EVP_PKEY *key)
{ {
if (key->pkey.ec) {
EC_KEY_free(key->pkey.ec); EC_KEY_free(key->pkey.ec);
}
} }
/* ------------------ private key functions -----------------------------*/ /* ------------------ private key functions -----------------------------*/

5
ssl/s3_clnt.c
View File

@ -1388,10 +1388,8 @@ int ssl3_get_key_exchange(SSL *s)
s->session->sess_cert->peer_dh_tmp = NULL; s->session->sess_cert->peer_dh_tmp = NULL;
#endif #endif
#ifndef OPENSSL_NO_EC #ifndef OPENSSL_NO_EC
if (s->session->sess_cert->peer_ecdh_tmp) {
EC_KEY_free(s->session->sess_cert->peer_ecdh_tmp); EC_KEY_free(s->session->sess_cert->peer_ecdh_tmp);
s->session->sess_cert->peer_ecdh_tmp = NULL; s->session->sess_cert->peer_ecdh_tmp = NULL;
}
#endif #endif
} else { } else {
s->session->sess_cert = ssl_sess_cert_new(); s->session->sess_cert = ssl_sess_cert_new();
@ -1959,7 +1957,6 @@ int ssl3_get_key_exchange(SSL *s)
#ifndef OPENSSL_NO_EC #ifndef OPENSSL_NO_EC
BN_CTX_free(bn_ctx); BN_CTX_free(bn_ctx);
EC_POINT_free(srvr_ecpoint); EC_POINT_free(srvr_ecpoint);
if (ecdh != NULL)
EC_KEY_free(ecdh); EC_KEY_free(ecdh);
#endif #endif
EVP_MD_CTX_cleanup(&md_ctx); EVP_MD_CTX_cleanup(&md_ctx);
@ -2760,7 +2757,6 @@ int ssl3_send_client_key_exchange(SSL *s)
BN_CTX_free(bn_ctx); BN_CTX_free(bn_ctx);
if (encodedPoint != NULL) if (encodedPoint != NULL)
OPENSSL_free(encodedPoint); OPENSSL_free(encodedPoint);
if (clnt_ecdh != NULL)
EC_KEY_free(clnt_ecdh); EC_KEY_free(clnt_ecdh);
EVP_PKEY_free(srvr_pub_pkey); EVP_PKEY_free(srvr_pub_pkey);
} }
@ -3069,7 +3065,6 @@ int ssl3_send_client_key_exchange(SSL *s)
BN_CTX_free(bn_ctx); BN_CTX_free(bn_ctx);
if (encodedPoint != NULL) if (encodedPoint != NULL)
OPENSSL_free(encodedPoint); OPENSSL_free(encodedPoint);
if (clnt_ecdh != NULL)
EC_KEY_free(clnt_ecdh); EC_KEY_free(clnt_ecdh);
EVP_PKEY_free(srvr_pub_pkey); EVP_PKEY_free(srvr_pub_pkey);
#endif #endif

6
ssl/s3_lib.c
View File

@ -3141,7 +3141,6 @@ void ssl3_free(SSL *s)
DH_free(s->s3->tmp.dh); DH_free(s->s3->tmp.dh);
#endif #endif
#ifndef OPENSSL_NO_EC #ifndef OPENSSL_NO_EC
if (s->s3->tmp.ecdh != NULL)
EC_KEY_free(s->s3->tmp.ecdh); EC_KEY_free(s->s3->tmp.ecdh);
#endif #endif
@ -3182,10 +3181,8 @@ void ssl3_clear(SSL *s)
s->s3->tmp.dh = NULL; s->s3->tmp.dh = NULL;
#endif #endif
#ifndef OPENSSL_NO_EC #ifndef OPENSSL_NO_EC
if (s->s3->tmp.ecdh != NULL) {
EC_KEY_free(s->s3->tmp.ecdh); EC_KEY_free(s->s3->tmp.ecdh);
s->s3->tmp.ecdh = NULL; s->s3->tmp.ecdh = NULL;
}
#endif #endif
#ifndef OPENSSL_NO_TLSEXT #ifndef OPENSSL_NO_TLSEXT
# ifndef OPENSSL_NO_EC # ifndef OPENSSL_NO_EC
@ -3356,7 +3353,6 @@ long ssl3_ctrl(SSL *s, int cmd, long larg, void *parg)
return (ret); return (ret);
} }
} }
if (s->cert->ecdh_tmp != NULL)
EC_KEY_free(s->cert->ecdh_tmp); EC_KEY_free(s->cert->ecdh_tmp);
s->cert->ecdh_tmp = ecdh; s->cert->ecdh_tmp = ecdh;
ret = 1; ret = 1;
@ -3829,9 +3825,7 @@ long ssl3_ctx_ctrl(SSL_CTX *ctx, int cmd, long larg, void *parg)
} }
} }
if (cert->ecdh_tmp != NULL) {
EC_KEY_free(cert->ecdh_tmp); EC_KEY_free(cert->ecdh_tmp);
}
cert->ecdh_tmp = ecdh; cert->ecdh_tmp = ecdh;
return 1; return 1;
} }

1
ssl/s3_srvr.c
View File

@ -2886,7 +2886,6 @@ int ssl3_get_client_key_exchange(SSL *s)
#ifndef OPENSSL_NO_EC #ifndef OPENSSL_NO_EC
EVP_PKEY_free(clnt_pub_pkey); EVP_PKEY_free(clnt_pub_pkey);
EC_POINT_free(clnt_ecpoint); EC_POINT_free(clnt_ecpoint);
if (srvr_ecdh != NULL)
EC_KEY_free(srvr_ecdh); EC_KEY_free(srvr_ecdh);
BN_CTX_free(bn_ctx); BN_CTX_free(bn_ctx);
#endif #endif

2
ssl/ssl_cert.c
View File

@ -449,7 +449,6 @@ void ssl_cert_free(CERT *c)
DH_free(c->dh_tmp); DH_free(c->dh_tmp);
#endif #endif
#ifndef OPENSSL_NO_EC #ifndef OPENSSL_NO_EC
if (c->ecdh_tmp)
EC_KEY_free(c->ecdh_tmp); EC_KEY_free(c->ecdh_tmp);
#endif #endif
@ -655,7 +654,6 @@ void ssl_sess_cert_free(SESS_CERT *sc)
DH_free(sc->peer_dh_tmp); DH_free(sc->peer_dh_tmp);
#endif #endif
#ifndef OPENSSL_NO_EC #ifndef OPENSSL_NO_EC
if (sc->peer_ecdh_tmp != NULL)
EC_KEY_free(sc->peer_ecdh_tmp); EC_KEY_free(sc->peer_ecdh_tmp);
#endif #endif