Revert "Add parameter validation to allbridge lf 14440 (#1275)"

This reverts commit efa7976.

Author: 0xDEnYO

conventions.md

@@ -201,10 +201,10 @@ Follow the folder structure to locate resources and generate or modify code in a
         );
     }
     ```
-  - When a non-EVM address is used, the `bridgeData.receiver` must contain `NON_EVM_ADDRESS` (found in src/Helpers/LiFiData.sol), ensuring proper handling:
+  - When a non-EVM address is used, the `bridgeData.receiver` must contain `LibAsset.NON_EVM_ADDRESS`, ensuring proper handling:
     ```
     if (
-        bridgeData.receiver != NON_EVM_ADDRESS
+        bridgeData.receiver != LibAsset.NON_EVM_ADDRESS
     ) {
         revert InvalidCallData();
     }

docs/ChainflipFacet.md

@@ -32,7 +32,7 @@ struct ChainflipData {
 }
 ```
 
-For non-EVM destinations (i.e. Solana, Bitcoin), set the `receiver` in `BridgeData` to `NON_EVM_ADDRESS` (inherit from src/Helpers/LiFiData.sol) and provide the destination address in `nonEVMReceiver`.
+For non-EVM destinations (i.e. Solana, Bitcoin), set the `receiver` in `BridgeData` to `LibAsset.NON_EVM_ADDRESS` and provide the destination address in `nonEVMReceiver`.
 
 ## Supported Chains
 

src/Errors/GenericErrors.sol

@@ -19,7 +19,6 @@ error InvalidConfig();
 error InvalidContract();
 error InvalidDestinationChain();
 error InvalidFallbackAddress();
-error InvalidNonEVMReceiver();
 error InvalidReceiver();
 error InvalidSendingToken();
 error NativeAssetNotSupported();

src/Facets/AllBridgeFacet.sol

@@ -8,55 +8,27 @@ import { SwapperV2 } from "../Helpers/SwapperV2.sol";
 import { ReentrancyGuard } from "../Helpers/ReentrancyGuard.sol";
 import { Validatable } from "../Helpers/Validatable.sol";
 import { LibSwap } from "../Libraries/LibSwap.sol";
-import { InvalidConfig, InvalidCallData, InvalidNonEVMReceiver, InvalidReceiver } from "../Errors/GenericErrors.sol";
-import { LiFiData } from "../Helpers/LiFiData.sol";
 
 /// @title Allbridge Facet
-/// @author LI.FI (https://li.fi)
+/// @author Li.Finance (https://li.finance)
 /// @notice Provides functionality for bridging through AllBridge
-/// @custom:version 2.1.0
-contract AllBridgeFacet is
-    ILiFi,
-    ReentrancyGuard,
-    SwapperV2,
-    Validatable,
-    LiFiData
-{
-    uint32 private constant ALLBRIDGE_ID_ETHEREUM = 1;
-    uint32 private constant ALLBRIDGE_ID_BSC = 2;
-    uint32 private constant ALLBRIDGE_ID_TRON = 3;
-    uint32 private constant ALLBRIDGE_ID_SOLANA = 4;
-    uint32 private constant ALLBRIDGE_ID_POLYGON = 5;
-    uint32 private constant ALLBRIDGE_ID_ARBITRUM = 6;
-    uint32 private constant ALLBRIDGE_ID_AVALANCHE = 8;
-    uint32 private constant ALLBRIDGE_ID_BASE = 9;
-    uint32 private constant ALLBRIDGE_ID_OPTIMISM = 10;
-    uint32 private constant ALLBRIDGE_ID_CELO = 11;
-    uint32 private constant ALLBRIDGE_ID_SUI = 13;
-    uint256 internal constant LIFI_CHAIN_ID_ARBITRUM = 42161;
-    uint256 internal constant LIFI_CHAIN_ID_AVALANCHE = 43114;
-    uint256 internal constant LIFI_CHAIN_ID_BASE = 8453;
-    uint256 internal constant LIFI_CHAIN_ID_BSC = 56;
-    uint256 internal constant LIFI_CHAIN_ID_CELO = 42220;
-    uint256 internal constant LIFI_CHAIN_ID_POLYGON = 137;
-
-    error UnsupportedAllBridgeChainId();
-
+/// @custom:version 2.0.0
+contract AllBridgeFacet is ILiFi, ReentrancyGuard, SwapperV2, Validatable {
     /// @notice The contract address of the AllBridge router on the source chain.
     // solhint-disable-next-line immutable-vars-naming
-    IAllBridge private immutable ALLBRIDGE;
+    IAllBridge private immutable allBridge;
 
     /// @notice The struct for the AllBridge data.
-    /// @param recipient The address of the token receiver after bridging.
     /// @param fees The amount of token to pay the messenger and the bridge
+    /// @param recipient The address of the token receiver after bridging.
     /// @param destinationChainId The destination chain id.
     /// @param receiveToken The token to receive on the destination chain.
     /// @param nonce A random nonce to associate with the tx.
     /// @param messenger The messenger protocol enum
     /// @param payFeeWithSendingAsset Whether to pay the relayer fee with the sending asset or not
     struct AllBridgeData {
-        bytes32 recipient;
         uint256 fees;
+        bytes32 recipient;
         uint256 destinationChainId;
         bytes32 receiveToken;
         uint256 nonce;
@@ -67,9 +39,7 @@ contract AllBridgeFacet is
     /// @notice Initializes the AllBridge contract
     /// @param _allBridge The address of the AllBridge contract
     constructor(IAllBridge _allBridge) {
-        if (address(_allBridge) == address(0)) revert InvalidConfig();
-
-        ALLBRIDGE = _allBridge;
+        allBridge = _allBridge;
     }
 
     /// @notice Bridge tokens to another chain via AllBridge
@@ -126,38 +96,14 @@ contract AllBridgeFacet is
         ILiFi.BridgeData memory _bridgeData,
         AllBridgeData calldata _allBridgeData
     ) internal {
-        // make sure destinationChainId matches in bridgeData and allBridgeData
-        if (
-            _allBridgeData.destinationChainId !=
-            _getAllBridgeChainId(_bridgeData.destinationChainId)
-        ) revert InvalidCallData();
-
-        // validate receiver address
-        if (_bridgeData.receiver == NON_EVM_ADDRESS) {
-            // destination chain is non-EVM
-            // make sure it's non-zero (we cannot validate further)
-            if (_allBridgeData.recipient == bytes32(0))
-                revert InvalidNonEVMReceiver();
-        } else {
-            // destination chain is EVM
-            // make sure that bridgeData and allBridgeData receiver addresses match
-            if (
-                _bridgeData.receiver !=
-                address(uint160(uint256(_allBridgeData.recipient)))
-            ) revert InvalidReceiver();
-        }
-
-        // set max approval to allBridge, if current allowance is insufficient
         LibAsset.maxApproveERC20(
             IERC20(_bridgeData.sendingAssetId),
-            address(ALLBRIDGE),
+            address(allBridge),
             _bridgeData.minAmount
         );
 
-        // check if bridge fee should be paid with sending or native asset
         if (_allBridgeData.payFeeWithSendingAsset) {
-            // pay fee with sending asset
-            ALLBRIDGE.swapAndBridge(
+            allBridge.swapAndBridge(
                 bytes32(uint256(uint160(_bridgeData.sendingAssetId))),
                 _bridgeData.minAmount,
                 _allBridgeData.recipient,
@@ -168,8 +114,7 @@ contract AllBridgeFacet is
                 _allBridgeData.fees
             );
         } else {
-            // pay fee with native asset
-            ALLBRIDGE.swapAndBridge{ value: _allBridgeData.fees }(
+            allBridge.swapAndBridge{ value: _allBridgeData.fees }(
                 bytes32(uint256(uint160(_bridgeData.sendingAssetId))),
                 _bridgeData.minAmount,
                 _allBridgeData.recipient,
@@ -183,41 +128,4 @@ contract AllBridgeFacet is
 
         emit LiFiTransferStarted(_bridgeData);
     }
-
-    /// @notice Converts LiFi internal chain IDs to AllBridge chain IDs
-    /// @param _destinationChainId The LiFi chain ID to convert
-    /// @return The corresponding Chainflip chain ID
-    /// @dev Reverts if the destination chain is not supported
-    function _getAllBridgeChainId(
-        uint256 _destinationChainId
-    ) internal pure returns (uint256) {
-        // split possible values in half for more efficient search/matching
-
-        // first try to match cases where chainId is the same and does not need to be mapped
-        if (
-            _destinationChainId == ALLBRIDGE_ID_ETHEREUM ||
-            _destinationChainId == ALLBRIDGE_ID_OPTIMISM
-        ) return _destinationChainId;
-        // all others have custom chainIds
-        else if (_destinationChainId == LIFI_CHAIN_ID_BSC)
-            return ALLBRIDGE_ID_BSC;
-        else if (_destinationChainId == LIFI_CHAIN_ID_TRON)
-            return ALLBRIDGE_ID_TRON;
-        else if (_destinationChainId == LIFI_CHAIN_ID_SOLANA)
-            return ALLBRIDGE_ID_SOLANA;
-        else if (_destinationChainId == LIFI_CHAIN_ID_POLYGON)
-            return ALLBRIDGE_ID_POLYGON;
-        else if (_destinationChainId == LIFI_CHAIN_ID_ARBITRUM)
-            return ALLBRIDGE_ID_ARBITRUM;
-        else if (_destinationChainId == LIFI_CHAIN_ID_AVALANCHE)
-            return ALLBRIDGE_ID_AVALANCHE;
-        else if (_destinationChainId == LIFI_CHAIN_ID_BASE)
-            return ALLBRIDGE_ID_BASE;
-        else if (_destinationChainId == LIFI_CHAIN_ID_CELO)
-            return ALLBRIDGE_ID_CELO;
-        else if (_destinationChainId == LIFI_CHAIN_ID_SUI)
-            return ALLBRIDGE_ID_SUI;
-        // revert if no match found
-        else revert UnsupportedAllBridgeChainId();
-    }
 }

src/Facets/ChainflipFacet.sol

@@ -10,19 +10,12 @@ import { Validatable } from "../Helpers/Validatable.sol";
 import { IChainflipVault } from "../Interfaces/IChainflip.sol";
 import { IERC20 } from "@openzeppelin/contracts/token/ERC20/IERC20.sol";
 import { InformationMismatch, InvalidConfig, InvalidReceiver } from "../Errors/GenericErrors.sol";
-import { LiFiData } from "../Helpers/LiFiData.sol";
 
-/// @title ChainflipFacet
+/// @title Chainflip Facet
 /// @author LI.FI (https://li.fi)
 /// @notice Allows bridging assets via Chainflip
-/// @custom:version 1.0.1
-contract ChainflipFacet is
-    ILiFi,
-    ReentrancyGuard,
-    SwapperV2,
-    Validatable,
-    LiFiData
-{
+/// @custom:version 1.0.0
+contract ChainflipFacet is ILiFi, ReentrancyGuard, SwapperV2, Validatable {
     /// Events ///
     event BridgeToNonEVMChain(
         bytes32 indexed transactionId,
@@ -141,7 +134,7 @@ contract ChainflipFacet is
 
         // Handle address encoding based on destination chain type
         bytes memory encodedDstAddress;
-        if (_bridgeData.receiver == NON_EVM_ADDRESS) {
+        if (_bridgeData.receiver == LibAsset.NON_EVM_ADDRESS) {
             if (_chainflipData.nonEVMReceiver.length == 0) {
                 revert EmptyNonEvmAddress();
             }

src/Facets/RelayFacet.sol

@@ -9,20 +9,12 @@ import { ReentrancyGuard } from "../Helpers/ReentrancyGuard.sol";
 import { SwapperV2 } from "../Helpers/SwapperV2.sol";
 import { Validatable } from "../Helpers/Validatable.sol";
 import { ECDSA } from "solady/utils/ECDSA.sol";
-import { LiFiData } from "../Helpers/LiFiData.sol";
-import { InvalidConfig } from "../Errors/GenericErrors.sol";
 
-/// @title RelayFacet
+/// @title Relay Facet
 /// @author LI.FI (https://li.fi)
 /// @notice Provides functionality for bridging through Relay Protocol
-/// @custom:version 1.0.1
-contract RelayFacet is
-    ILiFi,
-    ReentrancyGuard,
-    SwapperV2,
-    Validatable,
-    LiFiData
-{
+/// @custom:version 1.0.0
+contract RelayFacet is ILiFi, ReentrancyGuard, SwapperV2, Validatable {
     // Receiver for native transfers
     // solhint-disable-next-line immutable-vars-naming
     address public immutable relayReceiver;
@@ -75,7 +67,7 @@ contract RelayFacet is
 
         // Ensure nonEVMAddress is not empty
         if (
-            _bridgeData.receiver == NON_EVM_ADDRESS &&
+            _bridgeData.receiver == LibAsset.NON_EVM_ADDRESS &&
             _relayData.nonEVMReceiver == bytes32(0)
         ) {
             revert InvalidQuote();
@@ -92,7 +84,7 @@ contract RelayFacet is
                     bytes32(uint256(uint160(address(this)))),
                     bytes32(uint256(uint160(_bridgeData.sendingAssetId))),
                     _getMappedChainId(_bridgeData.destinationChainId),
-                    _bridgeData.receiver == NON_EVM_ADDRESS
+                    _bridgeData.receiver == LibAsset.NON_EVM_ADDRESS
                         ? _relayData.nonEVMReceiver
                         : bytes32(uint256(uint160(_bridgeData.receiver))),
                     _relayData.receivingAssetId
@@ -111,10 +103,6 @@ contract RelayFacet is
     /// @param _relayReceiver The receiver for native transfers
     /// @param _relaySolver The relayer wallet for ERC20 transfers
     constructor(address _relayReceiver, address _relaySolver) {
-        if (_relayReceiver == address(0) || _relaySolver == address(0)) {
-            revert InvalidConfig();
-        }
-
         relayReceiver = _relayReceiver;
         relaySolver = _relaySolver;
     }
@@ -215,7 +203,7 @@ contract RelayFacet is
         consumedIds[_relayData.requestId] = true;
 
         // Emit special event if bridging to non-EVM chain
-        if (_bridgeData.receiver == NON_EVM_ADDRESS) {
+        if (_bridgeData.receiver == LibAsset.NON_EVM_ADDRESS) {
             emit BridgeToNonEVMChain(
                 _bridgeData.transactionId,
                 _getMappedChainId(_bridgeData.destinationChainId),

src/Helpers/LiFiData.sol

@@ -1,4 +1,5 @@
 // SPDX-License-Identifier: LGPL-3.0-only
+/// @custom:version 1.0.0
 pragma solidity ^0.8.17;
 
 import { ILiFi } from "../Interfaces/ILiFi.sol";
@@ -7,10 +8,9 @@ import { LibAsset } from "../Libraries/LibAsset.sol";
 import { LibAllowList } from "../Libraries/LibAllowList.sol";
 import { ContractCallNotAllowed, NoSwapDataProvided, CumulativeSlippageTooHigh } from "../Errors/GenericErrors.sol";
 
-/// @title SwapperV2
+/// @title Swapper
 /// @author LI.FI (https://li.fi)
 /// @notice Abstract contract to provide swap functionality
-/// @custom:version 1.0.1
 contract SwapperV2 is ILiFi {
     /// Types ///
 
@@ -119,7 +119,7 @@ contract SwapperV2 is ILiFi {
 
         if (finalBalance > initialBalance) {
             LibAsset.transferAsset(
-                LibAsset.NULL_ADDRESS,
+                LibAsset.NATIVE_ASSETID,
                 _refundReceiver,
                 finalBalance - initialBalance
             );

src/Helpers/SwapperV2.sol

@@ -7,17 +7,23 @@ import { SafeTransferLib } from "solady/utils/SafeTransferLib.sol";
 import { InvalidReceiver, NullAddrIsNotAValidSpender, InvalidAmount, NullAddrIsNotAnERC20Token } from "../Errors/GenericErrors.sol";
 
 /// @title LibAsset
-/// @custom:version 2.1.1
+/// @custom:version 2.1.0
 /// @notice This library contains helpers for dealing with onchain transfers
 ///         of assets, including accounting for the native asset `assetId`
 ///         conventions and any noncompliant ERC20 transfers
 library LibAsset {
     using SafeTransferLib for address;
     using SafeTransferLib for address payable;
 
+    address internal constant NULL_ADDRESS = address(0);
+
+    address internal constant NON_EVM_ADDRESS =
+        0x11f111f111f111F111f111f111F111f111f111F1;
+
     /// @dev All native assets use the empty address for their asset id
     ///      by convention
-    address internal constant NULL_ADDRESS = address(0);
+
+    address internal constant NATIVE_ASSETID = NULL_ADDRESS;
 
     /// @dev EIP-7702 delegation designator prefix for Account Abstraction
     bytes3 internal constant DELEGATION_DESIGNATOR = 0xef0100;
@@ -185,7 +191,7 @@ library LibAsset {
     /// @param assetId The asset identifier to evaluate
     /// @return Boolean indicating if the asset is the native asset
     function isNativeAsset(address assetId) internal pure returns (bool) {
-        return assetId == NULL_ADDRESS;
+        return assetId == NATIVE_ASSETID;
     }
 
     /// @notice Checks if the given address is a contract

src/Libraries/LibAsset.sol

@@ -12,7 +12,7 @@ import { InvalidConfig } from "../Errors/GenericErrors.sol";
 /// @title ReceiverChainflip
 /// @author LI.FI (https://li.fi)
 /// @notice Receiver contract for Chainflip cross-chain swaps and message passing
-/// @custom:version 1.0.1
+/// @custom:version 1.0.0
 contract ReceiverChainflip is ILiFi, WithdrawablePeriphery {
     using SafeTransferLib for address;
     using SafeTransferLib for address payable;
@@ -114,7 +114,7 @@ contract ReceiverChainflip is ILiFi, WithdrawablePeriphery {
     ) private {
         // Group address conversion and store in memory to avoid multiple storage reads
         address actualAssetId = assetId == CHAINFLIP_NATIVE_ADDRESS
-            ? LibAsset.NULL_ADDRESS
+            ? LibAsset.NATIVE_ASSETID
             : assetId;
         bool isNative = LibAsset.isNativeAsset(actualAssetId);