Skip to content

test(bb): prove invalid ECDSA verification returns false#23187

Closed
AztecBot wants to merge 1 commit into
merge-train/barretenbergfrom
cb/c934ea0ea5cb
Closed

test(bb): prove invalid ECDSA verification returns false#23187
AztecBot wants to merge 1 commit into
merge-train/barretenbergfrom
cb/c934ea0ea5cb

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

Summary

  • add ACIR/dsl ECDSA regression coverage for zero r, zero s, and public-key-not-on-curve inputs
  • set the ECDSA opcode result witness to false for those invalid verification cases
  • build the circuit from the ACIR opcode path and construct/verify a Honk proof for Ultra and Mega builders over secp256k1 and secp256r1

Tests

  • cmake --preset clang20-no-avm && cmake --build --preset clang20-no-avm --target dsl_tests -j$(nproc)
  • /home/aztec-dev/aztec-packages/barretenberg/cpp/build/bin/dsl_tests --gtest_filter='EcdsaConstraintsTest*InvalidVerificationInputsReturnFalseAndProve*'
  • /home/aztec-dev/aztec-packages/barretenberg/cpp/build/bin/dsl_tests --gtest_filter='EcdsaConstraintsTest*'

Created by claudebox · group: aztec

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label May 12, 2026
@AztecBot AztecBot changed the base branch from merge-train/fairies to merge-train/barretenberg May 12, 2026 12:27
@TomAFrench TomAFrench closed this May 12, 2026
@TomAFrench TomAFrench deleted the cb/c934ea0ea5cb branch May 12, 2026 12:29
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