Skip to content

Cap fee components before proposal validation#24713

Draft
AztecBot wants to merge 1 commit into
nextfrom
cb/l1-audit-remediation-593-fee-header-caps
Draft

Cap fee components before proposal validation#24713
AztecBot wants to merge 1 commit into
nextfrom
cb/l1-audit-remediation-593-fee-header-caps

Conversation

@AztecBot

Copy link
Copy Markdown
Collaborator

Summary

Fee header compression capped congestion and prover components only when storing checkpoint metadata, while proposal validation could still charge the uncapped congestion amount. That let proof-time reward distribution burn only the stored capped value and route the remainder to sequencer rewards.

Fix

  • Cap fee-asset prover and congestion components in getManaMinFeeComponentsAt before summedMinFee and proposal validation use them.
  • Keep FeeHeaderLib.compress defensive and update overflow tests to assert validation and storage use the same capped component values.

Testing

  • FOUNDRY_PROFILE=coverage forge test --match-path test/fees/FeeHeaderOverflow.t.sol
  • FOUNDRY_PROFILE=coverage forge test --match-path test/fees/FeeRollup.t.sol
  • git diff --check

Issues

  • Closes AztecProtocol/aztec-claude#593 — FeeHeader congestion cap silently redirects rollup burn to sequencer rewards at proof time.

Created by claudebox · group: l1-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