Merge #1992

1992: initializing reentrancy guard in constructor r=Deniallugo a=Barichek



Co-authored-by: Ihor Barenblat <[email protected]>

Author: bors-zksync-dev[bot]

contracts/contracts/DeployFactory.sol

@@ -40,8 +40,6 @@ contract DeployFactory is TokenDeployInit {
         require(_governor != address(0), "governor check");
         require(_feeAccountAddress != address(0), "fee acc address check");
 
-        ZkSync(_zkSyncTarget).initialize(abi.encode(address(0), address(0), address(0), bytes32(0)));
-
         deployProxyContracts(_govTarget, _verifierTarget, _zkSyncTarget, _genesisRoot, _firstValidator, _governor);
 
         selfdestruct(msg.sender);

contracts/contracts/ZkSync.sol

@@ -118,6 +118,10 @@ contract ZkSync is UpgradeableMaster, Storage, Config, Events, ReentrancyGuard {
         return !exodusMode;
     }
 
+    constructor() {
+        initializeReentrancyGuard();
+    }
+
     /// @notice zkSync contract initialization. Can be external because Proxy contract intercepts illegal calls of this function.
     /// @param initializationParameters Encoded representation of initialization parameters:
     /// @dev _governanceAddress The address of Governance contract