Skip to content

fix: search for all note nonces instead of just the one for the note index (backport #21438)#21439

Closed
AztecBot wants to merge 1 commit intobackport-to-v4-next-stagingfrom
claudebox/backport-21438-note-nonces-v4-next
Closed

fix: search for all note nonces instead of just the one for the note index (backport #21438)#21439
AztecBot wants to merge 1 commit intobackport-to-v4-next-stagingfrom
claudebox/backport-21438-note-nonces-v4-next

Conversation

@AztecBot
Copy link
Collaborator

Summary

Backport of #21438 to v4-next.

Cherry-pick applied cleanly — no conflicts. The note_metadata.nr comment reformatting was already present on v4-next so only nonce_discovery.nr was changed.

Changes

  • Searches all possible note hash indices for nonces instead of just the one corresponding to the note's index, working around a kernel issue where nonces may use any index.
  • Adds bvec_filter helper and duplicate note hash detection.
  • Adds tests for misaligned nonces, out-of-bounds indices, and duplicate unique note hashes.

Original PR

#21438 by @nventuro

ClaudeBox log: https://claudebox.work/s/d70f5d1bf6bdaab3?run=1

…index (#21438)

We now search in all indices instead of just the one for the
corresponding note hash.
@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 12, 2026
@nventuro
Copy link
Contributor

Closing in favor of #21440

@nventuro nventuro closed this Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants