Skip to content

Commit 345de0e

Browse files
committed
fix(aggregation): drop proverAddress from AggregationOutputs to match v117 contract
The zkVM aggregation program committed a 7-field AggregationOutputs (224B) including `proverAddress` (from upstream succinctlabs#421 front-run protection), while the on-chain OPSuccinctL2OutputOracle (v117 / adam/v1.1.7-2) reconstructs a 6-field struct (192B). The mismatched publicValues made sha256 diverge, so the SP1 verifier pairing check failed with InvalidProof (0x09bde339) on every proposeL2Output. MANTLE_CHANGES.md 3.4b realigned the host/contract call ABI back to v117's 4-param proposeL2Output but missed the program's committed struct. Drop proverAddress from the committed AggregationOutputs only; AggregationInputs. prover_address and the host stdin plumbing stay, matching mantle v2.2.4-mainnet.5-rc1 (keeps it as input, never commits it). Requires `just build-elfs` (agg vkey changes) + updateAggregationVkey on-chain.
1 parent bb213f1 commit 345de0e

2 files changed

Lines changed: 0 additions & 2 deletions

File tree

‎programs/aggregation/src/main.rs‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,6 @@ pub fn main() {
9191
l2BlockNumber: final_boot_info.l2BlockNumber,
9292
rollupConfigHash: final_boot_info.rollupConfigHash,
9393
multiBlockVKey: multi_block_vkey_b256,
94-
proverAddress: agg_inputs.prover_address,
9594
};
9695

9796
// Commit to the aggregated [`AggregationOutputs`].

‎utils/client/src/types.rs‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ sol! {
2121
uint64 l2BlockNumber;
2222
bytes32 rollupConfigHash;
2323
bytes32 multiBlockVKey;
24-
address proverAddress;
2524
}
2625
}
2726

0 commit comments

Comments
 (0)