core: Check both secret and public key in ViewKey ct_eq()

dusk-network · Sep 24, 2024 · a0e3662 · a0e3662
1 parent c6dd5c6
commit a0e3662
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/core/CHANGELOG.md b/core/CHANGELOG.md
@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 
 - Call `gen_note_sk` in `SecretKey::owns` to avoid code duplication [#246]
+- `ViewKey` now checks both `a` and `B` in `ct_eq()`
 
 ## [0.32.0] - 2024-08-14
 

diff --git a/core/src/keys/view.rs b/core/src/keys/view.rs
@@ -32,14 +32,13 @@ pub struct ViewKey {
 
 impl ConstantTimeEq for ViewKey {
     fn ct_eq(&self, other: &Self) -> Choice {
-        // TODO - Why self.a is not checked?
-        self.B.ct_eq(&other.B)
+        self.a.ct_eq(&other.a) & self.B.ct_eq(&other.B)
     }
 }
 
 impl PartialEq for ViewKey {
     fn eq(&self, other: &Self) -> bool {
-        self.a == other.a && self.ct_eq(other).into()
+        self.ct_eq(other).into()
     }
 }