diff options
author | Peter Dettman <peter.dettman@bouncycastle.org> | 2014-01-26 15:01:53 +0700 |
---|---|---|
committer | Peter Dettman <peter.dettman@bouncycastle.org> | 2014-01-26 15:01:53 +0700 |
commit | 08223b1c37e1e00a5abd0411c9ee3f04f2edbb60 (patch) | |
tree | 393c95433a60f8cf01bd1132b456c295ab176716 /crypto/src/math | |
parent | Refactor DecompressPoint (diff) | |
download | BouncyCastle.NET-ed25519-08223b1c37e1e00a5abd0411c9ee3f04f2edbb60.tar.xz |
Fix coord access in Negate()
Reformatting
Diffstat (limited to 'crypto/src/math')
-rw-r--r-- | crypto/src/math/ec/ECPoint.cs | 44 |
1 files changed, 8 insertions, 36 deletions
diff --git a/crypto/src/math/ec/ECPoint.cs b/crypto/src/math/ec/ECPoint.cs index 036da0d7d..eb7128e1d 100644 --- a/crypto/src/math/ec/ECPoint.cs +++ b/crypto/src/math/ec/ECPoint.cs @@ -498,10 +498,7 @@ namespace Org.BouncyCastle.Math.EC * @param x affine x co-ordinate * @param y affine y co-ordinate */ - public FpPoint( - ECCurve curve, - ECFieldElement x, - ECFieldElement y) + public FpPoint(ECCurve curve, ECFieldElement x, ECFieldElement y) : this(curve, x, y, false) { } @@ -514,11 +511,7 @@ namespace Org.BouncyCastle.Math.EC * @param y affine y co-ordinate * @param withCompression if true encode with point compression */ - public FpPoint( - ECCurve curve, - ECFieldElement x, - ECFieldElement y, - bool withCompression) + public FpPoint(ECCurve curve, ECFieldElement x, ECFieldElement y, bool withCompression) : base(curve, x, y, withCompression) { if ((x == null) != (y == null)) @@ -546,21 +539,14 @@ namespace Org.BouncyCastle.Math.EC } // B.3 pg 62 - public override ECPoint Add( - ECPoint b) + public override ECPoint Add(ECPoint b) { if (this.IsInfinity) - { return b; - } if (b.IsInfinity) - { return this; - } if (this == b) - { return Twice(); - } ECCurve curve = this.Curve; int coord = curve.CoordinateSystem; @@ -776,17 +762,13 @@ namespace Org.BouncyCastle.Math.EC public override ECPoint Twice() { if (this.IsInfinity) - { return this; - } ECCurve curve = this.Curve; ECFieldElement Y1 = this.RawYCoord; if (Y1.IsZero) - { return curve.Infinity; - } int coord = curve.CoordinateSystem; @@ -908,23 +890,15 @@ namespace Org.BouncyCastle.Math.EC public override ECPoint TwicePlus(ECPoint b) { if (this == b) - { return ThreeTimes(); - } if (this.IsInfinity) - { return b; - } if (b.IsInfinity) - { return Twice(); - } ECFieldElement Y1 = this.RawYCoord; if (Y1.IsZero) - { return b; - } ECCurve curve = this.Curve; int coord = curve.CoordinateSystem; @@ -984,7 +958,7 @@ namespace Org.BouncyCastle.Math.EC public override ECPoint ThreeTimes() { - if (IsInfinity) + if (this.IsInfinity) return this; ECFieldElement Y1 = this.RawYCoord; @@ -1026,7 +1000,7 @@ namespace Org.BouncyCastle.Math.EC } default: { - // NOTE: Be careful about recursions between twicePlus and threeTimes + // NOTE: Be careful about recursions between TwicePlus and ThreeTimes return Twice().Add(this); } } @@ -1076,19 +1050,17 @@ namespace Org.BouncyCastle.Math.EC public override ECPoint Negate() { if (IsInfinity) - { return this; - } - ECCurve curve = this.Curve; + ECCurve curve = Curve; int coord = curve.CoordinateSystem; if (ECCurve.COORD_AFFINE != coord) { - return new FpPoint(curve, XCoord, YCoord.Negate(), this.m_zs, IsCompressed); + return new FpPoint(curve, RawXCoord, RawYCoord.Negate(), RawZCoords, IsCompressed); } - return new FpPoint(curve, XCoord, YCoord.Negate(), IsCompressed); + return new FpPoint(curve, RawXCoord, RawYCoord.Negate(), IsCompressed); } protected virtual ECFieldElement CalculateJacobianModifiedW(ECFieldElement Z, ECFieldElement ZSquared) |