Always clear EC groups and points on free.

Rather than sometimes clearing, turn the free functions into ones that
always clear (as we've done elsewhere). Turn the EC_GROUP_clear_free() and
EC_POINT_clear_free() functions into wrappers that call the *_free()
version. Do similar for the EC_METHOD implementations, removing the
group_clear_finish() and point_clear_finish() hooks in the process.

ok tb@

1 files changed, 1 insertions, 10 deletions

diff --git a/src/lib/libcrypto/ec/ecp_mont.c b/src/lib/libcrypto/ec/ecp_mont.c
index d0d497b011..8b85bf32fa 100644
--- a/src/lib/libcrypto/ec/ecp_mont.c
+++ b/src/lib/libcrypto/ec/ecp_mont.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: ecp_mont.c,v 1.26 2023/03/08 04:50:27 jsing Exp $ */
+/* $OpenBSD: ecp_mont.c,v 1.27 2023/03/08 05:45:31 jsing Exp $ */
 /*
  * Originally written by Bodo Moeller for the OpenSSL project.
  */
@@ -93,13 +93,6 @@ ec_GFp_mont_group_finish(EC_GROUP *group)
         ec_GFp_simple_group_finish(group);
 }
 
-static void
-ec_GFp_mont_group_clear_finish(EC_GROUP *group)
-{
-        ec_GFp_mont_group_clear(group);
-        ec_GFp_simple_group_clear_finish(group);
-}
-
 static int
 ec_GFp_mont_group_copy(EC_GROUP *dest, const EC_GROUP *src)
 {
@@ -236,7 +229,6 @@ static const EC_METHOD ec_GFp_mont_method = {
         .field_type = NID_X9_62_prime_field,
         .group_init = ec_GFp_mont_group_init,
         .group_finish = ec_GFp_mont_group_finish,
-        .group_clear_finish = ec_GFp_mont_group_clear_finish,
         .group_copy = ec_GFp_mont_group_copy,
         .group_set_curve = ec_GFp_mont_group_set_curve,
         .group_get_curve = ec_GFp_simple_group_get_curve,
@@ -245,7 +237,6 @@ static const EC_METHOD ec_GFp_mont_method = {
         .group_check_discriminant = ec_GFp_simple_group_check_discriminant,
         .point_init = ec_GFp_simple_point_init,
         .point_finish = ec_GFp_simple_point_finish,
-        .point_clear_finish = ec_GFp_simple_point_clear_finish,
         .point_copy = ec_GFp_simple_point_copy,
         .point_set_to_infinity = ec_GFp_simple_point_set_to_infinity,
         .point_set_Jprojective_coordinates =