Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
MetaCar.addLiquidity(uint256,uint256) (#750-763) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in MetaCar._transfer(address,address,uint256) (#659-697):
External calls:
- swapAndLiquify(contractTokenBalance) (#681)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#739-745)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#681)
- recipient.transfer(amount) (#623)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#684)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#692)
- finalAmount = takeFee(sender,recipient,amount) (#686-687)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#777)
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.
Contract ticker (Lichun SPRING) contains non-alphanumeric characters.
Not a direct threat, but may indicate unreliable intentions of developer. Non-alphanumeric chars (,.;!#*&") are extremely rare among low risk tokens.
MetaCar.addLiquidity(uint256,uint256) (#750-763) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
Ensure that all the return values of the function calls are used.
Additional information: link
MetaCar.allowance(address,address).owner (#514) shadows:
- Ownable.owner() (#164-166) (function)
MetaCar._approve(address,address,uint256).owner (#537) shadows:
- Ownable.owner() (#164-166) (function)
Rename the local variables that shadow another component.
Additional information: link
MetaCar.setBuyTaxes(uint256,uint256,uint256) (#557-563) should emit an event for:
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#562)
MetaCar.setSelTaxes(uint256,uint256,uint256) (#565-571) should emit an event for:
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#570)
MetaCar.setDistributionSettings(uint256,uint256,uint256) (#573-579) should emit an event for:
- _liquidityShare = newLiquidityShare (#574)
- _teamShare = newTeamShare (#576)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#578)
MetaCar.setMaxTxAmount(uint256) (#581-583) should emit an event for:
- _maxTxAmount = maxTxAmount (#582)
MetaCar.setWalletLimit(uint256) (#593-595) should emit an event for:
- _walletMax = newLimit (#594)
MetaCar.setNumTokensBeforeSwap(uint256) (#597-599) should emit an event for:
- minimumTokensBeforeSwap = newLimit (#598)
Emit an event for critical parameter changes.
Additional information: link
MetaCar.setMarketingWalletAddress(address).newAddress (#601) lacks a zero-check on :
- marketingWalletAddress = address(newAddress) (#602)
MetaCar.setTeamWalletAddress(address).newAddress (#605) lacks a zero-check on :
- teamWalletAddress = address(newAddress) (#606)
Check that the address is not zero.
Additional information: link
Reentrancy in MetaCar.changeRouterVersion(address) (#626-643):
External calls:
- newPairAddress = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#634-635)
State variables written after the call(s):
- isMarketPair[address(uniswapPair)] = true (#642)
- isWalletLimitExempt[address(uniswapPair)] = true (#641)
- uniswapPair = newPairAddress (#638)
- uniswapV2Router = _uniswapV2Router (#639)
Reentrancy in MetaCar.constructor() (#463-492):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#468-469)
State variables written after the call(s):
- _allowances[address(this)][address(uniswapV2Router)] = _totalSupply (#472)
- _balances[_msgSender()] = _totalSupply (#490)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#479)
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#477)
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#478)
- isExcludedFromFee[owner()] = true (#474)
- isExcludedFromFee[address(this)] = true (#475)
- isMarketPair[address(uniswapPair)] = true (#488)
- isTxLimitExempt[owner()] = true (#485)
- isTxLimitExempt[address(this)] = true (#486)
- isWalletLimitExempt[owner()] = true (#481)
- isWalletLimitExempt[address(uniswapPair)] = true (#482)
- isWalletLimitExempt[address(this)] = true (#483)
- uniswapV2Router = _uniswapV2Router (#471)
Reentrancy in MetaCar.swapAndLiquify(uint256) (#706-728):
External calls:
- swapTokensForEth(tokensForSwap) (#711)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#739-745)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#727)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#721)
- recipient.transfer(amount) (#623)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#724)
- recipient.transfer(amount) (#623)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#727)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#727)
- _allowances[owner][spender] = amount (#541)
Reentrancy in MetaCar.transferFrom(address,address,uint256) (#653-657):
External calls:
- _transfer(sender,recipient,amount) (#654)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#739-745)
External calls sending eth:
- _transfer(sender,recipient,amount) (#654)
- recipient.transfer(amount) (#623)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#655)
- _allowances[owner][spender] = amount (#541)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in MetaCar._transfer(address,address,uint256) (#659-697):
External calls:
- swapAndLiquify(contractTokenBalance) (#681)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#739-745)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#681)
- recipient.transfer(amount) (#623)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#778)
- finalAmount = takeFee(sender,recipient,amount) (#686-687)
- Transfer(sender,recipient,finalAmount) (#694)
Reentrancy in MetaCar.constructor() (#463-492):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#468-469)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_totalSupply) (#491)
Reentrancy in MetaCar.swapAndLiquify(uint256) (#706-728):
External calls:
- swapTokensForEth(tokensForSwap) (#711)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#739-745)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#727)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#721)
- recipient.transfer(amount) (#623)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#724)
- recipient.transfer(amount) (#623)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#727)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#542)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#727)
Reentrancy in MetaCar.swapTokensForEth(uint256) (#730-748):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#739-745)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#747)
Reentrancy in MetaCar.transferFrom(address,address,uint256) (#653-657):
External calls:
- _transfer(sender,recipient,amount) (#654)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#739-745)
External calls sending eth:
- _transfer(sender,recipient,amount) (#654)
- recipient.transfer(amount) (#623)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#542)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#655)
Apply the check-effects-interactions pattern.
Additional information: link
Address.isContract(address) (#95-104) uses assembly
- INLINE ASM (#102)
Address._functionCallWithValue(address,bytes,uint256,string) (#131-148) uses assembly
- INLINE ASM (#140-143)
Do not use evm assembly.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#131-148) is never used and should be removed
Address.functionCall(address,bytes) (#114-116) is never used and should be removed
Address.functionCall(address,bytes,string) (#118-120) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#122-124) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#126-129) is never used and should be removed
Address.isContract(address) (#95-104) is never used and should be removed
Address.sendValue(address,uint256) (#106-112) is never used and should be removed
Context._msgData() (#22-25) is never used and should be removed
SafeMath.mod(uint256,uint256) (#83-85) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#87-90) is never used and should be removed
Remove unused functions.
Additional information: link
MetaCar._totalSupply (#427) is set pre-construction with a non-constant function or state variable:
- 20000000000 * 10 ** _decimals
MetaCar._maxTxAmount (#428) is set pre-construction with a non-constant function or state variable:
- 20000000000 * 10 ** _decimals
MetaCar._walletMax (#429) is set pre-construction with a non-constant function or state variable:
- 20000000000 * 10 ** _decimals
MetaCar.minimumTokensBeforeSwap (#430) is set pre-construction with a non-constant function or state variable:
- 100 * 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) (#106-112):
- (success) = recipient.call{value: amount}() (#110)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#131-148):
- (success,returndata) = target.call{value: weiValue}(data) (#134)
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() (#221) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#222) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#238) is not in mixedCase
Function IUniswapV2Router01.WETH() (#257) is not in mixedCase
Parameter MetaCar.setSwapAndLiquifyEnabled(bool)._enabled (#609) is not in mixedCase
Variable MetaCar._balances (#403) is not in mixedCase
Variable MetaCar._buyLiquidityFee (#411) is not in mixedCase
Variable MetaCar._buyMarketingFee (#412) is not in mixedCase
Variable MetaCar._buyTeamFee (#413) is not in mixedCase
Variable MetaCar._sellLiquidityFee (#415) is not in mixedCase
Variable MetaCar._sellMarketingFee (#416) is not in mixedCase
Variable MetaCar._sellTeamFee (#417) is not in mixedCase
Variable MetaCar._liquidityShare (#419) is not in mixedCase
Variable MetaCar._marketingShare (#420) is not in mixedCase
Variable MetaCar._teamShare (#421) is not in mixedCase
Variable MetaCar._totalTaxIfBuying (#423) is not in mixedCase
Variable MetaCar._totalTaxIfSelling (#424) is not in mixedCase
Variable MetaCar._totalDistributionShares (#425) is not in mixedCase
Variable MetaCar._maxTxAmount (#428) is not in mixedCase
Variable MetaCar._walletMax (#429) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#23)" inContext (#16-26)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in MetaCar._transfer(address,address,uint256) (#659-697):
External calls:
- swapAndLiquify(contractTokenBalance) (#681)
- recipient.transfer(amount) (#623)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#681)
- recipient.transfer(amount) (#623)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#684)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#692)
- finalAmount = takeFee(sender,recipient,amount) (#686-687)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#777)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#778)
- finalAmount = takeFee(sender,recipient,amount) (#686-687)
- Transfer(sender,recipient,finalAmount) (#694)
Reentrancy in MetaCar.swapAndLiquify(uint256) (#706-728):
External calls:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#721)
- recipient.transfer(amount) (#623)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#724)
- recipient.transfer(amount) (#623)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#721)
- recipient.transfer(amount) (#623)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#724)
- recipient.transfer(amount) (#623)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#727)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#727)
- _allowances[owner][spender] = amount (#541)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#542)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#727)
Reentrancy in MetaCar.transferFrom(address,address,uint256) (#653-657):
External calls:
- _transfer(sender,recipient,amount) (#654)
- recipient.transfer(amount) (#623)
External calls sending eth:
- _transfer(sender,recipient,amount) (#654)
- recipient.transfer(amount) (#623)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#755-762)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#655)
- _allowances[owner][spender] = amount (#541)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#542)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#655)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#262) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#263)
Prevent variables from having similar names.
Additional information: link
Ownable.waiveOwnership() (#173-176) uses literals with too many digits:
- OwnershipTransferred(_owner,address(0x000000000000000000000000000000000000dEaD)) (#174)
Ownable.waiveOwnership() (#173-176) uses literals with too many digits:
- _owner = address(0x000000000000000000000000000000000000dEaD) (#175)
MetaCar.slitherConstructorVariables() (#390-785) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#401)
MetaCar.slitherConstructorVariables() (#390-785) uses literals with too many digits:
- _totalSupply = 20000000000 * 10 ** _decimals (#427)
MetaCar.slitherConstructorVariables() (#390-785) uses literals with too many digits:
- _maxTxAmount = 20000000000 * 10 ** _decimals (#428)
MetaCar.slitherConstructorVariables() (#390-785) uses literals with too many digits:
- _walletMax = 20000000000 * 10 ** _decimals (#429)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
Ownable.asdasd (#153) is never used in MetaCar (#390-785)
Ownable._lockTime (#154) is never used in MetaCar (#390-785)
Remove unused state variables.
Additional information: link
MetaCar._decimals (#397) should be constant
MetaCar._name (#395) should be constant
MetaCar._symbol (#396) should be constant
Ownable._lockTime (#154) should be constant
Ownable.asdasd (#153) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
waiveOwnership() should be declared external:
- Ownable.waiveOwnership() (#173-176)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#178-182)
getTime() should be declared external:
- Ownable.getTime() (#184-186)
name() should be declared external:
- MetaCar.name() (#494-496)
symbol() should be declared external:
- MetaCar.symbol() (#498-500)
decimals() should be declared external:
- MetaCar.decimals() (#502-504)
totalSupply() should be declared external:
- MetaCar.totalSupply() (#506-508)
allowance(address,address) should be declared external:
- MetaCar.allowance(address,address) (#514-516)
increaseAllowance(address,uint256) should be declared external:
- MetaCar.increaseAllowance(address,uint256) (#518-521)
decreaseAllowance(address,uint256) should be declared external:
- MetaCar.decreaseAllowance(address,uint256) (#523-526)
minimumTokensBeforeSwapAmount() should be declared external:
- MetaCar.minimumTokensBeforeSwapAmount() (#528-530)
approve(address,uint256) should be declared external:
- MetaCar.approve(address,uint256) (#532-535)
setMarketPairStatus(address,bool) should be declared external:
- MetaCar.setMarketPairStatus(address,bool) (#545-547)
setIsExcludedFromFee(address,bool) should be declared external:
- MetaCar.setIsExcludedFromFee(address,bool) (#553-555)
setSwapAndLiquifyEnabled(bool) should be declared external:
- MetaCar.setSwapAndLiquifyEnabled(bool) (#609-612)
setSwapAndLiquifyByLimitOnly(bool) should be declared external:
- MetaCar.setSwapAndLiquifyByLimitOnly(bool) (#614-616)
getCirculatingSupply() should be declared external:
- MetaCar.getCirculatingSupply() (#618-620)
changeRouterVersion(address) should be declared external:
- MetaCar.changeRouterVersion(address) (#626-643)
transfer(address,uint256) should be declared external:
- MetaCar.transfer(address,uint256) (#648-651)
transferFrom(address,address,uint256) should be declared external:
- MetaCar.transferFrom(address,address,uint256) (#653-657)
Use the external attribute for functions never called from the contract.
Additional information: link
Unable to find website, listings and other project-related information
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 Twitter account
Telegram account has relatively few subscribers
Unable to find Blog account (Reddit or Medium)
Unable to find Youtube account
Unable to find Discord account