Skip to content

Harden attestation payload parsing#34

Merged
leopoldjoy merged 1 commit into
mainfrom
harden-attestation-payload-consumption
Jun 17, 2026
Merged

Harden attestation payload parsing#34
leopoldjoy merged 1 commit into
mainfrom
harden-attestation-payload-consumption

Conversation

@leopoldjoy

@leopoldjoy leopoldjoy commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • require _parseAttestation to consume the full outer payload map
  • reject trailing bytes after definite-map entries or after an indefinite-map break marker
  • add regression tests for both malformed payload shapes

Testing

  • forge fmt --check src test
  • forge test --match-contract NitroValidatorIndefiniteLengthTest -vvv
  • forge test
  • forge build --sizes

Co-authored-by: OpenCode <opencode-noreply@coinbase.com>
@leopoldjoy leopoldjoy merged commit 5e1584b into main Jun 17, 2026
8 checks passed
@leopoldjoy leopoldjoy deleted the harden-attestation-payload-consumption branch June 17, 2026 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants