evm: Fix minor issues from PR#4

wormholelabs-xyz · Dec 23, 2024 · f7c7f72 · f7c7f72
1 parent 04afa0f
commit f7c7f72
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/evm/src/NttManager/ManagerBase.sol b/evm/src/NttManager/ManagerBase.sol
@@ -354,6 +354,7 @@ abstract contract ManagerBase is
         assert(this.mode() == mode);
         assert(this.chainId() == chainId);
         assert(this.endpoint() == endpoint);
+        assert(this.executor() == executor);
     }
 
     function _checkThresholdInvariants() internal view {

diff --git a/evm/src/NttManager/NttManager.sol b/evm/src/NttManager/NttManager.sol
@@ -662,6 +662,8 @@ contract NttManager is INttManager, RateLimiter, ManagerBase {
 
         uint128 gasLimit = peerData.gasLimit;
         if (gasLimit == 0) {
+            // The gas limit can only be zero when a contract has been migrated from an older version,
+            // where the gasLimit was not defined in the manager peer struct and was not set during the upgrade.
             revert InvalidGasLimitZero(args.recipientChain);
         }