Skip to content

test: cover AVM accumulated-data tail padding#24719

Draft
AztecBot wants to merge 1 commit into
nextfrom
cb/rollup-audit-remediation-1612-avm-tail-padding-tests
Draft

test: cover AVM accumulated-data tail padding#24719
AztecBot wants to merge 1 commit into
nextfrom
cb/rollup-audit-remediation-1612-avm-tail-padding-tests

Conversation

@AztecBot

Copy link
Copy Markdown
Collaborator

Summary

The public tx-base validator already rejects non-zero AVM accumulated-data cells past their claimed lengths, but the regression suite did not exercise those defensive checks. This adds targeted negative coverage for every AVM accumulated-data tail that the validator zero-padding guard protects.

Fix

  • Add public tx-base negative tests for AVM note hashes, nullifiers, L2-to-L1 messages, public data writes, and public log payload tails.
  • Assert each test fails with the corresponding zero-padding validation message.

Testing

  • PATH=/home/aztec-dev/aztec-packages/noir/noir-repo/target/debug:$PATH nargo test --package rollup_lib avm_ --silence-warnings

Issues

  • Closes AztecProtocol/aztec-claude#1612 — AVM accumulated-data tail padding lacks negative regression tests

Created by claudebox · group: rollup-audit-remediation

@AztecBot AztecBot added ci-draft Run CI on draft PRs. ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure claudebox Owned by claudebox. it can push to this PR. labels Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant