Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
cat.addLiquidity(uint256,uint256) (#742-755) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in cat._transfer(address,address,uint256) (#642-691):
External calls:
- swapAndLiquify(contractTokenBalance) (#675)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#731-737)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#675)
- recipient.transfer(amount) (#606)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#678)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#686)
- finalAmount = takeFee(sender,recipient,amount) (#680-681)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#769)
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 (Condom Cat) 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.
cat.addLiquidity(uint256,uint256) (#742-755) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
Ensure that all the return values of the function calls are used.
Additional information: link
cat.allowance(address,address).owner (#501) shadows:
- Ownable.owner() (#150-152) (function)
cat._approve(address,address,uint256).owner (#524) shadows:
- Ownable.owner() (#150-152) (function)
Rename the local variables that shadow another component.
Additional information: link
cat.setBuyTaxes(uint256,uint256) (#544-549) should emit an event for:
- _buyLiquidityFee = newLiquidityTax (#545)
- _buyMarketingFee = newMarketingTax (#546)
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee) (#548)
cat.setSellTaxes(uint256,uint256) (#551-557) should emit an event for:
- _sellLiquidityFee = newLiquidityTax (#552)
- _sellMarketingFee = newMarketingTax (#553)
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee) (#556)
cat.setDistributionSettings(uint256,uint256) (#559-565) should emit an event for:
- _liquidityShare = newLiquidityShare (#560)
- _totalDistributionShares = _liquidityShare.add(_marketingShare) (#564)
cat.setMaxTxAmount(uint256) (#567-569) should emit an event for:
- _maxTxAmount = maxTxAmount (#568)
cat.setWalletLimit(uint256) (#579-581) should emit an event for:
- _walletMax = newLimit (#580)
cat.setNumTokensBeforeSwap(uint256) (#583-585) should emit an event for:
- minimumTokensBeforeSwap = newLimit (#584)
cat.UPfeeTXtime(uint256) (#784-786) should emit an event for:
- feeTXtime = _feeTXtime (#785)
cat.UPendtime(uint256) (#788-790) should emit an event for:
- endtime = _endtime (#789)
Emit an event for critical parameter changes.
Additional information: link
cat.setMarketingWalletAddress(address).newAddress (#587) lacks a zero-check on :
- marketingWalletAddress = address(newAddress) (#588)
Check that the address is not zero.
Additional information: link
Reentrancy in cat.changeRouterVersion(address) (#609-626):
External calls:
- newPairAddress = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#617-618)
State variables written after the call(s):
- isMarketPair[address(uniswapPair)] = true (#625)
- isWalletLimitExempt[address(uniswapPair)] = true (#624)
- uniswapPair = newPairAddress (#621)
- uniswapV2Router = _uniswapV2Router (#622)
Reentrancy in cat.constructor() (#452-479):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#456-457)
State variables written after the call(s):
- _allowances[address(this)][address(uniswapV2Router)] = _totalSupply (#460)
- _balances[_msgSender()] = _totalSupply (#477)
- _totalDistributionShares = _liquidityShare.add(_marketingShare) (#467)
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee) (#465)
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee) (#466)
- isExcludedFromFee[owner()] = true (#462)
- isExcludedFromFee[address(this)] = true (#463)
- isMarketPair[address(uniswapPair)] = true (#475)
- isTxLimitExempt[owner()] = true (#472)
- isTxLimitExempt[address(this)] = true (#473)
- isWalletLimitExempt[owner()] = true (#468)
- isWalletLimitExempt[address(uniswapPair)] = true (#469)
- isWalletLimitExempt[address(this)] = true (#470)
- uniswapV2Router = _uniswapV2Router (#459)
Reentrancy in cat.swapAndLiquify(uint256) (#700-720):
External calls:
- swapTokensForEth(tokensForSwap) (#705)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#731-737)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#719)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#715)
- recipient.transfer(amount) (#606)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#719)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#719)
- _allowances[owner][spender] = amount (#528)
Reentrancy in cat.transferFrom(address,address,uint256) (#636-640):
External calls:
- _transfer(sender,recipient,amount) (#637)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#731-737)
External calls sending eth:
- _transfer(sender,recipient,amount) (#637)
- recipient.transfer(amount) (#606)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#638)
- _allowances[owner][spender] = amount (#528)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in cat._transfer(address,address,uint256) (#642-691):
External calls:
- swapAndLiquify(contractTokenBalance) (#675)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#731-737)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#675)
- recipient.transfer(amount) (#606)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#770)
- finalAmount = takeFee(sender,recipient,amount) (#680-681)
- Transfer(sender,recipient,finalAmount) (#688)
Reentrancy in cat.constructor() (#452-479):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#456-457)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_totalSupply) (#478)
Reentrancy in cat.swapAndLiquify(uint256) (#700-720):
External calls:
- swapTokensForEth(tokensForSwap) (#705)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#731-737)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#719)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#715)
- recipient.transfer(amount) (#606)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#719)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#529)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#719)
Reentrancy in cat.swapTokensForEth(uint256) (#722-740):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#731-737)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#739)
Reentrancy in cat.transferFrom(address,address,uint256) (#636-640):
External calls:
- _transfer(sender,recipient,amount) (#637)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#731-737)
External calls sending eth:
- _transfer(sender,recipient,amount) (#637)
- recipient.transfer(amount) (#606)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#529)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#638)
Apply the check-effects-interactions pattern.
Additional information: link
cat._transfer(address,address,uint256) (#642-691) uses timestamp for comparisons
Dangerous comparisons:
- endtime == 0 (#647)
- endtime > block.timestamp (#650)
Avoid relying on block.timestamp.
Additional information: link
Address.isContract(address) (#83-92) uses assembly
- INLINE ASM (#90)
Address._functionCallWithValue(address,bytes,uint256,string) (#119-136) uses assembly
- INLINE ASM (#128-131)
Do not use evm assembly.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#119-136) is never used and should be removed
Address.functionCall(address,bytes) (#102-104) is never used and should be removed
Address.functionCall(address,bytes,string) (#106-108) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#110-112) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#114-117) is never used and should be removed
Address.isContract(address) (#83-92) is never used and should be removed
Address.sendValue(address,uint256) (#94-100) is never used and should be removed
Context._msgData() (#10-13) is never used and should be removed
SafeMath.mod(uint256,uint256) (#71-73) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#75-78) is never used and should be removed
Remove unused functions.
Additional information: link
cat._liquidityShare (#407) is set pre-construction with a non-constant function or state variable:
- _buyLiquidityFee.add(_sellLiquidityFee)
cat._marketingShare (#408) is set pre-construction with a non-constant function or state variable:
- _buyMarketingFee.add(_sellMarketingFee)
cat._totalSupply (#414) is set pre-construction with a non-constant function or state variable:
- 100000000 * 10 ** _decimals
cat._maxTxAmount (#415) is set pre-construction with a non-constant function or state variable:
- _totalSupply.div(1)
cat._walletMax (#416) is set pre-construction with a non-constant function or state variable:
- _totalSupply.div(1)
cat.minimumTokensBeforeSwap (#417) is set pre-construction with a non-constant function or state variable:
- _totalSupply.div(10000)
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) (#94-100):
- (success) = recipient.call{value: amount}() (#98)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#119-136):
- (success,returndata) = target.call{value: weiValue}(data) (#122)
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() (#209) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#210) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#226) is not in mixedCase
Function IUniswapV2Router01.WETH() (#245) is not in mixedCase
Contract cat (#378-792) is not in CapWords
Parameter cat.setSwapAndLiquifyEnabled(bool)._enabled (#592) is not in mixedCase
Parameter cat.setblocklist(address)._account (#776) is not in mixedCase
Function cat.UPfeeTXtime(uint256) (#784-786) is not in mixedCase
Parameter cat.UPfeeTXtime(uint256)._feeTXtime (#784) is not in mixedCase
Function cat.UPendtime(uint256) (#788-790) is not in mixedCase
Parameter cat.UPendtime(uint256)._endtime (#788) is not in mixedCase
Variable cat._balances (#390) is not in mixedCase
Variable cat._buyLiquidityFee (#401) is not in mixedCase
Variable cat._buyMarketingFee (#402) is not in mixedCase
Variable cat._sellLiquidityFee (#404) is not in mixedCase
Variable cat._sellMarketingFee (#405) is not in mixedCase
Variable cat._liquidityShare (#407) is not in mixedCase
Variable cat._marketingShare (#408) is not in mixedCase
Variable cat._totalTaxIfBuying (#410) is not in mixedCase
Variable cat._totalTaxIfSelling (#411) is not in mixedCase
Variable cat._totalDistributionShares (#412) is not in mixedCase
Variable cat._maxTxAmount (#415) is not in mixedCase
Variable cat._walletMax (#416) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#11)" inContext (#4-14)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in cat._transfer(address,address,uint256) (#642-691):
External calls:
- swapAndLiquify(contractTokenBalance) (#675)
- recipient.transfer(amount) (#606)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#675)
- recipient.transfer(amount) (#606)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#678)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#686)
- finalAmount = takeFee(sender,recipient,amount) (#680-681)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#769)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#770)
- finalAmount = takeFee(sender,recipient,amount) (#680-681)
- Transfer(sender,recipient,finalAmount) (#688)
Reentrancy in cat.swapAndLiquify(uint256) (#700-720):
External calls:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#715)
- recipient.transfer(amount) (#606)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#715)
- recipient.transfer(amount) (#606)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#719)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#719)
- _allowances[owner][spender] = amount (#528)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#529)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#719)
Reentrancy in cat.transferFrom(address,address,uint256) (#636-640):
External calls:
- _transfer(sender,recipient,amount) (#637)
- recipient.transfer(amount) (#606)
External calls sending eth:
- _transfer(sender,recipient,amount) (#637)
- recipient.transfer(amount) (#606)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#747-754)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#638)
- _allowances[owner][spender] = amount (#528)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#529)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#638)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#250) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#251)
Prevent variables from having similar names.
Additional information: link
cat.slitherConstructorVariables() (#378-792) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#388)
cat.slitherConstructorVariables() (#378-792) uses literals with too many digits:
- _totalSupply = 100000000 * 10 ** _decimals (#414)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
cat._decimals (#385) should be constant
cat._name (#383) should be constant
cat._symbol (#384) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
waiveOwnership() should be declared external:
- Ownable.waiveOwnership() (#159-162)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#164-168)
getTime() should be declared external:
- Ownable.getTime() (#171-173)
name() should be declared external:
- cat.name() (#481-483)
symbol() should be declared external:
- cat.symbol() (#485-487)
decimals() should be declared external:
- cat.decimals() (#489-491)
totalSupply() should be declared external:
- cat.totalSupply() (#493-495)
allowance(address,address) should be declared external:
- cat.allowance(address,address) (#501-503)
increaseAllowance(address,uint256) should be declared external:
- cat.increaseAllowance(address,uint256) (#505-508)
decreaseAllowance(address,uint256) should be declared external:
- cat.decreaseAllowance(address,uint256) (#510-513)
minimumTokensBeforeSwapAmount() should be declared external:
- cat.minimumTokensBeforeSwapAmount() (#515-517)
approve(address,uint256) should be declared external:
- cat.approve(address,uint256) (#519-522)
setMarketPairStatus(address,bool) should be declared external:
- cat.setMarketPairStatus(address,bool) (#532-534)
setIsExcludedFromFee(address,bool) should be declared external:
- cat.setIsExcludedFromFee(address,bool) (#540-542)
setSwapAndLiquifyEnabled(bool) should be declared external:
- cat.setSwapAndLiquifyEnabled(bool) (#592-595)
setSwapAndLiquifyByLimitOnly(bool) should be declared external:
- cat.setSwapAndLiquifyByLimitOnly(bool) (#597-599)
getCirculatingSupply() should be declared external:
- cat.getCirculatingSupply() (#601-603)
changeRouterVersion(address) should be declared external:
- cat.changeRouterVersion(address) (#609-626)
transfer(address,uint256) should be declared external:
- cat.transfer(address,uint256) (#631-634)
transferFrom(address,address,uint256) should be declared external:
- cat.transferFrom(address,address,uint256) (#636-640)
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