qyriad/miden-crypto
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(merkle): fix overflow in to_scalar_index for nodes at depth 64

Browse source
This commit is contained in:
Qyriad 2024-09-18 16:37:03 -06:00
parent e5dd7c6d6a
commit 0e7e6705d8
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/merkle/index.rs
View file

@ -114,8 +114,8 @@ impl NodeIndex {
/// Returns the scalar representation of the depth/value pair. /// Returns the scalar representation of the depth/value pair.
/// ///
/// It is computed as `2^depth + value`. /// It is computed as `2^depth + value`.
pub const fn to_scalar_index(&self) -> u64 { pub const fn to_scalar_index(&self) -> u128 {
(1 << self.depth as u64) + self.value (1 << self.depth as u64) + (self.value as u128)
} }
/// Returns the depth of the current instance. /// Returns the depth of the current instance.