From 9e4ec01fa263e6a0c3bc0456fc962af71ac836da Mon Sep 17 00:00:00 2001 From: Aaron Feickert <66188213+AaronFeickert@users.noreply.github.com> Date: Mon, 7 Jul 2025 16:02:51 -0500 Subject: [PATCH] curve: Use constant-time compressed edwards equality testing (#684) --- curve25519-dalek/src/edwards.rs | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/curve25519-dalek/src/edwards.rs b/curve25519-dalek/src/edwards.rs index 2252475..7206d64 100644 --- a/curve25519-dalek/src/edwards.rs +++ b/curve25519-dalek/src/edwards.rs @@ -170,7 +170,8 @@ use alloc::vec::Vec; /// /// The first 255 bits of a `CompressedEdwardsY` represent the /// \\(y\\)-coordinate. The high bit of the 32nd byte gives the sign of \\(x\\). -#[derive(Copy, Clone, Eq, PartialEq, Hash)] +#[allow(clippy::derived_hash_with_manual_eq)] +#[derive(Copy, Clone, Hash)] pub struct CompressedEdwardsY(pub [u8; 32]); impl ConstantTimeEq for CompressedEdwardsY { @@ -179,6 +180,13 @@ impl ConstantTimeEq for CompressedEdwardsY { } } +impl Eq for CompressedEdwardsY {} +impl PartialEq for CompressedEdwardsY { + fn eq(&self, other: &Self) -> bool { + self.ct_eq(other).into() + } +} + impl Debug for CompressedEdwardsY { fn fmt(&self, f: &mut core::fmt::Formatter<'_>) -> core::fmt::Result { write!(f, "CompressedEdwardsY: {:?}", self.as_bytes())