mirror of
https://github.com/pseXperiments/icicle.git
synced 2026-01-08 23:17:54 -05:00
Fix affine to projective zero point bug (#552)
## Describe the changes This PR fixes affine to projective functions in bindings by adding a condition if the point in affine form is zero then return the projective zero --------- Co-authored-by: Jeremy Felder <jeremy.felder1@gmail.com>
This commit is contained in:
@@ -32,9 +32,13 @@ func (p *MockProjective) FromAffine(a MockAffine) MockProjective {
|
||||
z := MockBaseField{}
|
||||
z.One()
|
||||
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
p.Zero()
|
||||
} else {
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z.One()
|
||||
}
|
||||
|
||||
return *p
|
||||
}
|
||||
@@ -68,6 +72,11 @@ func (a *MockAffine) FromLimbs(x, y []uint32) MockAffine {
|
||||
func (a MockAffine) ToProjective() MockProjective {
|
||||
var z MockBaseField
|
||||
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
var p MockProjective
|
||||
return p.Zero()
|
||||
}
|
||||
|
||||
return MockProjective{
|
||||
X: a.X,
|
||||
Y: a.Y,
|
||||
|
||||
@@ -43,9 +43,13 @@ func (p *Projective) FromAffine(a Affine) Projective {
|
||||
z := BaseField{}
|
||||
z.One()
|
||||
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
p.Zero()
|
||||
} else {
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z.One()
|
||||
}
|
||||
|
||||
return *p
|
||||
}
|
||||
@@ -109,6 +113,11 @@ func (a *Affine) FromLimbs(x, y []uint32) Affine {
|
||||
func (a Affine) ToProjective() Projective {
|
||||
var z BaseField
|
||||
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
var p Projective
|
||||
return p.Zero()
|
||||
}
|
||||
|
||||
return Projective{
|
||||
X: a.X,
|
||||
Y: a.Y,
|
||||
|
||||
@@ -43,9 +43,13 @@ func (p *G2Projective) FromAffine(a G2Affine) G2Projective {
|
||||
z := G2BaseField{}
|
||||
z.One()
|
||||
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
p.Zero()
|
||||
} else {
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z.One()
|
||||
}
|
||||
|
||||
return *p
|
||||
}
|
||||
@@ -109,6 +113,11 @@ func (a *G2Affine) FromLimbs(x, y []uint32) G2Affine {
|
||||
func (a G2Affine) ToProjective() G2Projective {
|
||||
var z G2BaseField
|
||||
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
var p G2Projective
|
||||
return p.Zero()
|
||||
}
|
||||
|
||||
return G2Projective{
|
||||
X: a.X,
|
||||
Y: a.Y,
|
||||
|
||||
@@ -43,9 +43,13 @@ func (p *Projective) FromAffine(a Affine) Projective {
|
||||
z := BaseField{}
|
||||
z.One()
|
||||
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
p.Zero()
|
||||
} else {
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z.One()
|
||||
}
|
||||
|
||||
return *p
|
||||
}
|
||||
@@ -109,6 +113,11 @@ func (a *Affine) FromLimbs(x, y []uint32) Affine {
|
||||
func (a Affine) ToProjective() Projective {
|
||||
var z BaseField
|
||||
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
var p Projective
|
||||
return p.Zero()
|
||||
}
|
||||
|
||||
return Projective{
|
||||
X: a.X,
|
||||
Y: a.Y,
|
||||
|
||||
@@ -43,9 +43,13 @@ func (p *G2Projective) FromAffine(a G2Affine) G2Projective {
|
||||
z := G2BaseField{}
|
||||
z.One()
|
||||
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
p.Zero()
|
||||
} else {
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z.One()
|
||||
}
|
||||
|
||||
return *p
|
||||
}
|
||||
@@ -109,6 +113,11 @@ func (a *G2Affine) FromLimbs(x, y []uint32) G2Affine {
|
||||
func (a G2Affine) ToProjective() G2Projective {
|
||||
var z G2BaseField
|
||||
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
var p G2Projective
|
||||
return p.Zero()
|
||||
}
|
||||
|
||||
return G2Projective{
|
||||
X: a.X,
|
||||
Y: a.Y,
|
||||
|
||||
@@ -43,9 +43,13 @@ func (p *Projective) FromAffine(a Affine) Projective {
|
||||
z := BaseField{}
|
||||
z.One()
|
||||
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
p.Zero()
|
||||
} else {
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z.One()
|
||||
}
|
||||
|
||||
return *p
|
||||
}
|
||||
@@ -109,6 +113,11 @@ func (a *Affine) FromLimbs(x, y []uint32) Affine {
|
||||
func (a Affine) ToProjective() Projective {
|
||||
var z BaseField
|
||||
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
var p Projective
|
||||
return p.Zero()
|
||||
}
|
||||
|
||||
return Projective{
|
||||
X: a.X,
|
||||
Y: a.Y,
|
||||
|
||||
@@ -43,9 +43,13 @@ func (p *G2Projective) FromAffine(a G2Affine) G2Projective {
|
||||
z := G2BaseField{}
|
||||
z.One()
|
||||
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
p.Zero()
|
||||
} else {
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z.One()
|
||||
}
|
||||
|
||||
return *p
|
||||
}
|
||||
@@ -109,6 +113,11 @@ func (a *G2Affine) FromLimbs(x, y []uint32) G2Affine {
|
||||
func (a G2Affine) ToProjective() G2Projective {
|
||||
var z G2BaseField
|
||||
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
var p G2Projective
|
||||
return p.Zero()
|
||||
}
|
||||
|
||||
return G2Projective{
|
||||
X: a.X,
|
||||
Y: a.Y,
|
||||
|
||||
@@ -43,9 +43,13 @@ func (p *Projective) FromAffine(a Affine) Projective {
|
||||
z := BaseField{}
|
||||
z.One()
|
||||
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
p.Zero()
|
||||
} else {
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z.One()
|
||||
}
|
||||
|
||||
return *p
|
||||
}
|
||||
@@ -109,6 +113,11 @@ func (a *Affine) FromLimbs(x, y []uint32) Affine {
|
||||
func (a Affine) ToProjective() Projective {
|
||||
var z BaseField
|
||||
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
var p Projective
|
||||
return p.Zero()
|
||||
}
|
||||
|
||||
return Projective{
|
||||
X: a.X,
|
||||
Y: a.Y,
|
||||
|
||||
@@ -43,9 +43,13 @@ func (p *G2Projective) FromAffine(a G2Affine) G2Projective {
|
||||
z := G2BaseField{}
|
||||
z.One()
|
||||
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
p.Zero()
|
||||
} else {
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z.One()
|
||||
}
|
||||
|
||||
return *p
|
||||
}
|
||||
@@ -109,6 +113,11 @@ func (a *G2Affine) FromLimbs(x, y []uint32) G2Affine {
|
||||
func (a G2Affine) ToProjective() G2Projective {
|
||||
var z G2BaseField
|
||||
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
var p G2Projective
|
||||
return p.Zero()
|
||||
}
|
||||
|
||||
return G2Projective{
|
||||
X: a.X,
|
||||
Y: a.Y,
|
||||
|
||||
@@ -43,9 +43,13 @@ func (p *Projective) FromAffine(a Affine) Projective {
|
||||
z := BaseField{}
|
||||
z.One()
|
||||
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
p.Zero()
|
||||
} else {
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z.One()
|
||||
}
|
||||
|
||||
return *p
|
||||
}
|
||||
@@ -109,6 +113,11 @@ func (a *Affine) FromLimbs(x, y []uint32) Affine {
|
||||
func (a Affine) ToProjective() Projective {
|
||||
var z BaseField
|
||||
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
var p Projective
|
||||
return p.Zero()
|
||||
}
|
||||
|
||||
return Projective{
|
||||
X: a.X,
|
||||
Y: a.Y,
|
||||
|
||||
@@ -43,9 +43,13 @@ func (p *{{.CurvePrefix}}Projective) FromAffine(a {{.CurvePrefix}}Affine) {{.Cur
|
||||
z := {{.CurvePrefix}}BaseField{}
|
||||
z.One()
|
||||
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
p.Zero()
|
||||
}else{
|
||||
p.X = a.X
|
||||
p.Y = a.Y
|
||||
p.Z = z
|
||||
p.Z = z.One()
|
||||
}
|
||||
|
||||
return *p
|
||||
}
|
||||
@@ -109,6 +113,11 @@ func (a *{{.CurvePrefix}}Affine) FromLimbs(x, y []uint32) {{.CurvePrefix}}Affine
|
||||
func (a {{.CurvePrefix}}Affine) ToProjective() {{.CurvePrefix}}Projective {
|
||||
var z {{.CurvePrefix}}BaseField
|
||||
|
||||
if (a.X == z.Zero()) && (a.Y == z.Zero()) {
|
||||
var p {{.CurvePrefix}}Projective
|
||||
return p.Zero()
|
||||
}
|
||||
|
||||
return {{.CurvePrefix}}Projective{
|
||||
X: a.X,
|
||||
Y: a.Y,
|
||||
|
||||
@@ -79,6 +79,9 @@ impl<C: Curve> Affine<C> {
|
||||
}
|
||||
|
||||
pub fn to_projective(&self) -> Projective<C> {
|
||||
if *self == Self::zero() {
|
||||
return Projective::<C>::zero();
|
||||
}
|
||||
Projective {
|
||||
x: self.x,
|
||||
y: self.y,
|
||||
|
||||
Reference in New Issue
Block a user