Remove EC_FLAGS_DEFAULT_OCT.

The EC code has an amazing array of function pointer hooks, such that a
method can hook into almost any operation... and then there is the
EC_FLAGS_DEFAULT_OCT flag, which adds a bunch of complex code and #ifdef
so you can avoid setting three of those function pointers!

Remove EC_FLAGS_DEFAULT_OCT, the now unused flags field from EC_METHOD,
along with the various code that was wrapped in EC_FLAGS_DEFAULT_OCT,
setting the three function pointers that need to be set in each of the
EC_METHODs.

ok beck@ tb@

1 files changed, 5 insertions, 2 deletions

diff --git a/src/lib/libcrypto/ec/ecp_nist.c b/src/lib/libcrypto/ec/ecp_nist.c
index 3a81a0e8d9..e3c13f7c65 100644
--- a/src/lib/libcrypto/ec/ecp_nist.c
+++ b/src/lib/libcrypto/ec/ecp_nist.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: ecp_nist.c,v 1.22 2023/03/07 05:45:14 jsing Exp $ */
+/* $OpenBSD: ecp_nist.c,v 1.23 2023/03/08 04:50:27 jsing Exp $ */
 /*
  * Written by Nils Larsch for the OpenSSL project.
  */
@@ -168,7 +168,6 @@ ec_GFp_nist_field_sqr(const EC_GROUP *group, BIGNUM *r, const BIGNUM *a,
 }
 
 static const EC_METHOD ec_GFp_nist_method = {
-        .flags = EC_FLAGS_DEFAULT_OCT,
         .field_type = NID_X9_62_prime_field,
         .group_init = ec_GFp_simple_group_init,
         .group_finish = ec_GFp_simple_group_finish,
@@ -192,6 +191,10 @@ static const EC_METHOD ec_GFp_nist_method = {
             ec_GFp_simple_point_set_affine_coordinates,
         .point_get_affine_coordinates =
             ec_GFp_simple_point_get_affine_coordinates,
+        .point_set_compressed_coordinates =
+            ec_GFp_simple_set_compressed_coordinates,
+        .point2oct = ec_GFp_simple_point2oct,
+        .oct2point = ec_GFp_simple_oct2point,
         .add = ec_GFp_simple_add,
         .dbl = ec_GFp_simple_dbl,
         .invert = ec_GFp_simple_invert,