ROCKY [Rocky Inu] Token

RockyInu.addLiquidity(uint256,uint256) (#1150-1160) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
Ensure that an arbitrary user cannot withdraw unauthorized funds.

Additional information: link

Reentrancy in RockyInu._transfer(address,address,uint256) (#987-1100):
External calls:
- swapBack() (#1058)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1141-1147)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1132)
External calls sending eth:
- swapBack() (#1058)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1132)
State variables written after the call(s):
- _tokenTransfer(from,to,amount) (#1097)
- _liquidityTokensToSwap += tLiquidity * _buyLiquidityFee / _liquidityFee (#1375)
- _liquidityTokensToSwap += tLiquidity * _sellLiquidityFee / _liquidityFee (#1378)
- _tokenTransfer(from,to,amount) (#1097)
- _marketingTokensToSwap += tLiquidity * _buyMarketingFee / _liquidityFee (#1376)
- _marketingTokensToSwap += tLiquidity * _sellMarketingFee / _liquidityFee (#1379)
- _tokenTransfer(from,to,amount) (#1097)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1227)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1207)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1208)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1249)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1270)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1229)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1250)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1272)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#1383)
- _tokenTransfer(from,to,amount) (#1097)
- _rTotal = _rTotal.sub(rFee) (#1279)
- _tokenTransfer(from,to,amount) (#1097)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1248)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1269)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1228)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1271)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#1385)
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 ownership is not renounced (belongs to a wallet)

RockyInu._transfer(address,address,uint256) (#987-1100) uses a dangerous strict equality:
- from != owner() && to != uniswapV2Pair && block.number == tradingActiveBlock (#1009)
Don't use strict equality to determine if an account has enough Ether or tokens.

Additional information: link

Reentrancy in RockyInu.swapBack() (#1102-1134):
External calls:
- swapTokensForBNB(amountToSwapForBNB) (#1112)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1141-1147)
State variables written after the call(s):
- _liquidityTokensToSwap = 0 (#1120)
- _marketingTokensToSwap = 0 (#1121)
Apply the check-effects-interactions pattern.

Additional information: link

RockyInu._transfer(address,address,uint256) (#987-1100) uses tx.origin for authorization: require(bool,string)(_holderLastTransferTimestamp[tx.origin] < block.number,_transfer:: Transfer Delay enabled. Only one purchase per block allowed.) (#1022)
Do not use tx.origin for authorization.

Additional information: link

RockyInu.addLiquidity(uint256,uint256) (#1150-1160) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
Ensure that all the return values of the function calls are used.

Additional information: link

RockyInu.allowance(address,address).owner (#790) shadows:
- Ownable.owner() (#221-223) (function)
RockyInu._approve(address,address,uint256).owner (#976) shadows:
- Ownable.owner() (#221-223) (function)
Rename the local variables that shadow another component.

Additional information: link

RockyInu.setGasPriceLimit(uint256) (#904-907) should emit an event for:
- gasPriceLimit = gas * 1000000000 (#906)
Emit an event for critical parameter changes.

Additional information: link

Reentrancy in RockyInu._transfer(address,address,uint256) (#987-1100):
External calls:
- swapBack() (#1058)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1141-1147)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1132)
External calls sending eth:
- swapBack() (#1058)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1132)
State variables written after the call(s):
- removeAllFee() (#1066)
- _burnFee = 0 (#1415)
- removeAllFee() (#1071)
- _burnFee = 0 (#1415)
- _burnFee = _buyBurnFee (#1074)
- removeAllFee() (#1079)
- _burnFee = 0 (#1415)
- _burnFee = _sellBurnFee (#1082)
- removeAllFee() (#1092)
- _burnFee = 0 (#1415)
- _tokenTransfer(from,to,amount) (#1097)
- _burnFee = _previousBurnFee (#1421)
- _burnFee = 0 (#1415)
- restoreAllFee() (#1099)
- _burnFee = _previousBurnFee (#1421)
- removeAllFee() (#1066)
- _liquidityFee = 0 (#1414)
- removeAllFee() (#1071)
- _liquidityFee = 0 (#1414)
- _liquidityFee = _buyLiquidityFee + _buyMarketingFee (#1073)
- removeAllFee() (#1079)
- _liquidityFee = 0 (#1414)
- _liquidityFee = _sellLiquidityFee + _sellMarketingFee (#1081)
- _liquidityFee = _liquidityFee * 5 (#1087)
- removeAllFee() (#1092)
- _liquidityFee = 0 (#1414)
- _tokenTransfer(from,to,amount) (#1097)
- _liquidityFee = _previousLiquidityFee (#1420)
- _liquidityFee = 0 (#1414)
- restoreAllFee() (#1099)
- _liquidityFee = _previousLiquidityFee (#1420)
- removeAllFee() (#1066)
- _previousBurnFee = _burnFee (#1411)
- removeAllFee() (#1071)
- _previousBurnFee = _burnFee (#1411)
- removeAllFee() (#1079)
- _previousBurnFee = _burnFee (#1411)
- removeAllFee() (#1092)
- _previousBurnFee = _burnFee (#1411)
- _tokenTransfer(from,to,amount) (#1097)
- _previousBurnFee = _burnFee (#1411)
- removeAllFee() (#1066)
- _previousLiquidityFee = _liquidityFee (#1410)
- removeAllFee() (#1071)
- _previousLiquidityFee = _liquidityFee (#1410)
- removeAllFee() (#1079)
- _previousLiquidityFee = _liquidityFee (#1410)
- removeAllFee() (#1092)
- _previousLiquidityFee = _liquidityFee (#1410)
- _tokenTransfer(from,to,amount) (#1097)
- _previousLiquidityFee = _liquidityFee (#1410)
- removeAllFee() (#1066)
- _previousTaxFee = _taxFee (#1409)
- removeAllFee() (#1071)
- _previousTaxFee = _taxFee (#1409)
- removeAllFee() (#1079)
- _previousTaxFee = _taxFee (#1409)
- removeAllFee() (#1092)
- _previousTaxFee = _taxFee (#1409)
- _tokenTransfer(from,to,amount) (#1097)
- _previousTaxFee = _taxFee (#1409)
- _tokenTransfer(from,to,amount) (#1097)
- _tFeeTotal = _tFeeTotal.add(tFee) (#1280)
- removeAllFee() (#1066)
- _taxFee = 0 (#1413)
- removeAllFee() (#1071)
- _taxFee = 0 (#1413)
- _taxFee = _buyTaxFee (#1072)
- removeAllFee() (#1079)
- _taxFee = 0 (#1413)
- _taxFee = _sellTaxFee (#1080)
- _taxFee = _taxFee * 5 (#1086)
- removeAllFee() (#1092)
- _taxFee = 0 (#1413)
- _tokenTransfer(from,to,amount) (#1097)
- _taxFee = _previousTaxFee (#1419)
- _taxFee = 0 (#1413)
- restoreAllFee() (#1099)
- _taxFee = _previousTaxFee (#1419)
- buyOrSellSwitch = TRANSFER (#1067)
- buyOrSellSwitch = BUY (#1075)
- buyOrSellSwitch = SELL (#1083)
- buyOrSellSwitch = TRANSFER (#1093)
Reentrancy in RockyInu.changeRouterVersion(address) (#1491-1515):
External calls:
- _pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1505-1508)
State variables written after the call(s):
- uniswapV2Pair = _pair (#1510)
- uniswapV2Router = _uniswapV2Router (#1513)
Reentrancy in RockyInu.constructor() (#729-758):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#742-743)
State variables written after the call(s):
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#748)
- _excluded[i] = _excluded[_excluded.length - 1] (#957)
- _excluded.push(account) (#950)
- _excluded.pop() (#960)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#748)
- _isExcluded[account] = true (#949)
- _isExcluded[account] = false (#959)
- _isExcludedFromFee[newOwner] = true (#750)
- _isExcludedFromFee[address(this)] = true (#751)
- _isExcludedFromFee[marketingAddress] = true (#752)
- _isExcludedFromFee[liquidityAddress] = true (#753)
- transferOwnership(newOwner) (#757)
- _owner = newOwner (#241)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#748)
- _tOwned[account] = tokenFromReflection(_rOwned[account]) (#947)
- _tOwned[account] = 0 (#958)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#748)
- automatedMarketMakerPairs[pair] = value (#895)
- uniswapV2Pair = _uniswapV2Pair (#746)
- uniswapV2Router = _uniswapV2Router (#745)
Reentrancy in RockyInu.swapBack() (#1102-1134):
External calls:
- swapTokensForBNB(amountToSwapForBNB) (#1112)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1141-1147)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- addLiquidity(tokensForLiquidity,bnbForLiquidity) (#1126)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
External calls sending eth:
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- addLiquidity(tokensForLiquidity,bnbForLiquidity) (#1126)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
State variables written after the call(s):
- addLiquidity(tokensForLiquidity,bnbForLiquidity) (#1126)
- _allowances[owner][spender] = amount (#983)
Reentrancy in RockyInu.transferFrom(address,address,uint256) (#808-823):
External calls:
- _transfer(sender,recipient,amount) (#813)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1141-1147)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1132)
External calls sending eth:
- _transfer(sender,recipient,amount) (#813)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1132)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#814-821)
- _allowances[owner][spender] = amount (#983)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in RockyInu._transfer(address,address,uint256) (#987-1100):
External calls:
- swapBack() (#1058)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1141-1147)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1132)
External calls sending eth:
- swapBack() (#1058)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1132)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (#1211)
- _tokenTransfer(from,to,amount) (#1097)
- Transfer(sender,recipient,tTransferAmount) (#1253)
- _tokenTransfer(from,to,amount) (#1097)
- Transfer(sender,recipient,tTransferAmount) (#1232)
- _tokenTransfer(from,to,amount) (#1097)
- Transfer(sender,recipient,tTransferAmount) (#1275)
- _tokenTransfer(from,to,amount) (#1097)
Reentrancy in RockyInu.changeRouterVersion(address) (#1491-1515):
External calls:
- _pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1505-1508)
Event emitted after the call(s):
- UpdatedRouter(_router) (#1514)
Reentrancy in RockyInu.constructor() (#729-758):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#742-743)
Event emitted after the call(s):
- OwnershipTransferred(_owner,newOwner) (#240)
- transferOwnership(newOwner) (#757)
- Transfer(address(0),newOwner,_tTotal) (#755)
Reentrancy in RockyInu.swapBack() (#1102-1134):
External calls:
- swapTokensForBNB(amountToSwapForBNB) (#1112)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1141-1147)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- addLiquidity(tokensForLiquidity,bnbForLiquidity) (#1126)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
External calls sending eth:
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- addLiquidity(tokensForLiquidity,bnbForLiquidity) (#1126)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#984)
- addLiquidity(tokensForLiquidity,bnbForLiquidity) (#1126)
- SwapAndLiquify(amountToSwapForBNB,bnbForLiquidity,tokensForLiquidity) (#1127)
Reentrancy in RockyInu.transferForeignToken(address,address) (#1520-1530):
External calls:
- _sent = IERC20(_token).transfer(_to,_contractBalance) (#1528)
Event emitted after the call(s):
- TransferForeignToken(_token,_contractBalance) (#1529)
Reentrancy in RockyInu.transferFrom(address,address,uint256) (#808-823):
External calls:
- _transfer(sender,recipient,amount) (#813)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1141-1147)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1132)
External calls sending eth:
- _transfer(sender,recipient,amount) (#813)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#1152-1159)
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1132)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#984)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#814-821)
Apply the check-effects-interactions pattern.

Additional information: link

RockyInu._transfer(address,address,uint256) (#987-1100) uses timestamp for comparisons
Dangerous comparisons:
- boughtEarly[from] && earlyBuyPenaltyEnd <= block.number (#1085)
- require(bool,string)(! boughtEarly[from] || earlyBuyPenaltyEnd <= block.timestamp,Snipers can't transfer tokens to sell cheaper until penalty timeframe is over. DM a Mod.) (#1091)
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

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

RockyInu._previousTaxFee (#636) is set pre-construction with a non-constant function or state variable:
- _taxFee
RockyInu._previousLiquidityFee (#639) is set pre-construction with a non-constant function or state variable:
- _liquidityFee
RockyInu._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 RockyInu.swapBack() (#1102-1134):
- (success) = address(marketingAddress).call{value: bnbForMarketing}() (#1123)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#1132)
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 RockyInu.calculateTaxFee(uint256)._amount (#1388) is not in mixedCase
Parameter RockyInu.calculateLiquidityFee(uint256)._amount (#1392) is not in mixedCase
Parameter RockyInu.calculateBurnFee(uint256)._amount (#1400) is not in mixedCase
Parameter RockyInu.setMarketingAddress(address)._marketingAddress (#1468) is not in mixedCase
Parameter RockyInu.setLiquidityAddress(address)._liquidityAddress (#1475) is not in mixedCase
Parameter RockyInu.setSwapAndLiquifyEnabled(bool)._enabled (#1482) is not in mixedCase
Parameter RockyInu.changeRouterVersion(address)._router (#1491) is not in mixedCase
Parameter RockyInu.transferForeignToken(address,address)._token (#1520) is not in mixedCase
Parameter RockyInu.transferForeignToken(address,address)._to (#1520) is not in mixedCase
Constant RockyInu._tTotal (#616) is not in UPPER_CASE_WITH_UNDERSCORES
Constant RockyInu._name (#620) is not in UPPER_CASE_WITH_UNDERSCORES
Constant RockyInu._symbol (#621) is not in UPPER_CASE_WITH_UNDERSCORES
Constant RockyInu._decimals (#622) is not in UPPER_CASE_WITH_UNDERSCORES
Variable RockyInu._buyTaxFee (#644) is not in mixedCase
Variable RockyInu._buyLiquidityFee (#645) is not in mixedCase
Variable RockyInu._buyMarketingFee (#646) is not in mixedCase
Variable RockyInu._buyBurnFee (#647) is not in mixedCase
Variable RockyInu._sellTaxFee (#649) is not in mixedCase
Variable RockyInu._sellLiquidityFee (#650) is not in mixedCase
Variable RockyInu._sellMarketingFee (#651) is not in mixedCase
Variable RockyInu._sellBurnFee (#652) is not in mixedCase
Variable RockyInu._isExcludedMaxTransactionAmount (#663) 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 RockyInu._transferBothExcluded(address,address,uint256).rTransferAmount (#1263) is too similar to RockyInu._getValues(uint256).tTransferAmount (#1296)
Variable RockyInu._transferStandard(address,address,uint256).rTransferAmount (#1201) is too similar to RockyInu._transferStandard(address,address,uint256).tTransferAmount (#1203)
Variable RockyInu._transferStandard(address,address,uint256).rTransferAmount (#1201) is too similar to RockyInu._getValues(uint256).tTransferAmount (#1296)
Variable RockyInu._transferStandard(address,address,uint256).rTransferAmount (#1201) is too similar to RockyInu._getTValues(uint256).tTransferAmount (#1327)
Variable RockyInu._transferFromExcluded(address,address,uint256).rTransferAmount (#1242) is too similar to RockyInu._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable RockyInu._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1348) is too similar to RockyInu._getValues(uint256).tTransferAmount (#1296)
Variable RockyInu._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1348) is too similar to RockyInu._getTValues(uint256).tTransferAmount (#1327)
Variable RockyInu._transferFromExcluded(address,address,uint256).rTransferAmount (#1242) is too similar to RockyInu._transferFromExcluded(address,address,uint256).tTransferAmount (#1244)
Variable RockyInu._transferBothExcluded(address,address,uint256).rTransferAmount (#1263) is too similar to RockyInu._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable RockyInu.reflectionFromToken(uint256,bool).rTransferAmount (#925) is too similar to RockyInu._getValues(uint256).tTransferAmount (#1296)
Variable RockyInu._transferStandard(address,address,uint256).rTransferAmount (#1201) is too similar to RockyInu._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable RockyInu._transferBothExcluded(address,address,uint256).rTransferAmount (#1263) is too similar to RockyInu._transferFromExcluded(address,address,uint256).tTransferAmount (#1244)
Variable RockyInu._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1348) is too similar to RockyInu._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable RockyInu._transferStandard(address,address,uint256).rTransferAmount (#1201) is too similar to RockyInu._transferFromExcluded(address,address,uint256).tTransferAmount (#1244)
Variable RockyInu._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1348) is too similar to RockyInu._transferFromExcluded(address,address,uint256).tTransferAmount (#1244)
Variable RockyInu._transferFromExcluded(address,address,uint256).rTransferAmount (#1242) is too similar to RockyInu._getValues(uint256).tTransferAmount (#1296)
Variable RockyInu._getValues(uint256).rTransferAmount (#1300) is too similar to RockyInu._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable RockyInu._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1348) is too similar to RockyInu._transferStandard(address,address,uint256).tTransferAmount (#1203)
Variable RockyInu._transferBothExcluded(address,address,uint256).rTransferAmount (#1263) is too similar to RockyInu._getTValues(uint256).tTransferAmount (#1327)
Variable RockyInu._transferToExcluded(address,address,uint256).rTransferAmount (#1221) is too similar to RockyInu._transferFromExcluded(address,address,uint256).tTransferAmount (#1244)
Variable RockyInu._transferStandard(address,address,uint256).rTransferAmount (#1201) is too similar to RockyInu._transferToExcluded(address,address,uint256).tTransferAmount (#1223)
Variable RockyInu.reflectionFromToken(uint256,bool).rTransferAmount (#925) is too similar to RockyInu._transferFromExcluded(address,address,uint256).tTransferAmount (#1244)
Variable RockyInu._getValues(uint256).rTransferAmount (#1300) is too similar to RockyInu._transferStandard(address,address,uint256).tTransferAmount (#1203)
Variable RockyInu._transferFromExcluded(address,address,uint256).rTransferAmount (#1242) is too similar to RockyInu._getTValues(uint256).tTransferAmount (#1327)
Variable RockyInu._transferFromExcluded(address,address,uint256).rTransferAmount (#1242) is too similar to RockyInu._transferStandard(address,address,uint256).tTransferAmount (#1203)
Variable RockyInu._getValues(uint256).rTransferAmount (#1300) is too similar to RockyInu._getTValues(uint256).tTransferAmount (#1327)
Variable RockyInu._transferToExcluded(address,address,uint256).rTransferAmount (#1221) is too similar to RockyInu._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable RockyInu.reflectionFromToken(uint256,bool).rTransferAmount (#925) is too similar to RockyInu._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable RockyInu._transferToExcluded(address,address,uint256).rTransferAmount (#1221) is too similar to RockyInu._getTValues(uint256).tTransferAmount (#1327)
Variable RockyInu._transferToExcluded(address,address,uint256).rTransferAmount (#1221) is too similar to RockyInu._transferStandard(address,address,uint256).tTransferAmount (#1203)
Variable RockyInu.reflectionFromToken(uint256,bool).rTransferAmount (#925) is too similar to RockyInu._transferStandard(address,address,uint256).tTransferAmount (#1203)
Variable RockyInu._transferBothExcluded(address,address,uint256).rTransferAmount (#1263) is too similar to RockyInu._transferToExcluded(address,address,uint256).tTransferAmount (#1223)
Variable RockyInu.reflectionFromToken(uint256,bool).rTransferAmount (#925) is too similar to RockyInu._getTValues(uint256).tTransferAmount (#1327)
Variable RockyInu._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1348) is too similar to RockyInu._transferToExcluded(address,address,uint256).tTransferAmount (#1223)
Variable RockyInu._getValues(uint256).rTransferAmount (#1300) is too similar to RockyInu._getValues(uint256).tTransferAmount (#1296)
Variable RockyInu._getValues(uint256).rTransferAmount (#1300) is too similar to RockyInu._transferToExcluded(address,address,uint256).tTransferAmount (#1223)
Variable RockyInu._transferFromExcluded(address,address,uint256).rTransferAmount (#1242) is too similar to RockyInu._transferToExcluded(address,address,uint256).tTransferAmount (#1223)
Variable RockyInu._transferToExcluded(address,address,uint256).rTransferAmount (#1221) is too similar to RockyInu._getValues(uint256).tTransferAmount (#1296)
Variable RockyInu._getValues(uint256).rTransferAmount (#1300) is too similar to RockyInu._transferFromExcluded(address,address,uint256).tTransferAmount (#1244)
Variable RockyInu._transferToExcluded(address,address,uint256).rTransferAmount (#1221) is too similar to RockyInu._transferToExcluded(address,address,uint256).tTransferAmount (#1223)
Variable RockyInu.reflectionFromToken(uint256,bool).rTransferAmount (#925) is too similar to RockyInu._transferToExcluded(address,address,uint256).tTransferAmount (#1223)
Variable RockyInu._transferBothExcluded(address,address,uint256).rTransferAmount (#1263) is too similar to RockyInu._transferStandard(address,address,uint256).tTransferAmount (#1203)
Prevent variables from having similar names.

Additional information: link

RockyInu.slitherConstructorVariables() (#593-1533) uses literals with too many digits:
- liquidityAddress = address(0x000000000000000000000000000000000000dEaD) (#600-601)
RockyInu.slitherConstructorVariables() (#593-1533) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#603-604)
RockyInu.slitherConstructorVariables() (#593-1533) uses literals with too many digits:
- gasMaxLimit = 1000000 * 1000000000 (#667)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

Ownable._previousOwner (#208) is never used in RockyInu (#593-1533)
Remove unused state variables.

Additional information: link

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:
- RockyInu.excludeFromMaxTransaction(address,bool) (#880-882)
setSwapAndLiquifyEnabled(bool) should be declared external:
- RockyInu.setSwapAndLiquifyEnabled(bool) (#1482-1485)
Use the external attribute for functions never called from the contract.

Additional information: link

RockyInu.includeInReward(address) (#953-964) has costly operations inside a loop:
- _excluded.pop() (#960)
Use a local variable to hold the loop computation result.

Additional information: link

Contract has 14% buy tax and 24% sell tax.
Taxes are suspiciously high (over 10%) and contract ownership is not renounced. Token has a high risk of becoming a honeypot.

Average 30d number of PancakeSwap swaps is low.

Token is deployed only at one blockchain

Token is not listed at Mobula.Finance

Additional information: link

Token has relatively low CoinMarketCap rank

Young tokens have high risks of price dump / death

Young tokens have high risks of price dump / death

Young tokens have high risks of price dump / death

Token has relatively low CoinGecko rank