Fix no-ec2m in ec_curve.c (1.1.0) OpenSSL_1_1_0-stable
authorNicola Tuveri <nic.tuv@gmail.com>
Wed, 11 Sep 2019 22:57:47 +0000 (01:57 +0300)
committerNicola Tuveri <nic.tuv@gmail.com>
Wed, 11 Sep 2019 23:08:27 +0000 (02:08 +0300)
I made a mistake in d4a5dac9f9242c580fb9d0a4389440eccd3494a7 and
inverted the GF2m and GFp calls in ec_point_get_affine_coordinates, this
fixes it.

Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/9873)

crypto/ec/ec_curve.c

index 2d28d7f70bb307bd59b2a266f50506b07f3528cf..6a58b3a23e0c949021cb2cfdbb827d97eebee322 100644 (file)
@@ -3200,11 +3200,11 @@ int ec_point_get_affine_coordinates(const EC_GROUP *group,
 
 #ifndef OPENSSL_NO_EC2M
     if (field_nid == NID_X9_62_characteristic_two_field) {
-        return EC_POINT_get_affine_coordinates_GFp(group, point, x, y, ctx);
+        return EC_POINT_get_affine_coordinates_GF2m(group, point, x, y, ctx);
     } else
 #endif /* !def(OPENSSL_NO_EC2M) */
     if (field_nid == NID_X9_62_prime_field) {
-        return EC_POINT_get_affine_coordinates_GF2m(group, point, x, y, ctx);
+        return EC_POINT_get_affine_coordinates_GFp(group, point, x, y, ctx);
     } else {
         /* this should never happen */
         return 0;