Hello everyone,
I'm encountering an issue with calling the transfer and transferToken functions from this below smart contract. I've deployed the contract in question, but attempts to transfer USDT from it using these functions have failed. I've thoroughly reviewed the code and am unable to pinpoint the exact reason for the failure.
Could someone please offer insights into why these functions might not be executing as expected when called from another smart contract? Any assistance or suggestions on how to debug and resolve this issue would be greatly appreciated.
Thank you in advance for your help!
Bridge Contract Code Where I try to call USDT token transfer and transferFrom function.
TokenBridge.sol
// SPDX-License-Identifier: MIT
pragma solidity ^0.8.9;
import "./Ownable.sol";
import "./IERC20.sol";
contract BridgeContract is Ownable {
uint256 private numberOfTransfer;
address public controller;
constructor(address _controller) {
controller = _controller;
}
function updateControler(address _controller) public onlyOwner {
controller = _controller;
}
function getNumberOfTransfer() public view returns (uint256) {
return numberOfTransfer;
}
function transfer(address tokenAddress, uint256 amount, address to, uint256 nonce) public {
require(msg.sender == controller, "Only for controller");
require(tokenAddress != address(0), "Invalid token address");
require(to != address(0), "Invalid recipient address");
require(nonce == numberOfTransfer + 1, "The previous nonce is missing");
IERC20 token = IERC20(tokenAddress);
bool success = token.transfer(to, amount);
require(success, "ERC20 transfer failed");
numberOfTransfer = numberOfTransfer + 1;
}
function transferFrom(address tokenAddress, uint256 amount, address from, address to, uint256 nonce) public {
require(msg.sender == controller, "Only for controller");
require(tokenAddress != address(0), "Invalid token address");
require(to != address(0), "Invalid recipient address");
require(nonce == numberOfTransfer + 1, "The previous nonce is missing");
IERC20 token = IERC20(tokenAddress);
bool success = token.transferFrom(from, to, amount);
require(success, "ERC20 transfer failed");
numberOfTransfer = numberOfTransfer + 1;
}
function transferTokens(address tokenAddress, address to, uint256 amount) public onlyOwner {
require(tokenAddress != address(0), "Invalid token address");
require(to != address(0), "Invalid recipient address");
IERC20 token = IERC20(tokenAddress);
bool success = token.transfer(to, amount);
require(success, "ERC20 transfer failed");
}
}
Context.sol
pragma solidity ^0.8.9;
// SPDX-License-Identifier: MIT
/*
* @dev Provides information about the current execution context, including the
* sender of the transaction and its data. While these are generally available
* via msg.sender and msg.data, they should not be accessed in such a direct
* manner, since when dealing with GSN meta-transactions the account sending and
* paying for execution may not be the actual sender (as far as an application
* is concerned).
*
* This contract is only required for intermediate, library-like contracts.
*/
abstract contract Context {
function _msgSender() internal view virtual returns (address payable) {
return payable(msg.sender);
}
function _msgData() internal view virtual returns (bytes memory) {
this; // silence state mutability warning without generating bytecode - see https://github.com/ethereum/solidity/issues/2691
return msg.data;
}
}
IERC20.sol
pragma solidity ^0.8.9;
// SPDX-License-Identifier: MIT
interface IERC20 {
function transfer(address recipient, uint256 amount) external returns (bool);
function transferFrom(address sender, address recipient, uint256 amount) external returns (bool);
}
Ownable.sol
pragma solidity ^0.8.9;
// SPDX-License-Identifier: MIT
import "./Context.sol";
/**
* @dev Contract module which provides a basic access control mechanism, where
* there is an account (an owner) that can be granted exclusive access to
* specific functions.
*
* By default, the owner account will be the one that deploys the contract. This
* can later be changed with {transferOwnership}.
*
* This module is used through inheritance. It will make available the modifier
* `onlyOwner`, which can be applied to your functions to restrict their use to
* the owner.
*/
abstract contract Ownable is Context {
address private _owner;
event OwnershipTransferred(address indexed previousOwner, address indexed newOwner);
/**
* @dev Initializes the contract setting the deployer as the initial owner.
*/
constructor () {
address msgSender = _msgSender();
_owner = msgSender;
emit OwnershipTransferred(address(0), msgSender);
}
/**
* @dev Returns the address of the current owner.
*/
function owner() public view returns (address) {
return _owner;
}
/**
* @dev Throws if called by any account other than the owner.
*/
modifier onlyOwner() {
require(_owner == _msgSender(), "Ownable: caller is not the owner");
_;
}
/**
* @dev Leaves the contract without owner. It will not be possible to call
* `onlyOwner` functions anymore. Can only be called by the current owner.
*
* NOTE: Renouncing ownership will leave the contract without an owner,
* thereby removing any functionality that is only available to the owner.
*/
function renounceOwnership() public virtual onlyOwner {
emit OwnershipTransferred(_owner, address(0));
_owner = address(0);
}
/**
* @dev Transfers ownership of the contract to a new account (`newOwner`).
* Can only be called by the current owner.
*/
function transferOwnership(address newOwner) public virtual onlyOwner {
require(newOwner != address(0), "Ownable: new owner is the zero address");
emit OwnershipTransferred(_owner, newOwner);
_owner = newOwner;
}
}
The transection Fail without any message.
