BLRS [BULL ROCK SWAP] Token

Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)

Reentrancy in BullRockSwap._transfer(address,address,uint256) (#1074-1121):
External calls:
- swapAndLiquify(contractTokenBalance) (#1108)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1155-1161)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1108)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#937)
- _rOwned[marketWallet] = _rOwned[marketWallet].add(rMarketFee) (#946)
- _rOwned[owner()] = _rOwned[owner()].add(rCreatorFee) (#955)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1240)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1228)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1267)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1229)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1254)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1242)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1255)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1269)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _rTotal = _rTotal.sub(rFee) (#930)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#939)
- _tOwned[marketWallet] = _tOwned[marketWallet].add(tMarketFee) (#948)
- _tOwned[owner()] = _tOwned[owner()].add(tCreatorFee) (#957)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1253)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1266)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1241)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1268)
Apply the check-effects-interactions pattern.

Additional information: link

BullRockSwap._transfer(address,address,uint256) (#1074-1121) compares to a boolean constant:
-from != owner() && selling == false (#1081)
Remove the equality to the boolean constant.

Additional information: link

BullRockSwap.includeInReward(address) (#867-878) has costly operations inside a loop:
- _excluded.pop() (#874)
Use a local variable to hold the loop computation result.

Additional information: link

BullRockSwap.addLiquidity(uint256,uint256) (#1164-1177) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
Ensure that all the return values of the function calls are used.

Additional information: link

BullRockSwap.allowance(address,address).owner (#782) shadows:
- Ownable.owner() (#413-415) (function)
BullRockSwap._approve(address,address,uint256).owner (#1066) shadows:
- Ownable.owner() (#413-415) (function)
Rename the local variables that shadow another component.

Additional information: link

BullRockSwap.setHolderPercent(uint256) (#888-890) should emit an event for:
- _holdersPercent = percentage (#889)
BullRockSwap.setLiquidityFeePercent(uint256) (#892-894) should emit an event for:
- _liquidityFee = liquidityFee (#893)
BullRockSwap.setMarketFeePercent(uint256) (#896-898) should emit an event for:
- _marketFee = marketFee (#897)
BullRockSwap.setCreatorFeePercent(uint256) (#900-902) should emit an event for:
- _creatorFee = CreatorFee (#901)
BullRockSwap.setBurnFeePercent(uint256) (#904-906) should emit an event for:
- _burnFee = burnFee (#905)
BullRockSwap.setMaxTxAmount(uint256) (#908-910) should emit an event for:
- _maxTxAmount = maxTxAmount (#909)
BullRockSwap.setNumTokensSellToAddToLiquidity(uint256) (#912-914) should emit an event for:
- _numTokensSellToAddToLiquidity = minAmountForLiquidity (#913)
Emit an event for critical parameter changes.

Additional information: link

BullRockSwap.constructor(address)._marketWallet (#735) lacks a zero-check on :
- marketWallet = _marketWallet (#737)
Check that the address is not zero.

Additional information: link

Reentrancy in BullRockSwap._transfer(address,address,uint256) (#1074-1121):
External calls:
- swapAndLiquify(contractTokenBalance) (#1108)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1155-1161)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1108)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _burnFee = _previousBurnFee (#1059)
- _burnFee = 0 (#1051)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _creatorFee = _previousCreatorFee (#1058)
- _creatorFee = 0 (#1050)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _holdersPercent = _previousTaxFee (#1055)
- _holdersPercent = 0 (#1047)
- _holdersPercent = fee (#1223)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _liquidityFee = _previousLiquidityFee (#1056)
- _liquidityFee = 0 (#1048)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _marketFee = _previousMarketFee (#1057)
- _marketFee = 0 (#1049)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _previousBurnFee = _burnFee (#1045)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _previousCreatorFee = _creatorFee (#1044)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _previousLiquidityFee = _liquidityFee (#1042)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _previousMarketFee = _marketFee (#1043)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _previousTaxFee = _holdersPercent (#1041)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- _tFeeTotal = _tFeeTotal.add(tFee) (#931)
Reentrancy in BullRockSwap.constructor(address) (#735-754):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#742-743)
State variables written after the call(s):
- _isExcludedFromFee[owner()] = true (#749)
- _isExcludedFromFee[address(this)] = true (#750)
- _isExcludedFromFee[marketWallet] = true (#751)
- uniswapV2Router = _uniswapV2Router (#746)
Reentrancy in BullRockSwap.swapAndLiquify(uint256) (#1123-1144):
External calls:
- swapTokensForEth(half) (#1135)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1155-1161)
- addLiquidity(otherHalf,newBalance) (#1141)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1141)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (#1141)
- _allowances[owner][spender] = amount (#1070)
Reentrancy in BullRockSwap.transferFrom(address,address,uint256) (#791-795):
External calls:
- _transfer(sender,recipient,amount) (#792)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1155-1161)
External calls sending eth:
- _transfer(sender,recipient,amount) (#792)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,BEP20: transfer amount exceeds allowance)) (#793)
- _allowances[owner][spender] = amount (#1070)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in BullRockSwap._transfer(address,address,uint256) (#1074-1121):
External calls:
- swapAndLiquify(contractTokenBalance) (#1108)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1155-1161)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1108)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
Event emitted after the call(s):
- Transfer(_msgSender(),burnAddress,tBurnFee) (#963)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- Transfer(_msgSender(),address(this),tLiquidity) (#940)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- Transfer(_msgSender(),owner(),tCreatorFee) (#958)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- Transfer(_msgSender(),marketWallet,tMarketFee) (#949)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- Transfer(sender,recipient,tTransferAmount) (#1235)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- Transfer(sender,recipient,tTransferAmount) (#1261)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- Transfer(sender,recipient,tTransferAmount) (#1248)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
- Transfer(sender,recipient,tTransferAmount) (#1275)
- _tokenTransfer(from,to,amount,takeFee) (#1120)
Reentrancy in BullRockSwap.constructor(address) (#735-754):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#742-743)
Event emitted after the call(s):
- Transfer(address(0),owner(),_tTotal) (#753)
Reentrancy in BullRockSwap.swapAndLiquify(uint256) (#1123-1144):
External calls:
- swapTokensForEth(half) (#1135)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1155-1161)
- addLiquidity(otherHalf,newBalance) (#1141)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1141)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1071)
- addLiquidity(otherHalf,newBalance) (#1141)
- SwapAndLiquify(half,newBalance,otherHalf) (#1143)
Reentrancy in BullRockSwap.transferFrom(address,address,uint256) (#791-795):
External calls:
- _transfer(sender,recipient,amount) (#792)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1155-1161)
External calls sending eth:
- _transfer(sender,recipient,amount) (#792)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1169-1176)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1071)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,BEP20: transfer amount exceeds allowance)) (#793)
Apply the check-effects-interactions pattern.

Additional information: link

Ownable.unlock() (#460-465) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(now > _lockTime,Contract is locked until 7 days) (#462)
Avoid relying on block.timestamp.

Additional information: link

Address.isContract(address) (#266-275) uses assembly
- INLINE ASM (#273)
Address._functionCallWithValue(address,bytes,uint256,string) (#359-380) uses assembly
- INLINE ASM (#372-375)
Do not use evm assembly.

Additional information: link

Address._functionCallWithValue(address,bytes,uint256,string) (#359-380) is never used and should be removed
Address.functionCall(address,bytes) (#319-321) is never used and should be removed
Address.functionCall(address,bytes,string) (#329-331) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#344-346) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#354-357) is never used and should be removed
Address.isContract(address) (#266-275) is never used and should be removed
Address.sendValue(address,uint256) (#293-299) is never used and should be removed
BullRockSwap._getRBurnFeeValues(uint256,uint256) (#1016-1019) is never used and should be removed
BullRockSwap._getRDevFeeValues(uint256,uint256) (#1001-1004) is never used and should be removed
Context._msgData() (#238-241) is never used and should be removed
SafeMath.mod(uint256,uint256) (#211-213) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#227-230) is never used and should be removed
Remove unused functions.

Additional information: link

BullRockSwap._rTotal (#686) is set pre-construction with a non-constant function or state variable:
- (MAX - (MAX % _tTotal))
BullRockSwap._previousTaxFee (#694) is set pre-construction with a non-constant function or state variable:
- _holdersPercent
BullRockSwap._previousLiquidityFee (#697) is set pre-construction with a non-constant function or state variable:
- _liquidityFee
BullRockSwap._previousMarketFee (#700) is set pre-construction with a non-constant function or state variable:
- _marketFee
BullRockSwap._previousCreatorFee (#703) is set pre-construction with a non-constant function or state variable:
- _creatorFee
BullRockSwap._previousBurnFee (#706) 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

Low level call in Address.sendValue(address,uint256) (#293-299):
- (success) = recipient.call{value: amount}() (#297)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#359-380):
- (success,returndata) = target.call{value: weiValue}(data) (#363)
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() (#499) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#500) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#517) is not in mixedCase
Function IUniswapV2Router01.WETH() (#537) is not in mixedCase
Parameter BullRockSwap.setCreatorFeePercent(uint256).CreatorFee (#900) is not in mixedCase
Parameter BullRockSwap.setSwapAndLiquifyEnabled(bool)._enabled (#916) is not in mixedCase
Parameter BullRockSwap.setSelling(bool)._enabled (#921) is not in mixedCase
Variable BullRockSwap._holdersPercent (#693) is not in mixedCase
Variable BullRockSwap._liquidityFee (#696) is not in mixedCase
Variable BullRockSwap._marketFee (#699) is not in mixedCase
Variable BullRockSwap._creatorFee (#702) is not in mixedCase
Variable BullRockSwap._burnFee (#705) is not in mixedCase
Variable BullRockSwap._maxTxAmount (#718) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#239)" inContext (#233-242)
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 (#542) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#543)
Variable BullRockSwap._creatorFee (#702) is too similar to BullRockSwap._takeCreatorFee(uint256).rCreatorFee (#954)
Variable BullRockSwap._creatorFee (#702) is too similar to BullRockSwap._getRCreatorFeeValues(uint256,uint256).rCreatorFee (#1012)
Variable BullRockSwap._takeCreatorFee(uint256).rCreatorFee (#954) is too similar to BullRockSwap._getTCreatorFeeValues(uint256).tCreatorFee (#985)
Variable BullRockSwap._getRCreatorFeeValues(uint256,uint256).rCreatorFee (#1012) is too similar to BullRockSwap._getTCreatorFeeValues(uint256).tCreatorFee (#985)
Variable BullRockSwap._transferFromExcluded(address,address,uint256).rTransferAmount (#1252) is too similar to BullRockSwap._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable BullRockSwap._transferFromExcluded(address,address,uint256).rTransferAmount (#1252) is too similar to BullRockSwap._transferToExcluded(address,address,uint256).tTransferAmount (#1239)
Variable BullRockSwap._transferStandard(address,address,uint256).rTransferAmount (#1227) is too similar to BullRockSwap._transferToExcluded(address,address,uint256).tTransferAmount (#1239)
Variable BullRockSwap._transferStandard(address,address,uint256).rTransferAmount (#1227) is too similar to BullRockSwap._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable BullRockSwap._transferBothExcluded(address,address,uint256).rTransferAmount (#1265) is too similar to BullRockSwap._getValues(uint256).tTransferAmount (#967)
Variable BullRockSwap._getRValues(uint256,uint256,uint256).rTransferAmount (#997) is too similar to BullRockSwap._transferToExcluded(address,address,uint256).tTransferAmount (#1239)
Variable BullRockSwap._transferStandard(address,address,uint256).rTransferAmount (#1227) is too similar to BullRockSwap._getValues(uint256).tTransferAmount (#967)
Variable BullRockSwap._transferFromExcluded(address,address,uint256).rTransferAmount (#1252) is too similar to BullRockSwap._getValues(uint256).tTransferAmount (#967)
Variable BullRockSwap._transferBothExcluded(address,address,uint256).rTransferAmount (#1265) is too similar to BullRockSwap._transferStandard(address,address,uint256).tTransferAmount (#1227)
Variable BullRockSwap._transferFromExcluded(address,address,uint256).rTransferAmount (#1252) is too similar to BullRockSwap._transferStandard(address,address,uint256).tTransferAmount (#1227)
Variable BullRockSwap._transferBothExcluded(address,address,uint256).rTransferAmount (#1265) is too similar to BullRockSwap._getTValues(uint256).tTransferAmount (#975)
Variable BullRockSwap._transferStandard(address,address,uint256).rTransferAmount (#1227) is too similar to BullRockSwap._transferStandard(address,address,uint256).tTransferAmount (#1227)
Variable BullRockSwap._getRValues(uint256,uint256,uint256).rTransferAmount (#997) is too similar to BullRockSwap._getValues(uint256).tTransferAmount (#967)
Variable BullRockSwap._transferStandard(address,address,uint256).rTransferAmount (#1227) is too similar to BullRockSwap._getTValues(uint256).tTransferAmount (#975)
Variable BullRockSwap._transferFromExcluded(address,address,uint256).rTransferAmount (#1252) is too similar to BullRockSwap._getTValues(uint256).tTransferAmount (#975)
Variable BullRockSwap._transferBothExcluded(address,address,uint256).rTransferAmount (#1265) is too similar to BullRockSwap._transferToExcluded(address,address,uint256).tTransferAmount (#1239)
Variable BullRockSwap._getRValues(uint256,uint256,uint256).rTransferAmount (#997) is too similar to BullRockSwap._transferStandard(address,address,uint256).tTransferAmount (#1227)
Variable BullRockSwap._transferStandard(address,address,uint256).rTransferAmount (#1227) is too similar to BullRockSwap._transferFromExcluded(address,address,uint256).tTransferAmount (#1252)
Variable BullRockSwap._transferFromExcluded(address,address,uint256).rTransferAmount (#1252) is too similar to BullRockSwap._transferFromExcluded(address,address,uint256).tTransferAmount (#1252)
Variable BullRockSwap._getRValues(uint256,uint256,uint256).rTransferAmount (#997) is too similar to BullRockSwap._getTValues(uint256).tTransferAmount (#975)
Variable BullRockSwap._getValues(uint256).rTransferAmount (#968) is too similar to BullRockSwap._getTValues(uint256).tTransferAmount (#975)
Variable BullRockSwap.reflectionFromToken(uint256,bool).rTransferAmount (#847) is too similar to BullRockSwap._getTValues(uint256).tTransferAmount (#975)
Variable BullRockSwap._transferBothExcluded(address,address,uint256).rTransferAmount (#1265) is too similar to BullRockSwap._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable BullRockSwap._creatorFee (#702) is too similar to BullRockSwap._getTCreatorFeeValues(uint256).tCreatorFee (#985)
Variable BullRockSwap._creatorFee (#702) is too similar to BullRockSwap._takeCreatorFee(uint256).tCreatorFee (#953)
Variable BullRockSwap._creatorFee (#702) is too similar to BullRockSwap._getRCreatorFeeValues(uint256,uint256).tCreatorFee (#1011)
Variable BullRockSwap._takeCreatorFee(uint256).rCreatorFee (#954) is too similar to BullRockSwap._getRCreatorFeeValues(uint256,uint256).tCreatorFee (#1011)
Variable BullRockSwap._getRCreatorFeeValues(uint256,uint256).rCreatorFee (#1012) is too similar to BullRockSwap._takeCreatorFee(uint256).tCreatorFee (#953)
Variable BullRockSwap._getRCreatorFeeValues(uint256,uint256).rCreatorFee (#1012) is too similar to BullRockSwap._getRCreatorFeeValues(uint256,uint256).tCreatorFee (#1011)
Variable BullRockSwap._takeCreatorFee(uint256).rCreatorFee (#954) is too similar to BullRockSwap._takeCreatorFee(uint256).tCreatorFee (#953)
Variable BullRockSwap._getValues(uint256).rTransferAmount (#968) is too similar to BullRockSwap._transferStandard(address,address,uint256).tTransferAmount (#1227)
Variable BullRockSwap._getRValues(uint256,uint256,uint256).rTransferAmount (#997) is too similar to BullRockSwap._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable BullRockSwap.reflectionFromToken(uint256,bool).rTransferAmount (#847) is too similar to BullRockSwap._transferToExcluded(address,address,uint256).tTransferAmount (#1239)
Variable BullRockSwap._getValues(uint256).rTransferAmount (#968) is too similar to BullRockSwap._transferFromExcluded(address,address,uint256).tTransferAmount (#1252)
Variable BullRockSwap._getValues(uint256).rTransferAmount (#968) is too similar to BullRockSwap._getValues(uint256).tTransferAmount (#967)
Variable BullRockSwap._getRValues(uint256,uint256,uint256).rTransferAmount (#997) is too similar to BullRockSwap._transferFromExcluded(address,address,uint256).tTransferAmount (#1252)
Variable BullRockSwap._transferToExcluded(address,address,uint256).rTransferAmount (#1239) is too similar to BullRockSwap._transferToExcluded(address,address,uint256).tTransferAmount (#1239)
Variable BullRockSwap._transferBothExcluded(address,address,uint256).rTransferAmount (#1265) is too similar to BullRockSwap._transferFromExcluded(address,address,uint256).tTransferAmount (#1252)
Variable BullRockSwap._getValues(uint256).rTransferAmount (#968) is too similar to BullRockSwap._transferToExcluded(address,address,uint256).tTransferAmount (#1239)
Variable BullRockSwap._transferToExcluded(address,address,uint256).rTransferAmount (#1239) is too similar to BullRockSwap._getTValues(uint256).tTransferAmount (#975)
Variable BullRockSwap.reflectionFromToken(uint256,bool).rTransferAmount (#847) is too similar to BullRockSwap._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable BullRockSwap._transferToExcluded(address,address,uint256).rTransferAmount (#1239) is too similar to BullRockSwap._getValues(uint256).tTransferAmount (#967)
Variable BullRockSwap.reflectionFromToken(uint256,bool).rTransferAmount (#847) is too similar to BullRockSwap._transferFromExcluded(address,address,uint256).tTransferAmount (#1252)
Variable BullRockSwap.reflectionFromToken(uint256,bool).rTransferAmount (#847) is too similar to BullRockSwap._transferStandard(address,address,uint256).tTransferAmount (#1227)
Variable BullRockSwap.reflectionFromToken(uint256,bool).rTransferAmount (#847) is too similar to BullRockSwap._getValues(uint256).tTransferAmount (#967)
Variable BullRockSwap._transferToExcluded(address,address,uint256).rTransferAmount (#1239) is too similar to BullRockSwap._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable BullRockSwap._transferToExcluded(address,address,uint256).rTransferAmount (#1239) is too similar to BullRockSwap._transferFromExcluded(address,address,uint256).tTransferAmount (#1252)
Variable BullRockSwap._getValues(uint256).rTransferAmount (#968) is too similar to BullRockSwap._transferBothExcluded(address,address,uint256).tTransferAmount (#1265)
Variable BullRockSwap._transferToExcluded(address,address,uint256).rTransferAmount (#1239) is too similar to BullRockSwap._transferStandard(address,address,uint256).tTransferAmount (#1227)
Prevent variables from having similar names.

Additional information: link

BullRockSwap.slitherConstructorVariables() (#671-1282) uses literals with too many digits:
- _tTotal = 1000000000000 * 1e7 (#685)
BullRockSwap.slitherConstructorVariables() (#671-1282) uses literals with too many digits:
- burnAddress = 0x000000000000000000000000000000000000dEaD (#712)
BullRockSwap.slitherConstructorVariables() (#671-1282) uses literals with too many digits:
- _maxTxAmount = 100000000 * 1e7 (#718)
BullRockSwap.slitherConstructorVariables() (#671-1282) uses literals with too many digits:
- _numTokensSellToAddToLiquidity = 100000 * 1e7 (#719)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

BullRockSwap._decimals (#691) should be constant
BullRockSwap._name (#689) should be constant
BullRockSwap._symbol (#690) should be constant
BullRockSwap.burnAddress (#712) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#432-435)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#441-445)
geUnlockTime() should be declared external:
- Ownable.geUnlockTime() (#447-449)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#452-457)
unlock() should be declared external:
- Ownable.unlock() (#460-465)
name() should be declared external:
- BullRockSwap.name() (#756-758)
symbol() should be declared external:
- BullRockSwap.symbol() (#760-762)
decimals() should be declared external:
- BullRockSwap.decimals() (#764-766)
totalSupply() should be declared external:
- BullRockSwap.totalSupply() (#768-770)
transfer(address,uint256) should be declared external:
- BullRockSwap.transfer(address,uint256) (#777-780)
allowance(address,address) should be declared external:
- BullRockSwap.allowance(address,address) (#782-784)
approve(address,uint256) should be declared external:
- BullRockSwap.approve(address,uint256) (#786-789)
transferFrom(address,address,uint256) should be declared external:
- BullRockSwap.transferFrom(address,address,uint256) (#791-795)
increaseAllowance(address,uint256) should be declared external:
- BullRockSwap.increaseAllowance(address,uint256) (#797-800)
decreaseAllowance(address,uint256) should be declared external:
- BullRockSwap.decreaseAllowance(address,uint256) (#802-805)
isExcludedFromReward(address) should be declared external:
- BullRockSwap.isExcludedFromReward(address) (#807-809)
totalFees() should be declared external:
- BullRockSwap.totalFees() (#811-813)
deliver(uint256) should be declared external:
- BullRockSwap.deliver(uint256) (#820-827)
reflectionFromToken(uint256,bool) should be declared external:
- BullRockSwap.reflectionFromToken(uint256,bool) (#841-850)
excludeFromReward(address) should be declared external:
- BullRockSwap.excludeFromReward(address) (#858-865)
excludeFromFee(address) should be declared external:
- BullRockSwap.excludeFromFee(address) (#880-882)
includeInFee(address) should be declared external:
- BullRockSwap.includeInFee(address) (#884-886)
isExcludedFromFee(address) should be declared external:
- BullRockSwap.isExcludedFromFee(address) (#1062-1064)
Use the external attribute for functions never called from the contract.

Additional information: link