Skip to content

Commit

Permalink
fix: always pad bytes with 10*0 (#267)
Browse files Browse the repository at this point in the history
  • Loading branch information
Al-Kindi-0 authored and bobbinth committed Feb 14, 2024
1 parent 3882e0f commit 0acceaa
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions src/hash/rescue/rpx/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -88,10 +88,8 @@ impl Hasher for Rpx256 {
// `i` is not zero, then the chunks count wasn't enough to fill the state range, and an
// additional permutation must be performed.
let i = bytes.chunks(BINARY_CHUNK_SIZE).fold(0, |i, chunk| {
// the last element of the iteration may or may not be a full chunk. if it's not, then
// we need to pad the remainder bytes of the chunk with zeroes, separated by a `1`.
// this will avoid collisions at the bytes level.
if chunk.len() == BINARY_CHUNK_SIZE {
// we always pad `bytes` with a 1 followed by as many 0's to fill up `buf`.
if i != num_field_elem - 1 {
buf[..BINARY_CHUNK_SIZE].copy_from_slice(chunk);
} else {
buf.fill(0);
Expand Down

0 comments on commit 0acceaa

Please sign in to comment.