From 2f1d41042396aad4aada103a025e63c4cddd9320 Mon Sep 17 00:00:00 2001 From: Alex Page Date: Wed, 28 Jun 2023 02:30:57 -0400 Subject: [PATCH] Remove b Turns out we don't need it --- umskt/src/bink1998.rs | 3 +-- umskt/src/bink2002.rs | 3 +-- umskt/src/crypto.rs | 4 +--- umskt/src/weierstrass_curve.rs | 5 ++--- xpkey/src/main.rs | 3 +-- 5 files changed, 6 insertions(+), 12 deletions(-) diff --git a/umskt/src/bink1998.rs b/umskt/src/bink1998.rs index 95c51c4..dfd4c37 100644 --- a/umskt/src/bink1998.rs +++ b/umskt/src/bink1998.rs @@ -311,13 +311,12 @@ mod tests { let p = bink["p"].as_str().unwrap(); let a = bink["a"].as_str().unwrap(); - let b = bink["b"].as_str().unwrap(); let gx = bink["g"]["x"].as_str().unwrap(); let gy = bink["g"]["y"].as_str().unwrap(); let kx = bink["pub"]["x"].as_str().unwrap(); let ky = bink["pub"]["y"].as_str().unwrap(); - let curve = EllipticCurve::new(p, a, b, gx, gy, kx, ky).unwrap(); + let curve = EllipticCurve::new(p, a, gx, gy, kx, ky).unwrap(); assert!(bink1998::ProductKey::from_key(&curve, product_key).is_ok()); assert!(bink1998::ProductKey::from_key(&curve, "11111-R6BG2-39J83-RYKHF-W47TT").is_err()); diff --git a/umskt/src/bink2002.rs b/umskt/src/bink2002.rs index bd8c3f6..b5cea35 100644 --- a/umskt/src/bink2002.rs +++ b/umskt/src/bink2002.rs @@ -382,13 +382,12 @@ mod tests { let p = bink["p"].as_str().unwrap(); let a = bink["a"].as_str().unwrap(); - let b = bink["b"].as_str().unwrap(); let gx = bink["g"]["x"].as_str().unwrap(); let gy = bink["g"]["y"].as_str().unwrap(); let kx = bink["pub"]["x"].as_str().unwrap(); let ky = bink["pub"]["y"].as_str().unwrap(); - let curve = EllipticCurve::new(p, a, b, gx, gy, kx, ky).unwrap(); + let curve = EllipticCurve::new(p, a, gx, gy, kx, ky).unwrap(); assert!(super::ProductKey::from_key(&curve, product_key).is_ok()); assert!(super::ProductKey::from_key(&curve, "11111-YRGC8-4KYTG-C3FCC-JCFDY").is_err()); diff --git a/umskt/src/crypto.rs b/umskt/src/crypto.rs index c3a12a8..3b584f0 100644 --- a/umskt/src/crypto.rs +++ b/umskt/src/crypto.rs @@ -30,7 +30,6 @@ impl EllipticCurve { pub fn new( p: &str, a: &str, - b: &str, generator_x: &str, generator_y: &str, public_key_x: &str, @@ -38,13 +37,12 @@ impl EllipticCurve { ) -> Result { let p = BigInt::from_str_radix(p, 10)?; let a = BigInt::from_str_radix(a, 10)?; - let b = BigInt::from_str_radix(b, 10)?; let generator_x = BigInt::from_str_radix(generator_x, 10)?; let generator_y = BigInt::from_str_radix(generator_y, 10)?; let public_key_x = BigInt::from_str_radix(public_key_x, 10)?; let public_key_y = BigInt::from_str_radix(public_key_y, 10)?; - let curve = WeierstrassCurve::new(a, b, p); + let curve = WeierstrassCurve::new(a, p); let gen_point = Point::Point { x: generator_x, diff --git a/umskt/src/weierstrass_curve.rs b/umskt/src/weierstrass_curve.rs index 58f116d..c03fe01 100644 --- a/umskt/src/weierstrass_curve.rs +++ b/umskt/src/weierstrass_curve.rs @@ -11,13 +11,12 @@ pub enum Point { #[derive(Debug, Clone)] pub struct WeierstrassCurve { a: BigInt, - b: BigInt, p: BigInt, } impl WeierstrassCurve { - pub fn new(a: BigInt, b: BigInt, p: BigInt) -> Self { - WeierstrassCurve { a, b, p } + pub fn new(a: BigInt, p: BigInt) -> Self { + WeierstrassCurve { a, p } } fn mod_inverse(a: &BigInt, p: &BigInt) -> BigInt { diff --git a/xpkey/src/main.rs b/xpkey/src/main.rs index f9e709e..600b8fd 100644 --- a/xpkey/src/main.rs +++ b/xpkey/src/main.rs @@ -102,7 +102,6 @@ fn validate(args: &ValidateArgs) -> Result<()> { fn initialize_curve(bink: &Bink, bink_id: &str) -> Result { let p = &bink.p; let a = &bink.a; - let b = &bink.b; let gx = &bink.g.x; let gy = &bink.g.y; let kx = &bink.public.x; @@ -110,7 +109,7 @@ fn initialize_curve(bink: &Bink, bink_id: &str) -> Result { log::info!("Elliptic curve parameters for BINK ID {bink_id}:\n{bink}"); - EllipticCurve::new(p, a, b, gx, gy, kx, ky) + EllipticCurve::new(p, a, gx, gy, kx, ky) } fn bink1998_generate(