Checks X chanin txs in bridge contracts

nucypher · Sep 7, 2023 · 34089fa · 34089fa
1 parent ff36f5b
commit 34089fa
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/contracts/contracts/coordination/TACoChildApplication.sol b/contracts/contracts/coordination/TACoChildApplication.sol
@@ -40,6 +40,7 @@ contract TACoChildApplication is ITACoRootToChild, ITACoChildApplication, Initia
             "Address for root application must be specified"
         );
         rootApplication = _rootApplication;
+        _disableInitializers();
     }
 
     /**

diff --git a/contracts/xchain/PolygonChild.sol b/contracts/xchain/PolygonChild.sol
@@ -16,7 +16,8 @@ contract PolygonChild is ITACoChildToRoot, FxBaseChildTunnel, Ownable {
         bytes memory data
     ) internal override validateSender(sender) {
         // solhint-disable-next-line avoid-low-level-calls
-        childApplication.call(data);
+        (bool success, ) = childApplication.call(data);
+        require(success, "Child tx failed");
     }
 
     function setChildApplication(address _childApplication) public onlyOwner {

diff --git a/contracts/xchain/PolygonRoot.sol b/contracts/xchain/PolygonRoot.sol
@@ -28,7 +28,8 @@ contract PolygonRoot is FxBaseRootTunnel, ITACoRootToChild {
 
     function _processMessageFromChild(bytes memory data) internal override {
         // solhint-disable-next-line avoid-low-level-calls
-        rootApplication.call(data);
+        (bool success, ) = rootApplication.call(data);
+        require(success, "Root tx failed");
     }
 
     function updateOperator(