oops, revert mistakenly committed EC changes
authorDr. Stephen Henson <steve@openssl.org>
Wed, 19 Jan 2011 14:42:42 +0000 (14:42 +0000)
committerDr. Stephen Henson <steve@openssl.org>
Wed, 19 Jan 2011 14:42:42 +0000 (14:42 +0000)
crypto/ec/ec2_smpl.c
crypto/ec/ec_key.c
crypto/ec/ec_lcl.h
crypto/ec/ecp_smpl.c

index 00ad347b9ac828bc0dbd6e02ea10ea48eaf4d9ee..cf357b462a27bcda50ff8f600bdfcf9635dab1e6 100644 (file)
@@ -363,12 +363,7 @@ int ec_GF2m_simple_point_set_affine_coordinates(const EC_GROUP *group, EC_POINT
        if (!BN_copy(&point->Z, BN_value_one())) goto err;
        BN_set_negative(&point->Z, 0);
        point->Z_is_one = 1;
-       if (BN_num_bits(x) > BN_num_bits(&group->field))
-               ret = 2;
-       else if (BN_num_bits(y) > BN_num_bits(&group->field))
-               ret = 2;
-       else
-               ret = 1;
+       ret = 1;
 
   err:
        return ret;
@@ -942,9 +937,6 @@ int ec_GF2m_simple_cmp(const EC_GROUP *group, const EC_POINT *a, const EC_POINT
                {
                return EC_POINT_is_at_infinity(group, b) ? 0 : 1;
                }
-
-       if (EC_POINT_is_at_infinity(group, b))
-               return 1;
        
        if (a->Z_is_one && b->Z_is_one)
                {
@@ -975,15 +967,6 @@ int ec_GF2m_simple_cmp(const EC_GROUP *group, const EC_POINT *a, const EC_POINT
        return ret;
        }
 
-int ec_GF2m_simple_range(const EC_GROUP *group, const EC_POINT *a)
-       {
-       if (BN_num_bits(&a->X) > BN_num_bits(&group->field))
-               return 0;
-       if (BN_num_bits(&a->Y) > BN_num_bits(&group->field))
-               return 0;
-       return 1;
-       }
-
 
 /* Forces the given EC_POINT to internally use affine coordinates. */
 int ec_GF2m_simple_make_affine(const EC_GROUP *group, EC_POINT *point, BN_CTX *ctx)
index 522802c07ae13c3c34bb31185da508cf20077233..12fb0e6d6d7378d57e0ec2965f97eb62cd001d6d 100644 (file)
@@ -304,13 +304,7 @@ int EC_KEY_check_key(const EC_KEY *eckey)
                ECerr(EC_F_EC_KEY_CHECK_KEY, ERR_R_PASSED_NULL_PARAMETER);
                return 0;
                }
-
-       if (EC_POINT_is_at_infinity(eckey->group, eckey->pub_key))
-               {
-               ECerr(EC_F_EC_KEY_CHECK_KEY, EC_R_POINT_AT_INFINITY);
-               goto err;
-               }
-
+       
        if ((ctx = BN_CTX_new()) == NULL)
                goto err;
        if ((point = EC_POINT_new(eckey->group)) == NULL)
index 4e9c4dab364f040bd4169f4cdd5b83d8ad1c2825..fd751e5eb18f6b754fa0af419c8851cad1851c2e 100644 (file)
@@ -323,7 +323,6 @@ int ec_GFp_simple_invert(const EC_GROUP *, EC_POINT *, BN_CTX *);
 int ec_GFp_simple_is_at_infinity(const EC_GROUP *, const EC_POINT *);
 int ec_GFp_simple_is_on_curve(const EC_GROUP *, const EC_POINT *, BN_CTX *);
 int ec_GFp_simple_cmp(const EC_GROUP *, const EC_POINT *a, const EC_POINT *b, BN_CTX *);
-int ec_GFp_simple_range(const EC_GROUP *group, const EC_POINT *a);
 int ec_GFp_simple_make_affine(const EC_GROUP *, EC_POINT *, BN_CTX *);
 int ec_GFp_simple_points_make_affine(const EC_GROUP *, size_t num, EC_POINT *[], BN_CTX *);
 int ec_GFp_simple_field_mul(const EC_GROUP *, BIGNUM *r, const BIGNUM *a, const BIGNUM *b, BN_CTX *);
@@ -380,7 +379,6 @@ int ec_GF2m_simple_invert(const EC_GROUP *, EC_POINT *, BN_CTX *);
 int ec_GF2m_simple_is_at_infinity(const EC_GROUP *, const EC_POINT *);
 int ec_GF2m_simple_is_on_curve(const EC_GROUP *, const EC_POINT *, BN_CTX *);
 int ec_GF2m_simple_cmp(const EC_GROUP *, const EC_POINT *a, const EC_POINT *b, BN_CTX *);
-int ec_GF2m_simple_range(const EC_GROUP *group, const EC_POINT *a);
 int ec_GF2m_simple_make_affine(const EC_GROUP *, EC_POINT *, BN_CTX *);
 int ec_GF2m_simple_points_make_affine(const EC_GROUP *, size_t num, EC_POINT *[], BN_CTX *);
 int ec_GF2m_simple_field_mul(const EC_GROUP *, BIGNUM *r, const BIGNUM *a, const BIGNUM *b, BN_CTX *);
index 3e56b71a211aa421baa3da6696ffe9890d1937c6..4d26f8bdf6921f4ae9a9e5be06fbde3f54b55628 100644 (file)
@@ -441,11 +441,8 @@ int ec_GFp_simple_set_Jprojective_coordinates_GFp(const EC_GROUP *group, EC_POIN
                        }
                point->Z_is_one = Z_is_one;
                }
-
-       if (BN_cmp(&point->X, x) || BN_cmp(&point->Y, y))
-               ret = 2;
-       else
-               ret = 1;
+       
+       ret = 1;
        
  err:
        if (new_ctx != NULL)
@@ -1409,9 +1406,6 @@ int ec_GFp_simple_cmp(const EC_GROUP *group, const EC_POINT *a, const EC_POINT *
                {
                return EC_POINT_is_at_infinity(group, b) ? 0 : 1;
                }
-
-       if (EC_POINT_is_at_infinity(group, b))
-               return 1;
        
        if (a->Z_is_one && b->Z_is_one)
                {
@@ -1500,6 +1494,7 @@ int ec_GFp_simple_cmp(const EC_GROUP *group, const EC_POINT *a, const EC_POINT *
        return ret;
        }
 
+
 int ec_GFp_simple_make_affine(const EC_GROUP *group, EC_POINT *point, BN_CTX *ctx)
        {
        BN_CTX *new_ctx = NULL;