Análisis del contrato del puente estándar de Optimism

/**
 * @title IL1ERC20Bridge
 */
interface IL1ERC20Bridge {
    /**********
     * Eventos *
     **********/

    event ERC20DepositInitiated(

    /********************
     * Funciones públicas *
     ********************/

    /**
     * @dev obtiene la dirección del contrato puente de l2 correspondiente.
     * @return Dirección del contrato puente de l2 correspondiente.
     */
    function l2TokenBridge() external returns (address);

    /**
     * @dev deposita una cantidad del ERC-20 en el saldo del llamador en l2.
     * @param _l1Token Dirección del ERC-20 de l1 que estamos depositando
     * @param _l2Token Dirección del ERC-20 de l2 respectivo de l1
     * @param _amount Cantidad del ERC-20 a depositar
     * @param _l2Gas Límite de gas requerido para completar el depósito en l2.
     * @param _data Datos opcionales para reenviar a l2. Estos datos se proporcionan
     *        únicamente como conveniencia para contratos externos. Aparte de imponer una
     *        longitud máxima, estos contratos no ofrecen garantías sobre su contenido.
     */
    function depositERC20(
        address _l1Token,
        address _l2Token,
        uint256 _amount,
        uint32 _l2Gas,
        bytes calldata _data
    ) external;

    /**
     * @dev deposita una cantidad de ERC-20 en el saldo de un destinatario en l2.
     * @param _l1Token Dirección del ERC-20 de l1 que estamos depositando
     * @param _l2Token Dirección del ERC-20 de l2 respectivo de l1
     * @param _to Dirección de l2 a la que acreditar el retiro.
     * @param _amount Cantidad del ERC-20 a depositar.
     * @param _l2Gas Límite de gas requerido para completar el depósito en l2.
     * @param _data Datos opcionales para reenviar a l2. Estos datos se proporcionan
     *        únicamente como conveniencia para contratos externos. Aparte de imponer una
     *        longitud máxima, estos contratos no ofrecen garantías sobre su contenido.
     */
    function depositERC20To(
        address _l1Token,
        address _l2Token,
        address _to,
        uint256 _amount,
        uint32 _l2Gas,
        bytes calldata _data
    ) external;

    /*************************
     * Funciones entre cadenas *
     *************************/

    /**
     * @dev Completa un retiro de l2 a l1, y acredita los fondos al saldo del destinatario del
     * token ERC-20 de l1.
     * Esta llamada fallará si el retiro inicializado desde l2 no ha sido finalizado.
     *
     * @param _l1Token Dirección del token de l1 para el cual hacer finalizeWithdrawal.
     * @param _l2Token Dirección del token de l2 donde se inició el retiro.
     * @param _from Dirección de l2 que inicia la transferencia.
     * @param _to Dirección de l1 a la que acreditar el retiro.
     * @param _amount Cantidad del ERC-20 a depositar.
     * @param _data Datos proporcionados por el remitente en l2. Estos datos se proporcionan
     *   únicamente como conveniencia para contratos externos. Aparte de imponer una longitud
     *   máxima, estos contratos no ofrecen garantías sobre su contenido.
     */
    function finalizeERC20Withdrawal(
        address _l1Token,
        address _l2Token,
        address _from,
        address _to,
        uint256 _amount,
        bytes calldata _data
    ) external;
}

// SPDX-License-Identifier: MIT
pragma solidity >0.5.0 <0.9.0;

import "./IL1ERC20Bridge.sol";

/**
 * @title IL1StandardBridge
 */
interface IL1StandardBridge is IL1ERC20Bridge {
    /**********
     * Eventos *
     **********/
    event ETHDepositInitiated(
        address indexed _from,
        address indexed _to,
        uint256 _amount,
        bytes _data
    );

    event ETHWithdrawalFinalized(
        .
        .
        .
    );

    /********************
     * Funciones públicas *
     ********************/

    /**
     * @dev Deposita una cantidad del ETH en el saldo del llamador en l2.
            .
            .
            .
     */
    function depositETH(uint32 _l2Gas, bytes calldata _data) external payable;

    /**
     * @dev Deposita una cantidad de ETH en el saldo de un destinatario en l2.
            .
            .
            .
     */
    function depositETHTo(
        address _to,
        uint32 _l2Gas,
        bytes calldata _data
    ) external payable;

