From b21fa8e2cd98d487d50755416c7ab90ebd6e620e Mon Sep 17 00:00:00 2001 From: HydroH Date: Tue, 6 May 2025 02:50:25 +0800 Subject: [PATCH] std: fix compile errors in `std.crypto.ecc` (#23797) Implemented `neg()` method for `AffineCoordinates` struct of p256, p384 and secp256k1 curves. Resolves: #20505 (partially) --- lib/std/crypto/pcurves/p256.zig | 4 ++++ lib/std/crypto/pcurves/p384.zig | 4 ++++ lib/std/crypto/pcurves/secp256k1.zig | 4 ++++ 3 files changed, 12 insertions(+) diff --git a/lib/std/crypto/pcurves/p256.zig b/lib/std/crypto/pcurves/p256.zig index ec176f78c5..aa9226ae0d 100644 --- a/lib/std/crypto/pcurves/p256.zig +++ b/lib/std/crypto/pcurves/p256.zig @@ -471,6 +471,10 @@ pub const AffineCoordinates = struct { /// Identity element in affine coordinates. pub const identityElement = AffineCoordinates{ .x = P256.identityElement.x, .y = P256.identityElement.y }; + pub fn neg(p: AffineCoordinates) AffineCoordinates { + return .{ .x = p.x, .y = p.y.neg() }; + } + fn cMov(p: *AffineCoordinates, a: AffineCoordinates, c: u1) void { p.x.cMov(a.x, c); p.y.cMov(a.y, c); diff --git a/lib/std/crypto/pcurves/p384.zig b/lib/std/crypto/pcurves/p384.zig index 3ab95da5e8..61632f61ed 100644 --- a/lib/std/crypto/pcurves/p384.zig +++ b/lib/std/crypto/pcurves/p384.zig @@ -471,6 +471,10 @@ pub const AffineCoordinates = struct { /// Identity element in affine coordinates. pub const identityElement = AffineCoordinates{ .x = P384.identityElement.x, .y = P384.identityElement.y }; + pub fn neg(p: AffineCoordinates) AffineCoordinates { + return .{ .x = p.x, .y = p.y.neg() }; + } + fn cMov(p: *AffineCoordinates, a: AffineCoordinates, c: u1) void { p.x.cMov(a.x, c); p.y.cMov(a.y, c); diff --git a/lib/std/crypto/pcurves/secp256k1.zig b/lib/std/crypto/pcurves/secp256k1.zig index 945abea931..c891f414f5 100644 --- a/lib/std/crypto/pcurves/secp256k1.zig +++ b/lib/std/crypto/pcurves/secp256k1.zig @@ -549,6 +549,10 @@ pub const AffineCoordinates = struct { /// Identity element in affine coordinates. pub const identityElement = AffineCoordinates{ .x = Secp256k1.identityElement.x, .y = Secp256k1.identityElement.y }; + pub fn neg(p: AffineCoordinates) AffineCoordinates { + return .{ .x = p.x, .y = p.y.neg() }; + } + fn cMov(p: *AffineCoordinates, a: AffineCoordinates, c: u1) void { p.x.cMov(a.x, c); p.y.cMov(a.y, c);