Skip to content

Commit

Permalink
fix: upgrade hyperchain add missing fields (matter-labs#415)
Browse files Browse the repository at this point in the history
  • Loading branch information
kelemeno authored Apr 26, 2024
1 parent f9ccb10 commit b824726
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions l1-contracts/contracts/upgrades/UpgradeHyperchains.sol
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,20 @@ contract UpgradeHyperchains is BaseZkSyncUpgrade {
/// @notice The main function that will be called by the upgrade proxy.
/// @param _proposedUpgrade The upgrade to be executed.
function upgrade(ProposedUpgrade calldata _proposedUpgrade) public override returns (bytes32) {
(uint256 chainId, address bridgehubAddress, address stateTransitionManager, address sharedBridgeAddress) = abi
.decode(_proposedUpgrade.postUpgradeCalldata, (uint256, address, address, address));
(
uint256 chainId,
address bridgehubAddress,
address stateTransitionManager,
address sharedBridgeAddress,
address chainAdmin,
address validatorTimelock
) = abi.decode(_proposedUpgrade.postUpgradeCalldata, (uint256, address, address, address, address, address));
require(chainId != 0, "UpgradeHyperchain: 1");
require(bridgehubAddress != address(0), "UpgradeHyperchain: 2");
require(stateTransitionManager != address(0), "UpgradeHyperchain: 3");
require(sharedBridgeAddress != address(0), "UpgradeHyperchain: 4");
require(chainAdmin != address(0), "UpgradeHyperchains: 5");
require(validatorTimelock != address(0), "UpgradeHyperchains: 6");

s.chainId = chainId;
s.bridgehub = bridgehubAddress;
Expand All @@ -27,6 +35,8 @@ contract UpgradeHyperchains is BaseZkSyncUpgrade {
s.baseToken = ETH_TOKEN_ADDRESS;
s.baseTokenGasPriceMultiplierNominator = 1;
s.baseTokenGasPriceMultiplierDenominator = 1;
s.admin = chainAdmin;
s.validators[validatorTimelock] = true;

super.upgrade(_proposedUpgrade);
return Diamond.DIAMOND_INIT_SUCCESS_RETURN_VALUE;
Expand Down

0 comments on commit b824726

Please sign in to comment.