Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
FOXCOIN.addLiquidity(uint256,uint256) (#761-774) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in FOXCOIN._transfer(address,address,uint256) (#670-708):
External calls:
- swapAndLiquify(contractTokenBalance) (#692)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#750-756)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#692)
- recipient.transfer(amount) (#634)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#695)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#703)
- finalAmount = takeFee(sender,recipient,amount) (#697-698)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#788)
Apply the check-effects-interactions pattern.
Additional information: link
Combination 1: Reentrancy vulnerabilities + Functions that send Ether to arbitraty destination. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.
FOXCOIN.addLiquidity(uint256,uint256) (#761-774) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
Ensure that all the return values of the function calls are used.
Additional information: link
FOXCOIN.allowance(address,address).owner (#525) shadows:
- Ownable.owner() (#159-161) (function)
FOXCOIN._approve(address,address,uint256).owner (#548) shadows:
- Ownable.owner() (#159-161) (function)
Rename the local variables that shadow another component.
Additional information: link
FOXCOIN.setBuyTaxes(uint256,uint256,uint256) (#568-574) should emit an event for:
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#573)
FOXCOIN.setSellTaxes(uint256,uint256,uint256) (#576-582) should emit an event for:
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#581)
FOXCOIN.setDistributionSettings(uint256,uint256,uint256) (#584-590) should emit an event for:
- _liquidityShare = newLiquidityShare (#585)
- _teamShare = newTeamShare (#587)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#589)
FOXCOIN.setMaxTxAmount(uint256) (#592-594) should emit an event for:
- _maxTxAmount = maxTxAmount (#593)
FOXCOIN.setWalletLimit(uint256) (#604-606) should emit an event for:
- _walletMax = newLimit (#605)
FOXCOIN.setNumTokensBeforeSwap(uint256) (#608-610) should emit an event for:
- minimumTokensBeforeSwap = newLimit (#609)
Emit an event for critical parameter changes.
Additional information: link
FOXCOIN.setMarketingWalletAddress(address).newAddress (#612) lacks a zero-check on :
- marketingWalletAddress = address(newAddress) (#613)
FOXCOIN.setTeamWalletAddress(address).newAddress (#616) lacks a zero-check on :
- teamWalletAddress = address(newAddress) (#617)
Check that the address is not zero.
Additional information: link
Reentrancy in FOXCOIN.changeRouterVersion(address) (#637-654):
External calls:
- newPairAddress = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#645-646)
State variables written after the call(s):
- isMarketPair[address(uniswapPair)] = true (#653)
- isWalletLimitExempt[address(uniswapPair)] = true (#652)
- uniswapPair = newPairAddress (#649)
- uniswapV2Router = _uniswapV2Router (#650)
Reentrancy in FOXCOIN.constructor() (#475-503):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#479-480)
State variables written after the call(s):
- _allowances[address(this)][address(uniswapV2Router)] = _totalSupply (#483)
- _balances[_msgSender()] = _totalSupply (#501)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#490)
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#488)
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#489)
- isExcludedFromFee[owner()] = true (#485)
- isExcludedFromFee[address(this)] = true (#486)
- isMarketPair[address(uniswapPair)] = true (#499)
- isTxLimitExempt[owner()] = true (#496)
- isTxLimitExempt[address(this)] = true (#497)
- isWalletLimitExempt[owner()] = true (#492)
- isWalletLimitExempt[address(uniswapPair)] = true (#493)
- isWalletLimitExempt[address(this)] = true (#494)
- uniswapV2Router = _uniswapV2Router (#482)
Reentrancy in FOXCOIN.swapAndLiquify(uint256) (#717-739):
External calls:
- swapTokensForEth(tokensForSwap) (#722)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#750-756)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#738)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#732)
- recipient.transfer(amount) (#634)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#735)
- recipient.transfer(amount) (#634)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#738)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#738)
- _allowances[owner][spender] = amount (#552)
Reentrancy in FOXCOIN.transferFrom(address,address,uint256) (#664-668):
External calls:
- _transfer(sender,recipient,amount) (#665)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#750-756)
External calls sending eth:
- _transfer(sender,recipient,amount) (#665)
- recipient.transfer(amount) (#634)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#666)
- _allowances[owner][spender] = amount (#552)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in FOXCOIN._transfer(address,address,uint256) (#670-708):
External calls:
- swapAndLiquify(contractTokenBalance) (#692)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#750-756)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#692)
- recipient.transfer(amount) (#634)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#789)
- finalAmount = takeFee(sender,recipient,amount) (#697-698)
- Transfer(sender,recipient,finalAmount) (#705)
Reentrancy in FOXCOIN.constructor() (#475-503):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#479-480)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_totalSupply) (#502)
Reentrancy in FOXCOIN.swapAndLiquify(uint256) (#717-739):
External calls:
- swapTokensForEth(tokensForSwap) (#722)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#750-756)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#738)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#732)
- recipient.transfer(amount) (#634)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#735)
- recipient.transfer(amount) (#634)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#738)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#553)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#738)
Reentrancy in FOXCOIN.swapTokensForEth(uint256) (#741-759):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#750-756)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#758)
Reentrancy in FOXCOIN.transferFrom(address,address,uint256) (#664-668):
External calls:
- _transfer(sender,recipient,amount) (#665)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#750-756)
External calls sending eth:
- _transfer(sender,recipient,amount) (#665)
- recipient.transfer(amount) (#634)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#553)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#666)
Apply the check-effects-interactions pattern.
Additional information: link
Ownable.unlock() (#194-199) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp > _lockTime,Contract is locked until 7 days) (#196)
Avoid relying on block.timestamp.
Additional information: link
Address.isContract(address) (#89-98) uses assembly
- INLINE ASM (#96)
Address._functionCallWithValue(address,bytes,uint256,string) (#125-142) uses assembly
- INLINE ASM (#134-137)
Do not use evm assembly.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#125-142) is never used and should be removed
Address.functionCall(address,bytes) (#108-110) is never used and should be removed
Address.functionCall(address,bytes,string) (#112-114) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#116-118) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#120-123) is never used and should be removed
Address.isContract(address) (#89-98) is never used and should be removed
Address.sendValue(address,uint256) (#100-106) is never used and should be removed
Context._msgData() (#16-19) is never used and should be removed
SafeMath.mod(uint256,uint256) (#77-79) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#81-84) is never used and should be removed
Remove unused functions.
Additional information: link
FOXCOIN._totalSupply (#439) is set pre-construction with a non-constant function or state variable:
- 1000 * 10 ** 6 * 10 ** 6 * 10 ** _decimals
FOXCOIN._maxTxAmount (#440) is set pre-construction with a non-constant function or state variable:
- 10 * 10 ** 6 * 10 ** 6 * 10 ** _decimals
FOXCOIN._walletMax (#441) is set pre-construction with a non-constant function or state variable:
- 20 * 10 ** 6 * 10 ** 6 * 10 ** _decimals
FOXCOIN.minimumTokensBeforeSwap (#442) is set pre-construction with a non-constant function or state variable:
- 10 * 10 ** 6 * 10 ** _decimals
Remove any initialization of state variables via non-constant state variables or function calls. If variables must be set upon contract deployment, locate initialization in the constructor instead.
Additional information: link
Low level call in Address.sendValue(address,uint256) (#100-106):
- (success) = recipient.call{value: amount}() (#104)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#125-142):
- (success,returndata) = target.call{value: weiValue}(data) (#128)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#233) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#234) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#250) is not in mixedCase
Function IUniswapV2Router01.WETH() (#269) is not in mixedCase
Parameter FOXCOIN.setSwapAndLiquifyEnabled(bool)._enabled (#620) is not in mixedCase
Variable FOXCOIN._balances (#415) is not in mixedCase
Variable FOXCOIN._buyLiquidityFee (#423) is not in mixedCase
Variable FOXCOIN._buyMarketingFee (#424) is not in mixedCase
Variable FOXCOIN._buyTeamFee (#425) is not in mixedCase
Variable FOXCOIN._sellLiquidityFee (#427) is not in mixedCase
Variable FOXCOIN._sellMarketingFee (#428) is not in mixedCase
Variable FOXCOIN._sellTeamFee (#429) is not in mixedCase
Variable FOXCOIN._liquidityShare (#431) is not in mixedCase
Variable FOXCOIN._marketingShare (#432) is not in mixedCase
Variable FOXCOIN._teamShare (#433) is not in mixedCase
Variable FOXCOIN._totalTaxIfBuying (#435) is not in mixedCase
Variable FOXCOIN._totalTaxIfSelling (#436) is not in mixedCase
Variable FOXCOIN._totalDistributionShares (#437) is not in mixedCase
Variable FOXCOIN._maxTxAmount (#440) is not in mixedCase
Variable FOXCOIN._walletMax (#441) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#17)" inContext (#10-20)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in FOXCOIN._transfer(address,address,uint256) (#670-708):
External calls:
- swapAndLiquify(contractTokenBalance) (#692)
- recipient.transfer(amount) (#634)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#692)
- recipient.transfer(amount) (#634)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#695)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#703)
- finalAmount = takeFee(sender,recipient,amount) (#697-698)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#788)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#789)
- finalAmount = takeFee(sender,recipient,amount) (#697-698)
- Transfer(sender,recipient,finalAmount) (#705)
Reentrancy in FOXCOIN.swapAndLiquify(uint256) (#717-739):
External calls:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#732)
- recipient.transfer(amount) (#634)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#735)
- recipient.transfer(amount) (#634)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#732)
- recipient.transfer(amount) (#634)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#735)
- recipient.transfer(amount) (#634)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#738)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#738)
- _allowances[owner][spender] = amount (#552)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#553)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#738)
Reentrancy in FOXCOIN.transferFrom(address,address,uint256) (#664-668):
External calls:
- _transfer(sender,recipient,amount) (#665)
- recipient.transfer(amount) (#634)
External calls sending eth:
- _transfer(sender,recipient,amount) (#665)
- recipient.transfer(amount) (#634)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#766-773)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#666)
- _allowances[owner][spender] = amount (#552)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#553)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#666)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#274) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#275)
Prevent variables from having similar names.
Additional information: link
FOXCOIN.slitherConstructorVariables() (#402-796) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#413)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
FOXCOIN._decimals (#409) should be constant
FOXCOIN._name (#407) should be constant
FOXCOIN._symbol (#408) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
waiveOwnership() should be declared external:
- Ownable.waiveOwnership() (#168-171)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#173-177)
getUnlockTime() should be declared external:
- Ownable.getUnlockTime() (#179-181)
getTime() should be declared external:
- Ownable.getTime() (#183-185)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#187-192)
unlock() should be declared external:
- Ownable.unlock() (#194-199)
name() should be declared external:
- FOXCOIN.name() (#505-507)
symbol() should be declared external:
- FOXCOIN.symbol() (#509-511)
decimals() should be declared external:
- FOXCOIN.decimals() (#513-515)
totalSupply() should be declared external:
- FOXCOIN.totalSupply() (#517-519)
allowance(address,address) should be declared external:
- FOXCOIN.allowance(address,address) (#525-527)
increaseAllowance(address,uint256) should be declared external:
- FOXCOIN.increaseAllowance(address,uint256) (#529-532)
decreaseAllowance(address,uint256) should be declared external:
- FOXCOIN.decreaseAllowance(address,uint256) (#534-537)
minimumTokensBeforeSwapAmount() should be declared external:
- FOXCOIN.minimumTokensBeforeSwapAmount() (#539-541)
approve(address,uint256) should be declared external:
- FOXCOIN.approve(address,uint256) (#543-546)
setMarketPairStatus(address,bool) should be declared external:
- FOXCOIN.setMarketPairStatus(address,bool) (#556-558)
setIsExcludedFromFee(address,bool) should be declared external:
- FOXCOIN.setIsExcludedFromFee(address,bool) (#564-566)
setSwapAndLiquifyEnabled(bool) should be declared external:
- FOXCOIN.setSwapAndLiquifyEnabled(bool) (#620-623)
setSwapAndLiquifyByLimitOnly(bool) should be declared external:
- FOXCOIN.setSwapAndLiquifyByLimitOnly(bool) (#625-627)
getCirculatingSupply() should be declared external:
- FOXCOIN.getCirculatingSupply() (#629-631)
changeRouterVersion(address) should be declared external:
- FOXCOIN.changeRouterVersion(address) (#637-654)
transfer(address,uint256) should be declared external:
- FOXCOIN.transfer(address,uint256) (#659-662)
transferFrom(address,address,uint256) should be declared external:
- FOXCOIN.transferFrom(address,address,uint256) (#664-668)
Use the external attribute for functions never called from the contract.
Additional information: link
Unable to find website, listings and other project-related information
Token is marked as scam (rug pull, honeypot, phishing, etc.)
Additional information: link
Young tokens have high risks of scam / price dump / death
Token has no active CoinGecko listing / rank
Token has no active CoinMarketCap listing / rank
Unable to find Telegram and Twitter accounts