diff options
author | tb@openbsd.org <tb@openbsd.org> | 2024-12-03 16:53:51 +0100 |
---|---|---|
committer | Damien Miller <djm@mindrot.org> | 2024-12-04 15:28:46 +0100 |
commit | 30c746265ebde29806dba77c92fb1fd3803cbf5c (patch) | |
tree | 8b575cd22ea7ec11cd805dc568516309a8fc27c7 | |
parent | don't ignore changes in regress Makefiles (diff) | |
download | openssh-30c746265ebde29806dba77c92fb1fd3803cbf5c.tar.xz openssh-30c746265ebde29806dba77c92fb1fd3803cbf5c.zip |
upstream: Remove redundant field of definition check
This will allow us to get rid of EC_GROUP_method_of() in the near future.
ok djm
OpenBSD-Commit-ID: b4a3d2e00990cf5c2ec6881c21ddca67327c2df8
-rw-r--r-- | sshkey.c | 15 |
1 files changed, 1 insertions, 14 deletions
@@ -1,4 +1,4 @@ -/* $OpenBSD: sshkey.c,v 1.147 2024/09/22 12:56:21 jsg Exp $ */ +/* $OpenBSD: sshkey.c,v 1.148 2024/12/03 15:53:51 tb Exp $ */ /* * Copyright (c) 2000, 2001 Markus Friedl. All rights reserved. * Copyright (c) 2008 Alexander von Gernler. All rights reserved. @@ -2705,14 +2705,6 @@ sshkey_ec_validate_public(const EC_GROUP *group, const EC_POINT *public) * EC_POINT_oct2point then the caller will need to explicitly check. */ - /* - * We shouldn't ever hit this case because bignum_get_ecpoint() - * refuses to load GF2m points. - */ - if (EC_METHOD_get_field_type(EC_GROUP_method_of(group)) != - NID_X9_62_prime_field) - goto out; - /* Q != infinity */ if (EC_POINT_is_at_infinity(group, public)) goto out; @@ -2812,11 +2804,6 @@ sshkey_dump_ec_point(const EC_GROUP *group, const EC_POINT *point) fprintf(stderr, "%s: BN_new failed\n", __func__); goto out; } - if (EC_METHOD_get_field_type(EC_GROUP_method_of(group)) != - NID_X9_62_prime_field) { - fprintf(stderr, "%s: group is not a prime field\n", __func__); - goto out; - } if (EC_POINT_get_affine_coordinates_GFp(group, point, x, y, NULL) != 1) { fprintf(stderr, "%s: EC_POINT_get_affine_coordinates_GFp\n", |