    /*************************
     * Funciones entre cadenas *
     *************************/

    /**
     * @dev Completa un retiro de l2 a l1, y acredita los fondos al saldo del destinatario del
     * token ETH de l1. Dado que solo el xDomainMessenger puede llamar a esta función, nunca será llamada
     * antes de que el retiro sea finalizado.
                .
                .
                .
     */
    function finalizeETHWithdrawal(
        address _from,
        address _to,
        uint256 _amount,
        bytes calldata _data
    ) external;
}

/**
 * @title CrossDomainEnabled
 * @dev Contrato auxiliar para contratos que realizan comunicaciones entre dominios
 *
 * Compilador utilizado: definido por el contrato que hereda
 */
contract CrossDomainEnabled {
    /*************
     * Variables *
     *************/

    // Contrato mensajero utilizado para enviar y recibir mensajes del otro dominio.
    address public messenger;

    /***************
     * Constructor *
     ***************/

    /**
     * @param _messenger Dirección del CrossDomainMessenger en la capa actual.
     */
    constructor(address _messenger) {
        messenger = _messenger;
    }

    /**********************
     * Modificadores de funciones *
     **********************/

    /**
     * Obliga a que la función modificada solo pueda ser llamada por una cuenta específica de dominio cruzado.
     * @param _sourceDomainAccount La única cuenta en el dominio de origen que está
     *  autenticada para llamar a esta función.
     */
    modifier onlyFromCrossDomainAccount(address _sourceDomainAccount) {

        _;
    }

    /**********************
     * Funciones internas *
     **********************/

    /**
     * Obtiene el mensajero, generalmente del almacenamiento. Esta función se expone en caso de que un contrato hijo
     * necesite anularla.
     * @return La dirección del contrato mensajero de dominio cruzado que debe usarse.
     */
    function getCrossDomainMessenger() internal virtual returns (ICrossDomainMessenger) {
        return ICrossDomainMessenger(messenger);
    }

    /**
     * Envía un mensaje a una cuenta en otro dominio
     * @param _crossDomainTarget El destinatario previsto en el dominio de destino
     * @param _message Los datos a enviar al objetivo (generalmente datos de llamada a una función con
     *  `onlyFromCrossDomainAccount()`)
     * @param _gasLimit El límite de gas para la recepción del mensaje en el dominio de destino.
     */
    function sendCrossDomainMessage(
        address _crossDomainTarget,
        uint32 _gasLimit,
        bytes memory _message

/**
 * @title L1StandardBridge
 * @dev El puente de ETH y ERC-20 de l1 es un contrato que almacena fondos depositados de l1 y tokens
 * estándar que están en uso en l2. Sincroniza un puente de l2 correspondiente, informándole de los depósitos
 * y escuchándolo para nuevos retiros finalizados.
 *
 */
contract L1StandardBridge is IL1StandardBridge, CrossDomainEnabled {
    using SafeERC20 for IERC20;

    /******************
     * Inicialización *
     ******************/

    /**
     * @param _l1messenger Dirección del mensajero de l1 que se utiliza para las comunicaciones entre cadenas.
     * @param _l2TokenBridge Dirección del puente estándar de l2.
     */
    // slither-disable-next-line external-function

    /**************
     * Depósitos *
     **************/

    /** @dev Modificador que requiere que el remitente sea una EOA. Esta comprobación podría ser eludida por un contrato
     *  malicioso a través de initcode, pero se encarga del error de usuario que queremos evitar.
     */
    modifier onlyEOA() {
        // Utilizado para detener depósitos desde contratos (evitar tokens perdidos accidentalmente)
        require(!Address.isContract(msg.sender), "Account not EOA");
        _;
    }

    /**
     * @dev Esta función puede ser llamada sin datos
     * para depositar una cantidad de ETH en el saldo del llamador en l2.
     * Dado que la función receive no toma datos, se reenvía una cantidad
     * predeterminada conservadora a l2.
     */
    receive() external payable onlyEOA {
        _initiateETHDeposit(msg.sender, msg.sender, 200_000, bytes(""));
    }

    /**
     * @inheritdoc IL1StandardBridge
     */
    function depositETH(uint32 _l2Gas, bytes calldata _data) external payable onlyEOA {
        _initiateETHDeposit(msg.sender, msg.sender, _l2Gas, _data);
    }

    /**
     * @inheritdoc IL1StandardBridge
     */
    function depositETHTo(
        address _to,
        uint32 _l2Gas,
        bytes calldata _data
    ) external payable {
        _initiateETHDeposit(msg.sender, _to, _l2Gas, _data);
    }

    /**
     * @dev Realiza la lógica para los depósitos almacenando el ETH e informando a la puerta de enlace de ETH de l2 sobre
     * el depósito.
     * @param _from Cuenta de la que extraer el depósito en l1.
     * @param _to Cuenta a la que dar el depósito en l2.
     * @param _l2Gas Límite de gas requerido para completar el depósito en l2.
     * @param _data Datos opcionales para reenviar a l2. Estos datos se proporcionan
     *        únicamente como conveniencia para contratos externos. Aparte de imponer una
     *        longitud máxima, estos contratos no ofrecen garantías sobre su contenido.
     */
    function _initiateETHDeposit(
        address _from,
        address _to,
        uint32 _l2Gas,
        bytes memory _data
    ) internal {
        // Construir datos de llamada para la llamada a finalizeDeposit
        bytes memory message = abi.encodeWithSelector(

    /**
     * @inheritdoc IL1ERC20Bridge
     */
    function depositERC20(
		.
		.
		.
    ) external virtual onlyEOA {
        _initiateERC20Deposit(_l1Token, _l2Token, msg.sender, msg.sender, _amount, _l2Gas, _data);
    }

    /**
     * @inheritdoc IL1ERC20Bridge
     */
    function depositERC20To(
		.
		.
		.
    ) external virtual {
        _initiateERC20Deposit(_l1Token, _l2Token, msg.sender, _to, _amount, _l2Gas, _data);
    }

    /**
     * @dev Realiza la lógica para los depósitos informando al contrato de token depositado de l2
     * sobre el depósito y llamando a un manejador para bloquear los fondos de l1. (ej., transferFrom)
     *
     * @param _l1Token Dirección del ERC-20 de l1 que estamos depositando
     * @param _l2Token Dirección del ERC-20 de l2 respectivo de l1
     * @param _from Cuenta de la que extraer el depósito en l1
     * @param _to Cuenta a la que dar el depósito en l2
     * @param _amount Cantidad del ERC-20 a depositar.
     * @param _l2Gas Límite de gas requerido para completar el depósito en l2.
     * @param _data Datos opcionales para reenviar a l2. Estos datos se proporcionan
     *        únicamente como conveniencia para contratos externos. Aparte de imponer una
     *        longitud máxima, estos contratos no ofrecen garantías sobre su contenido.
     */
    function _initiateERC20Deposit(
        address _l1Token,
        address _l2Token,
        address _from,
        address _to,
        uint256 _amount,
        uint32 _l2Gas,
        bytes calldata _data
    ) internal {

        // Construir datos de llamada para _l2Token.finalizeDeposit(_to, _amount)
        bytes memory message = abi.encodeWithSelector(
            IL2ERC20Bridge.finalizeDeposit.selector,
            _l1Token,
            _l2Token,
            _from,
            _to,
            _amount,
            _data
        );

        // Enviar datos de llamada a l2
        // slither-disable-next-line reentrancy-events, reentrancy-benign
        sendCrossDomainMessage(l2TokenBridge, _l2Gas, message);

        // slither-disable-next-line reentrancy-benign
        deposits[_l1Token][_l2Token] = deposits[_l1Token][_l2Token] + _amount;

        // slither-disable-next-line reentrancy-events
        emit ERC20DepositInitiated(_l1Token, _l2Token, _from, _to, _amount, _data);
    }

    /*************************
     * Funciones entre cadenas *
     *************************/

    /**
     * @inheritdoc IL1StandardBridge
     */
    function finalizeETHWithdrawal(
        address _from,
        address _to,
        uint256 _amount,
        bytes calldata _data

    }

    /**
     * @inheritdoc IL1ERC20Bridge
     */
    function finalizeERC20Withdrawal(
        address _l1Token,
        address _l2Token,
        address _from,
        address _to,
        uint256 _amount,
        bytes calldata _data
    ) external onlyFromCrossDomainAccount(l2TokenBridge) {

        // Cuando se finaliza un retiro en l1, el puente de l1 transfiere los fondos a quien realiza el retiro
        // slither-disable-next-line reentrancy-events
        IERC20(_l1Token).safeTransfer(_to, _amount);

        // slither-disable-next-line reentrancy-events
        emit ERC20WithdrawalFinalized(_l1Token, _l2Token, _from, _to, _amount, _data);
    }


    /*****************************
     * Temporal - Migrando ETH *
     *****************************/

    /**
     * @dev Añade saldo de ETH a la cuenta. Esto está destinado a permitir que el ETH
     * sea migrado de una puerta de enlace antigua a una nueva puerta de enlace.
     * NOTA: Esto se deja solo para una actualización para que podamos recibir el ETH migrado del
     * contrato antiguo
     */
    function donateETH() external payable {}
}

    /**
     * @param _l2Bridge Dirección del puente estándar de l2.
     * @param _l1Token Dirección del token de l1 correspondiente.
     * @param _name Nombre del ERC-20.
     * @param _symbol Símbolo del ERC-20.
     */
    constructor(
        address _l2Bridge,
        address _l1Token,
        string memory _name,
        string memory _symbol
    ) ERC20(_name, _symbol) {
        l1Token = _l1Token;
        l2Bridge = _l2Bridge;
    }

    modifier onlyL2Bridge() {
        require(msg.sender == l2Bridge, "Only L2 Bridge can mint and burn");
        _;
    }


    // slither-disable-next-line external-function
    function supportsInterface(bytes4 _interfaceId) public pure returns (bool) {
        bytes4 firstSupportedInterface = bytes4(keccak256("supportsInterface(bytes4)")); // ERC165
        bytes4 secondSupportedInterface = IL2StandardERC20.l1Token.selector ^
            IL2StandardERC20.mint.selector ^
            IL2StandardERC20.burn.selector;
        return _interfaceId == firstSupportedInterface || _interfaceId == secondSupportedInterface;
    }

    // slither-disable-next-line external-function
    function mint(address _to, uint256 _amount) public virtual onlyL2Bridge {
        _mint(_to, _amount);

        emit Mint(_to, _amount);
    }

    // slither-disable-next-line external-function
    function burn(address _from, uint256 _amount) public virtual onlyL2Bridge {
        _burn(_from, _amount);

        emit Burn(_from, _amount);
    }
}

/* Importaciones de bibliotecas */
import { ERC165Checker } from "@openzeppelin/contracts/utils/introspection/ERC165Checker.sol";
import { CrossDomainEnabled } from "../../libraries/bridge/CrossDomainEnabled.sol";
import { Lib_PredeployAddresses } from "../../libraries/constants/Lib_PredeployAddresses.sol";

/* Importaciones de contratos */
import { IL2StandardERC20 } from "../../standards/IL2StandardERC20.sol";

/**
 * @title L2StandardBridge
 * @dev El puente estándar de l2 es un contrato que trabaja junto con el puente estándar de l1 para
 * permitir transiciones de ETH y ERC-20 entre l1 y l2.
 * Este contrato actúa como acuñador de nuevos tokens cuando se entera de depósitos en el puente estándar
 * de l1.
 * Este contrato también actúa como quemador de los tokens destinados al retiro, informando al puente
 * de l1 para que libere los fondos de l1.
 */
contract L2StandardBridge is IL2ERC20Bridge, CrossDomainEnabled {
    /********************************
     * Referencias a contratos externos *
     ********************************/

    address public l1TokenBridge;

    /***************
     * Constructor *
     ***************/

    /**
     * @param _l2CrossDomainMessenger Mensajero de dominio cruzado utilizado por este contrato.
     * @param _l1TokenBridge Dirección del puente de l1 desplegado en la cadena principal.
     */
    constructor(address _l2CrossDomainMessenger, address _l1TokenBridge)
        CrossDomainEnabled(_l2CrossDomainMessenger)
    {
        l1TokenBridge = _l1TokenBridge;
    }

    /***************
     * Retiros *
     ***************/

    /**
     * @inheritdoc IL2ERC20Bridge
     */
    function withdraw(
        address _l2Token,
        uint256 _amount,
        uint32 _l1Gas,
        bytes calldata _data
    ) external virtual {
        _initiateWithdrawal(_l2Token, msg.sender, msg.sender, _amount, _l1Gas, _data);
    }

    /**
     * @inheritdoc IL2ERC20Bridge
     */
    function withdrawTo(
        address _l2Token,
        address _to,
        uint256 _amount,
        uint32 _l1Gas,
        bytes calldata _data
    ) external virtual {
        _initiateWithdrawal(_l2Token, msg.sender, _to, _amount, _l1Gas, _data);
    }

    /**
     * @dev Realiza la lógica para los retiros quemando el token e informando
     *      a la puerta de enlace del token de l1 sobre el retiro.
     * @param _l2Token Dirección del token de l2 donde se inicia el retiro.
     * @param _from Cuenta de la que extraer el retiro en l2.
     * @param _to Cuenta a la que dar el retiro en l1.
     * @param _amount Cantidad del token a retirar.
     * @param _l1Gas No utilizado, pero incluido por posibles consideraciones de compatibilidad futura.
     * @param _data Datos opcionales para reenviar a l1. Estos datos se proporcionan
     *        únicamente como conveniencia para contratos externos. Aparte de imponer una
     *        longitud máxima, estos contratos no ofrecen garantías sobre su contenido.
     */
    function _initiateWithdrawal(
        address _l2Token,
        address _from,
        address _to,
        uint256 _amount,
        uint32 _l1Gas,
        bytes calldata _data
    ) internal {
        // Cuando se inicia un retiro, quemamos los fondos de quien realiza el retiro para evitar el posterior uso
        // en l2
        // slither-disable-next-line reentrancy-events
        IL2StandardERC20(_l2Token).burn(msg.sender, _amount);

            message = abi.encodeWithSelector(
                IL1StandardBridge.finalizeETHWithdrawal.selector,
                _from,
                _to,
                _amount,
                _data
            );
        } else {
            message = abi.encodeWithSelector(
                IL1ERC20Bridge.finalizeERC20Withdrawal.selector,
                l1Token,
                _l2Token,
                _from,
                _to,
                _amount,
                _data
            );
        }

        // Enviar mensaje al puente de l1
        // slither-disable-next-line reentrancy-events
        sendCrossDomainMessage(l1TokenBridge, _l1Gas, message);

        // slither-disable-next-line reentrancy-events
        emit WithdrawalInitiated(l1Token, _l2Token, msg.sender, _to, _amount, _data);
    }

    /************************************
     * Función entre cadenas: Depósitos *
     ************************************/

    /**
     * @inheritdoc IL2ERC20Bridge
     */
    function finalizeDeposit(
        address _l1Token,
        address _l2Token,
        address _from,
        address _to,
        uint256 _amount,
        bytes calldata _data

        } else {
            // O bien el token de l2 en el que se está depositando no está de acuerdo sobre la dirección correcta
            // de su token de l1, o no admite la interfaz correcta.
            // Esto solo debería suceder si hay un token de l2 malicioso, o si un usuario de alguna manera
            // especificó la dirección incorrecta del token de l2 para depositar.
            // En cualquier caso, detenemos el proceso aquí y construimos un mensaje de
            // retiro para que los usuarios puedan sacar sus fondos en algunos casos.
            // No hay forma de prevenir por completo los contratos de tokens maliciosos, pero esto limita
            // el error del usuario y mitiga algunas formas de comportamiento malicioso del contrato.

            bytes memory message = abi.encodeWithSelector(
                IL1ERC20Bridge.finalizeERC20Withdrawal.selector,
                _l1Token,
                _l2Token,
                _to, // se intercambiaron _to y _from aquí para devolver el depósito al remitente
                _from,
                _amount,
                _data
            );

            // Enviar mensaje al puente de l1
            // slither-disable-next-line reentrancy-events
            sendCrossDomainMessage(l1TokenBridge, 0, message);
            // slither-disable-next-line reentrancy-events
            emit DepositFailed(_l1Token, _l2Token, _from, _to, _amount, _data);
        }
    }
}