Commit 4c8f79a3 authored by Bodo Möller's avatar Bodo Möller
Browse files

make sure 'neg' flag (which does not really matter for GF(2^m), but 

could cause confusion for ECDSA) is set to zero

Submitted by: Sheueling Chang
parent 7cc6ec7a

crypto/ec/ec2_smpl.c

+5 −0
Original line number Diff line number Diff line
@@ -364,8 +364,11 @@ int ec_GF2m_simple_point_set_affine_coordinates(const EC_GROUP *group, EC_POINT 
		}



	if (!BN_copy(&point->X, x)) goto err;

	point->X.neg = 0;

	if (!BN_copy(&point->Y, y)) goto err;

	point->Y.neg = 0;

	if (!BN_copy(&point->Z, BN_value_one())) goto err;

	point->Z.neg = 0;

	point->Z_is_one = 1;

	ret = 1;
@@ -396,10 +399,12 @@ int ec_GF2m_simple_point_get_affine_coordinates(const EC_GROUP *group, const EC_ 
	if (x != NULL)

		{

		if (!BN_copy(x, &point->X)) goto err;

		x->neg = 0;

		}

	if (y != NULL)

		{

		if (!BN_copy(y, &point->Y)) goto err;

		y->neg = 0;

		}

	ret = 1;