Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
EthCat.swapBack() (#370-411) sends eth to arbitrary user
Dangerous calls:
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#395)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#397)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in EthCat._transferFrom(address,address,uint256) (#294-323):
External calls:
- swapBack() (#312)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#381-387)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#395)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#397)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#401-408)
External calls sending eth:
- swapBack() (#312)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#395)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#397)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#401-408)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#316)
- _balances[recipient] = _balances[recipient].add(amountReceived) (#319)
- amountReceived = takeFee(sender,recipient,amount) (#318)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#349)
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 ($MEOW) 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.
EthCat.slitherConstructorVariables() (#190-507) performs a multiplication on the result of a division:
-swapThreshold = _totalSupply / 1000 * 3 (#231)
Consider ordering multiplication before division.
Additional information: link
EthCat.swapBack() (#370-411) ignores return value by router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#401-408)
Ensure that all the return values of the function calls are used.
Additional information: link
EthCat.setFees(uint256,uint256,uint256,uint256) (#440-447) should emit an event for:
- liquidityFee = _liquidityFee (#441)
- marketingFee = _marketingFee (#442)
- buybackFee = _buybackFee (#443)
- totalFee = _liquidityFee.add(_marketingFee).add(_buybackFee) (#444)
- feeDenominator = _feeDenominator (#445)
EthCat.setSellMultiplier(uint256) (#462-464) should emit an event for:
- _sellMultiplier = multiplier (#463)
EthCat.setTxLimit(uint256) (#470-473) should emit an event for:
- _maxTxAmount = amountBuy (#471)
Emit an event for critical parameter changes.
Additional information: link
Auth.transferOwnership(address).adr (#129) lacks a zero-check on :
- owner = adr (#130)
EthCat.setFeeReceiver(address,address)._marketingFeeReceiver (#465) lacks a zero-check on :
- marketingFeeReceiver = _marketingFeeReceiver (#466)
EthCat.setFeeReceiver(address,address)._buybackFeeReceiver (#465) lacks a zero-check on :
- buybackFeeReceiver = _buybackFeeReceiver (#467)
Check that the address is not zero.
Additional information: link
Reentrancy in EthCat._transferFrom(address,address,uint256) (#294-323):
External calls:
- swapBack() (#312)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#381-387)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#395)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#397)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#401-408)
External calls sending eth:
- swapBack() (#312)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#395)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#397)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#401-408)
State variables written after the call(s):
- launch() (#314)
- launchedAt = block.number (#431)
Reentrancy in EthCat.constructor() (#240-260):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#242)
State variables written after the call(s):
- _allowances[address(this)][address(router)] = type()(uint256).max (#243)
- _balances[_owner] = _totalSupply (#258)
- isFeeExempt[msg.sender] = true (#246)
- isTimelockExempt[msg.sender] = true (#253)
- isTimelockExempt[DEAD] = true (#254)
- isTimelockExempt[address(this)] = true (#255)
- isTxLimitExempt[msg.sender] = true (#248)
- isTxLimitExempt[address(this)] = true (#249)
- isTxLimitExempt[routerAddress] = true (#250)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in EthCat._transferFrom(address,address,uint256) (#294-323):
External calls:
- swapBack() (#312)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#381-387)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#395)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#397)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#401-408)
External calls sending eth:
- swapBack() (#312)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#395)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#397)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#401-408)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#350)
- amountReceived = takeFee(sender,recipient,amount) (#318)
- Transfer(sender,recipient,amountReceived) (#321)
Reentrancy in EthCat.constructor() (#240-260):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#242)
Event emitted after the call(s):
- Transfer(address(0),_owner,_totalSupply) (#259)
Reentrancy in EthCat.swapBack() (#370-411):
External calls:
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#381-387)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#395)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#397)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#401-408)
External calls sending eth:
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#395)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#397)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#401-408)
Event emitted after the call(s):
- AutoLiquify(amountBNBLiquidity,amountToLiquify) (#409)
Apply the check-effects-interactions pattern.
Additional information: link
EthCat._transferFrom(address,address,uint256) (#294-323) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(cooldownTimer[recipient] < block.timestamp,Please wait for 1min between two operations) (#309)
Avoid relying on block.timestamp.
Additional information: link
EthCat.buyTokens(uint256,address) (#413-424) is never used and should be removed
Remove unused functions.
Additional information: link
EthCat._maxTxAmount (#203) is set pre-construction with a non-constant function or state variable:
- (_totalSupply * 1) / 100
EthCat._maxWalletSize (#204) is set pre-construction with a non-constant function or state variable:
- (_totalSupply * 2) / 100
EthCat.swapThreshold (#231) is set pre-construction with a non-constant function or state variable:
- _totalSupply / 1000 * 3
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
Pragma version^0.8.11 (#11) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
solc-0.8.11 is not recommended for deployment
Deploy with any of the following Solidity versions: 0.5.16 - 0.5.17, 0.6.11 - 0.6.12, 0.7.5 - 0.7.6 Use a simple pragma version that allows any of these versions. Consider using the latest version of Solidity for testing.
Additional information: link
Low level call in EthCat.swapBack() (#370-411):
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#395)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#397)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Variable Auth._intAddr (#76) is not in mixedCase
Function IDEXRouter.WETH() (#144) is not in mixedCase
Parameter EthCat.tradingStatus(bool)._status (#356) is not in mixedCase
Parameter EthCat.setFees(uint256,uint256,uint256,uint256)._liquidityFee (#440) is not in mixedCase
Parameter EthCat.setFees(uint256,uint256,uint256,uint256)._marketingFee (#440) is not in mixedCase
Parameter EthCat.setFees(uint256,uint256,uint256,uint256)._buybackFee (#440) is not in mixedCase
Parameter EthCat.setFees(uint256,uint256,uint256,uint256)._feeDenominator (#440) is not in mixedCase
Parameter EthCat.cooldownEnabled(bool,uint8)._status (#449) is not in mixedCase
Parameter EthCat.cooldownEnabled(bool,uint8)._interval (#449) is not in mixedCase
Parameter EthCat.setFeeReceiver(address,address)._marketingFeeReceiver (#465) is not in mixedCase
Parameter EthCat.setFeeReceiver(address,address)._buybackFeeReceiver (#465) is not in mixedCase
Parameter EthCat.setSwapBackSettings(bool,uint256)._enabled (#474) is not in mixedCase
Parameter EthCat.setSwapBackSettings(bool,uint256)._amount (#474) is not in mixedCase
Parameter EthCat.blacklistAddress(address,bool)._address (#479) is not in mixedCase
Parameter EthCat.blacklistAddress(address,bool)._value (#479) is not in mixedCase
Parameter EthCat.transferForeignToken(address)._token (#487) is not in mixedCase
Variable EthCat.WBNB (#193) is not in mixedCase
Variable EthCat.DEAD (#194) is not in mixedCase
Variable EthCat.ZERO (#195) is not in mixedCase
Constant EthCat._name (#198) is not in UPPER_CASE_WITH_UNDERSCORES
Constant EthCat._symbol (#199) is not in UPPER_CASE_WITH_UNDERSCORES
Constant EthCat._decimals (#200) is not in UPPER_CASE_WITH_UNDERSCORES
Variable EthCat._totalSupply (#202) is not in mixedCase
Variable EthCat._maxTxAmount (#203) is not in mixedCase
Variable EthCat._maxWalletSize (#204) is not in mixedCase
Variable EthCat._balances (#206) is not in mixedCase
Variable EthCat._allowances (#207) is not in mixedCase
Variable EthCat._sellMultiplier (#219) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Variable IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#149) is too similar to IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#150)
Prevent variables from having similar names.
Additional information: link
EthCat.slitherConstructorVariables() (#190-507) uses literals with too many digits:
- DEAD = 0x000000000000000000000000000000000000dEaD (#194)
EthCat.slitherConstructorVariables() (#190-507) uses literals with too many digits:
- ZERO = 0x0000000000000000000000000000000000000000 (#195)
EthCat.slitherConstructorVariables() (#190-507) uses literals with too many digits:
- _totalSupply = 10000000 * (10 ** _decimals) (#202)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
EthCat.DEAD (#194) should be constant
EthCat.WBNB (#193) should be constant
EthCat.ZERO (#195) should be constant
EthCat._totalSupply (#202) should be constant
EthCat.routerAddress (#196) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
authorize(address) should be declared external:
- Auth.authorize(address) (#100-102)
unauthorize(address) should be declared external:
- Auth.unauthorize(address) (#108-110)
transferOwnership(address) should be declared external:
- Auth.transferOwnership(address) (#129-133)
tradingStatus(bool) should be declared external:
- EthCat.tradingStatus(bool) (#356-361)
cooldownEnabled(bool,uint8) should be declared external:
- EthCat.cooldownEnabled(bool,uint8) (#449-452)
blacklistAddress(address,bool) should be declared external:
- EthCat.blacklistAddress(address,bool) (#479-481)
transferForeignToken(address) should be declared external:
- EthCat.transferForeignToken(address) (#487-491)
isOverLiquified(uint256,uint256) should be declared external:
- EthCat.isOverLiquified(uint256,uint256) (#501-503)
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 Telegram and Twitter accounts