Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
CatKing.addLiquidity(uint256,uint256) (#1147-1157) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in CatKing._transfer(address,address,uint256) (#983-1097):
External calls:
- swapBack() (#1055)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1138-1144)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1129)
External calls sending eth:
- swapBack() (#1055)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1129)
State variables written after the call(s):
- _tokenTransfer(from,to,amount) (#1094)
- _liquidityTokensToSwap += tLiquidity * _buyLiquidityFee / _liquidityFee (#1372)
- _liquidityTokensToSwap += tLiquidity * _sellLiquidityFee / _liquidityFee (#1375)
- _tokenTransfer(from,to,amount) (#1094)
- _marketingTokensToSwap += tLiquidity * _buyMarketingFee / _liquidityFee (#1373)
- _marketingTokensToSwap += tLiquidity * _sellMarketingFee / _liquidityFee (#1376)
- _tokenTransfer(from,to,amount) (#1094)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1224)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1204)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1205)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1246)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1267)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1247)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1226)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#1380)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1269)
- _tokenTransfer(from,to,amount) (#1094)
- _rTotal = _rTotal.sub(rFee) (#1276)
- _tokenTransfer(from,to,amount) (#1094)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1245)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1266)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1225)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1268)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#1382)
Apply the check-effects-interactions pattern.
Additional information: link
CatKing.earlyBuyPenaltyEnd (#656) is never initialized. It is used in:
- CatKing._transfer(address,address,uint256) (#983-1097)
Initialize all the variables. If a variable is meant to be initialized to zero, explicitly set it to zero to improve code readability.
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.
Reentrancy in CatKing.swapBack() (#1099-1131):
External calls:
- swapTokensForBNB(amountToSwapForBNB) (#1109)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1138-1144)
State variables written after the call(s):
- _liquidityTokensToSwap = 0 (#1117)
- _marketingTokensToSwap = 0 (#1118)
Apply the check-effects-interactions pattern.
Additional information: link
CatKing._transfer(address,address,uint256) (#983-1097) uses tx.origin for authorization: require(bool,string)(_holderLastTransferTimestamp[tx.origin] < block.number,_transfer:: Transfer Delay enabled. Only one purchase per block allowed.) (#1018)
Do not use tx.origin for authorization.
Additional information: link
CatKing.addLiquidity(uint256,uint256) (#1147-1157) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
Ensure that all the return values of the function calls are used.
Additional information: link
CatKing.allowance(address,address).owner (#794) shadows:
- Ownable.owner() (#221-223) (function)
CatKing._approve(address,address,uint256).owner (#972) shadows:
- Ownable.owner() (#221-223) (function)
Rename the local variables that shadow another component.
Additional information: link
CatKing.setGasPriceLimit(uint256) (#900-903) should emit an event for:
- gasPriceLimit = gas * 1000000000 (#902)
CatKing.setMinimumTokensBeforeSwap(uint256) (#1488-1490) should emit an event for:
- minimumTokensBeforeSwap = newAmt * (10 ** 9) (#1489)
CatKing.setMaxBuyTxAmount(uint256) (#1492-1495) should emit an event for:
- _maxBuyTxAmount = maxBuyTxAmount * (10 ** 9) (#1494)
CatKing.setMaxSellTxAmount(uint256) (#1497-1500) should emit an event for:
- _maxSellTxAmount = maxSellTxAmount * (10 ** 9) (#1499)
Emit an event for critical parameter changes.
Additional information: link
Reentrancy in CatKing._transfer(address,address,uint256) (#983-1097):
External calls:
- swapBack() (#1055)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1138-1144)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1129)
External calls sending eth:
- swapBack() (#1055)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1129)
State variables written after the call(s):
- removeAllFee() (#1063)
- _burnFee = 0 (#1412)
- removeAllFee() (#1068)
- _burnFee = 0 (#1412)
- _burnFee = _buyBurnFee (#1071)
- removeAllFee() (#1076)
- _burnFee = 0 (#1412)
- _burnFee = _sellBurnFee (#1079)
- removeAllFee() (#1089)
- _burnFee = 0 (#1412)
- _tokenTransfer(from,to,amount) (#1094)
- _burnFee = _previousBurnFee (#1418)
- _burnFee = 0 (#1412)
- restoreAllFee() (#1096)
- _burnFee = _previousBurnFee (#1418)
- removeAllFee() (#1063)
- _liquidityFee = 0 (#1411)
- removeAllFee() (#1068)
- _liquidityFee = 0 (#1411)
- _liquidityFee = _buyLiquidityFee + _buyMarketingFee (#1070)
- removeAllFee() (#1076)
- _liquidityFee = 0 (#1411)
- _liquidityFee = _sellLiquidityFee + _sellMarketingFee (#1078)
- _liquidityFee = _liquidityFee * 5 (#1084)
- removeAllFee() (#1089)
- _liquidityFee = 0 (#1411)
- _tokenTransfer(from,to,amount) (#1094)
- _liquidityFee = _previousLiquidityFee (#1417)
- _liquidityFee = 0 (#1411)
- restoreAllFee() (#1096)
- _liquidityFee = _previousLiquidityFee (#1417)
- removeAllFee() (#1063)
- _previousBurnFee = _burnFee (#1408)
- removeAllFee() (#1068)
- _previousBurnFee = _burnFee (#1408)
- removeAllFee() (#1076)
- _previousBurnFee = _burnFee (#1408)
- removeAllFee() (#1089)
- _previousBurnFee = _burnFee (#1408)
- _tokenTransfer(from,to,amount) (#1094)
- _previousBurnFee = _burnFee (#1408)
- removeAllFee() (#1063)
- _previousLiquidityFee = _liquidityFee (#1407)
- removeAllFee() (#1068)
- _previousLiquidityFee = _liquidityFee (#1407)
- removeAllFee() (#1076)
- _previousLiquidityFee = _liquidityFee (#1407)
- removeAllFee() (#1089)
- _previousLiquidityFee = _liquidityFee (#1407)
- _tokenTransfer(from,to,amount) (#1094)
- _previousLiquidityFee = _liquidityFee (#1407)
- removeAllFee() (#1063)
- _previousTaxFee = _taxFee (#1406)
- removeAllFee() (#1068)
- _previousTaxFee = _taxFee (#1406)
- removeAllFee() (#1076)
- _previousTaxFee = _taxFee (#1406)
- removeAllFee() (#1089)
- _previousTaxFee = _taxFee (#1406)
- _tokenTransfer(from,to,amount) (#1094)
- _previousTaxFee = _taxFee (#1406)
- _tokenTransfer(from,to,amount) (#1094)
- _tFeeTotal = _tFeeTotal.add(tFee) (#1277)
- removeAllFee() (#1063)
- _taxFee = 0 (#1410)
- removeAllFee() (#1068)
- _taxFee = 0 (#1410)
- _taxFee = _buyTaxFee (#1069)
- removeAllFee() (#1076)
- _taxFee = 0 (#1410)
- _taxFee = _sellTaxFee (#1077)
- _taxFee = _taxFee * 5 (#1083)
- removeAllFee() (#1089)
- _taxFee = 0 (#1410)
- _tokenTransfer(from,to,amount) (#1094)
- _taxFee = _previousTaxFee (#1416)
- _taxFee = 0 (#1410)
- restoreAllFee() (#1096)
- _taxFee = _previousTaxFee (#1416)
- buyOrSellSwitch = TRANSFER (#1064)
- buyOrSellSwitch = BUY (#1072)
- buyOrSellSwitch = SELL (#1080)
- buyOrSellSwitch = TRANSFER (#1090)
Reentrancy in CatKing.changeRouterVersion(address) (#1502-1526):
External calls:
- _pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1516-1519)
State variables written after the call(s):
- uniswapV2Pair = _pair (#1521)
- uniswapV2Router = _uniswapV2Router (#1524)
Reentrancy in CatKing.constructor() (#731-762):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#746-747)
State variables written after the call(s):
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#752)
- _excluded[i] = _excluded[_excluded.length - 1] (#953)
- _excluded.push(account) (#946)
- _excluded.pop() (#956)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#752)
- _isExcluded[account] = true (#945)
- _isExcluded[account] = false (#955)
- _isExcludedFromFee[newOwner] = true (#754)
- _isExcludedFromFee[address(this)] = true (#755)
- _isExcludedFromFee[marketingAddress] = true (#756)
- _isExcludedFromFee[liquidityAddress] = true (#757)
- transferOwnership(newOwner) (#761)
- _owner = newOwner (#241)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#752)
- _tOwned[account] = tokenFromReflection(_rOwned[account]) (#943)
- _tOwned[account] = 0 (#954)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#752)
- automatedMarketMakerPairs[pair] = value (#891)
- uniswapV2Pair = _uniswapV2Pair (#750)
- uniswapV2Router = _uniswapV2Router (#749)
Reentrancy in CatKing.swapBack() (#1099-1131):
External calls:
- swapTokensForBNB(amountToSwapForBNB) (#1109)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1138-1144)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- addLiquidity(tokensForLiquidity,bnbForLiquidity) (#1123)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
External calls sending eth:
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- addLiquidity(tokensForLiquidity,bnbForLiquidity) (#1123)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
State variables written after the call(s):
- addLiquidity(tokensForLiquidity,bnbForLiquidity) (#1123)
- _allowances[owner][spender] = amount (#979)
Reentrancy in CatKing.transferFrom(address,address,uint256) (#812-827):
External calls:
- _transfer(sender,recipient,amount) (#817)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1138-1144)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1129)
External calls sending eth:
- _transfer(sender,recipient,amount) (#817)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1129)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#818-825)
- _allowances[owner][spender] = amount (#979)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in CatKing._transfer(address,address,uint256) (#983-1097):
External calls:
- swapBack() (#1055)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1138-1144)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1129)
External calls sending eth:
- swapBack() (#1055)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1129)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (#1208)
- _tokenTransfer(from,to,amount) (#1094)
- Transfer(sender,recipient,tTransferAmount) (#1229)
- _tokenTransfer(from,to,amount) (#1094)
- Transfer(sender,recipient,tTransferAmount) (#1250)
- _tokenTransfer(from,to,amount) (#1094)
- Transfer(sender,recipient,tTransferAmount) (#1272)
- _tokenTransfer(from,to,amount) (#1094)
Reentrancy in CatKing.changeRouterVersion(address) (#1502-1526):
External calls:
- _pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1516-1519)
Event emitted after the call(s):
- UpdatedRouter(_router) (#1525)
Reentrancy in CatKing.constructor() (#731-762):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#746-747)
Event emitted after the call(s):
- OwnershipTransferred(_owner,newOwner) (#240)
- transferOwnership(newOwner) (#761)
- Transfer(address(0),newOwner,_tTotal) (#759)
Reentrancy in CatKing.swapBack() (#1099-1131):
External calls:
- swapTokensForBNB(amountToSwapForBNB) (#1109)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1138-1144)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- addLiquidity(tokensForLiquidity,bnbForLiquidity) (#1123)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
External calls sending eth:
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- addLiquidity(tokensForLiquidity,bnbForLiquidity) (#1123)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#980)
- addLiquidity(tokensForLiquidity,bnbForLiquidity) (#1123)
- SwapAndLiquify(amountToSwapForBNB,bnbForLiquidity,tokensForLiquidity) (#1124)
Reentrancy in CatKing.transferForeignToken(address,address) (#1531-1541):
External calls:
- _sent = IERC20(_token).transfer(_to,_contractBalance) (#1539)
Event emitted after the call(s):
- TransferForeignToken(_token,_contractBalance) (#1540)
Reentrancy in CatKing.transferFrom(address,address,uint256) (#812-827):
External calls:
- _transfer(sender,recipient,amount) (#817)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1138-1144)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1129)
External calls sending eth:
- _transfer(sender,recipient,amount) (#817)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1149-1156)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1129)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#980)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#818-825)
Apply the check-effects-interactions pattern.
Additional information: link
CatKing._transfer(address,address,uint256) (#983-1097) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(! boughtEarly[from] || earlyBuyPenaltyEnd <= block.timestamp,Snipers can't transfer tokens to sell cheaper until penalty timeframe is over. DM a Mod.) (#1088)
Avoid relying on block.timestamp.
Additional information: link
Address.isContract(address) (#111-122) uses assembly
- INLINE ASM (#118-120)
Address._functionCallWithValue(address,bytes,uint256,string) (#180-203) uses assembly
- INLINE ASM (#195-198)
Do not use evm assembly.
Additional information: link
CatKing.includeInReward(address) (#949-960) has costly operations inside a loop:
- _excluded.pop() (#956)
Use a local variable to hold the loop computation result.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#180-203) is never used and should be removed
Address.functionCall(address,bytes) (#138-143) is never used and should be removed
Address.functionCall(address,bytes,string) (#145-151) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#153-165) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#167-178) is never used and should be removed
Address.isContract(address) (#111-122) is never used and should be removed
Address.sendValue(address,uint256) (#124-136) is never used and should be removed
Context._msgData() (#10-13) is never used and should be removed
SafeMath.mod(uint256,uint256) (#96-98) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#100-107) is never used and should be removed
Remove unused functions.
Additional information: link
CatKing._previousTaxFee (#636) is set pre-construction with a non-constant function or state variable:
- _taxFee
CatKing._previousLiquidityFee (#639) is set pre-construction with a non-constant function or state variable:
- _liquidityFee
CatKing._previousBurnFee (#642) is set pre-construction with a non-constant function or state variable:
- _burnFee
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 version0.8.9 (#3) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
solc-0.8.9 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 Address.sendValue(address,uint256) (#124-136):
- (success) = recipient.call{value: amount}() (#131)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#180-203):
- (success,returndata) = target.call{value: weiValue}(data) (#188-190)
Low level call in CatKing.swapBack() (#1099-1131):
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1120)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1129)
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() (#314) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#316) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#346) is not in mixedCase
Function IUniswapV2Router01.WETH() (#390) is not in mixedCase
Parameter CatKing.calculateTaxFee(uint256)._amount (#1385) is not in mixedCase
Parameter CatKing.calculateLiquidityFee(uint256)._amount (#1389) is not in mixedCase
Parameter CatKing.calculateBurnFee(uint256)._amount (#1397) is not in mixedCase
Parameter CatKing.setMarketingAddress(address)._marketingAddress (#1465) is not in mixedCase
Parameter CatKing.setLiquidityAddress(address)._liquidityAddress (#1472) is not in mixedCase
Parameter CatKing.setSwapAndLiquifyEnabled(bool)._enabled (#1479) is not in mixedCase
Parameter CatKing.changeRouterVersion(address)._router (#1502) is not in mixedCase
Parameter CatKing.transferForeignToken(address,address)._token (#1531) is not in mixedCase
Parameter CatKing.transferForeignToken(address,address)._to (#1531) is not in mixedCase
Constant CatKing._tTotal (#616) is not in UPPER_CASE_WITH_UNDERSCORES
Constant CatKing._name (#620) is not in UPPER_CASE_WITH_UNDERSCORES
Constant CatKing._symbol (#621) is not in UPPER_CASE_WITH_UNDERSCORES
Constant CatKing._decimals (#622) is not in UPPER_CASE_WITH_UNDERSCORES
Variable CatKing._isExcludedMaxTransactionAmount (#663) is not in mixedCase
Variable CatKing._maxBuyTxAmount (#674) is not in mixedCase
Variable CatKing._maxSellTxAmount (#675) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#11)" inContext (#5-14)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#395) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#396)
Variable CatKing.reflectionFromToken(uint256,bool).rTransferAmount (#921) is too similar to CatKing._transferStandard(address,address,uint256).tTransferAmount (#1200)
Variable CatKing._getValues(uint256).rTransferAmount (#1297) is too similar to CatKing._transferToExcluded(address,address,uint256).tTransferAmount (#1220)
Variable CatKing._transferFromExcluded(address,address,uint256).rTransferAmount (#1239) is too similar to CatKing._transferStandard(address,address,uint256).tTransferAmount (#1200)
Variable CatKing._transferBothExcluded(address,address,uint256).rTransferAmount (#1260) is too similar to CatKing._transferFromExcluded(address,address,uint256).tTransferAmount (#1241)
Variable CatKing._getValues(uint256).rTransferAmount (#1297) is too similar to CatKing._transferStandard(address,address,uint256).tTransferAmount (#1200)
Variable CatKing._transferBothExcluded(address,address,uint256).rTransferAmount (#1260) is too similar to CatKing._transferBothExcluded(address,address,uint256).tTransferAmount (#1262)
Variable CatKing._transferToExcluded(address,address,uint256).rTransferAmount (#1218) is too similar to CatKing._transferStandard(address,address,uint256).tTransferAmount (#1200)
Variable CatKing._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1345) is too similar to CatKing._transferFromExcluded(address,address,uint256).tTransferAmount (#1241)
Variable CatKing._transferStandard(address,address,uint256).rTransferAmount (#1198) is too similar to CatKing._transferStandard(address,address,uint256).tTransferAmount (#1200)
Variable CatKing._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1345) is too similar to CatKing._transferBothExcluded(address,address,uint256).tTransferAmount (#1262)
Variable CatKing.reflectionFromToken(uint256,bool).rTransferAmount (#921) is too similar to CatKing._transferFromExcluded(address,address,uint256).tTransferAmount (#1241)
Variable CatKing._transferBothExcluded(address,address,uint256).rTransferAmount (#1260) is too similar to CatKing._transferToExcluded(address,address,uint256).tTransferAmount (#1220)
Variable CatKing.reflectionFromToken(uint256,bool).rTransferAmount (#921) is too similar to CatKing._transferBothExcluded(address,address,uint256).tTransferAmount (#1262)
Variable CatKing._transferFromExcluded(address,address,uint256).rTransferAmount (#1239) is too similar to CatKing._transferFromExcluded(address,address,uint256).tTransferAmount (#1241)
Variable CatKing._transferFromExcluded(address,address,uint256).rTransferAmount (#1239) is too similar to CatKing._transferBothExcluded(address,address,uint256).tTransferAmount (#1262)
Variable CatKing._getValues(uint256).rTransferAmount (#1297) is too similar to CatKing._transferFromExcluded(address,address,uint256).tTransferAmount (#1241)
Variable CatKing._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1345) is too similar to CatKing._transferToExcluded(address,address,uint256).tTransferAmount (#1220)
Variable CatKing._transferToExcluded(address,address,uint256).rTransferAmount (#1218) is too similar to CatKing._transferFromExcluded(address,address,uint256).tTransferAmount (#1241)
Variable CatKing._getValues(uint256).rTransferAmount (#1297) is too similar to CatKing._transferBothExcluded(address,address,uint256).tTransferAmount (#1262)
Variable CatKing._transferBothExcluded(address,address,uint256).rTransferAmount (#1260) is too similar to CatKing._transferStandard(address,address,uint256).tTransferAmount (#1200)
Variable CatKing.reflectionFromToken(uint256,bool).rTransferAmount (#921) is too similar to CatKing._transferToExcluded(address,address,uint256).tTransferAmount (#1220)
Variable CatKing._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1345) is too similar to CatKing._transferStandard(address,address,uint256).tTransferAmount (#1200)
Variable CatKing._transferFromExcluded(address,address,uint256).rTransferAmount (#1239) is too similar to CatKing._transferToExcluded(address,address,uint256).tTransferAmount (#1220)
Variable CatKing.reflectionFromToken(uint256,bool).rTransferAmount (#921) is too similar to CatKing._getValues(uint256).tTransferAmount (#1293)
Variable CatKing._transferToExcluded(address,address,uint256).rTransferAmount (#1218) is too similar to CatKing._transferToExcluded(address,address,uint256).tTransferAmount (#1220)
Variable CatKing._transferStandard(address,address,uint256).rTransferAmount (#1198) is too similar to CatKing._transferBothExcluded(address,address,uint256).tTransferAmount (#1262)
Variable CatKing._transferFromExcluded(address,address,uint256).rTransferAmount (#1239) is too similar to CatKing._getValues(uint256).tTransferAmount (#1293)
Variable CatKing._getValues(uint256).rTransferAmount (#1297) is too similar to CatKing._getValues(uint256).tTransferAmount (#1293)
Variable CatKing._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1345) is too similar to CatKing._getValues(uint256).tTransferAmount (#1293)
Variable CatKing._transferToExcluded(address,address,uint256).rTransferAmount (#1218) is too similar to CatKing._getTValues(uint256).tTransferAmount (#1324)
Variable CatKing.reflectionFromToken(uint256,bool).rTransferAmount (#921) is too similar to CatKing._getTValues(uint256).tTransferAmount (#1324)
Variable CatKing._transferBothExcluded(address,address,uint256).rTransferAmount (#1260) is too similar to CatKing._getValues(uint256).tTransferAmount (#1293)
Variable CatKing._transferStandard(address,address,uint256).rTransferAmount (#1198) is too similar to CatKing._transferToExcluded(address,address,uint256).tTransferAmount (#1220)
Variable CatKing._transferStandard(address,address,uint256).rTransferAmount (#1198) is too similar to CatKing._transferFromExcluded(address,address,uint256).tTransferAmount (#1241)
Variable CatKing._transferStandard(address,address,uint256).rTransferAmount (#1198) is too similar to CatKing._getTValues(uint256).tTransferAmount (#1324)
Variable CatKing._transferToExcluded(address,address,uint256).rTransferAmount (#1218) is too similar to CatKing._getValues(uint256).tTransferAmount (#1293)
Variable CatKing._transferToExcluded(address,address,uint256).rTransferAmount (#1218) is too similar to CatKing._transferBothExcluded(address,address,uint256).tTransferAmount (#1262)
Variable CatKing._transferStandard(address,address,uint256).rTransferAmount (#1198) is too similar to CatKing._getValues(uint256).tTransferAmount (#1293)
Variable CatKing._getValues(uint256).rTransferAmount (#1297) is too similar to CatKing._getTValues(uint256).tTransferAmount (#1324)
Variable CatKing._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1345) is too similar to CatKing._getTValues(uint256).tTransferAmount (#1324)
Variable CatKing._transferFromExcluded(address,address,uint256).rTransferAmount (#1239) is too similar to CatKing._getTValues(uint256).tTransferAmount (#1324)
Variable CatKing._transferBothExcluded(address,address,uint256).rTransferAmount (#1260) is too similar to CatKing._getTValues(uint256).tTransferAmount (#1324)
Prevent variables from having similar names.
Additional information: link
CatKing.slitherConstructorVariables() (#593-1544) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#603-604)
CatKing.slitherConstructorVariables() (#593-1544) uses literals with too many digits:
- gasMaxLimit = 1000000 * 1000000000 (#667)
CatKing.slitherConstructorConstantVariables() (#593-1544) uses literals with too many digits:
- _tTotal = 1000000000000 * 1e9 (#616)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
Ownable._previousOwner (#208) is never used in CatKing (#593-1544)
Remove unused state variables.
Additional information: link
CatKing.earlyBuyPenaltyEnd (#656) should be constant
CatKing.tradingActive (#682) should be constant
CatKing.tradingActiveBlock (#654) should be constant
Ownable._previousOwner (#208) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
getTime() should be declared external:
- Ownable.getTime() (#244-246)
excludeFromMaxTransaction(address,bool) should be declared external:
- CatKing.excludeFromMaxTransaction(address,bool) (#876-878)
setSwapAndLiquifyEnabled(bool) should be declared external:
- CatKing.setSwapAndLiquifyEnabled(bool) (#1479-1482)
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