EVCoin [EverestCoin] Token

EverestCoin.sendBNBout(uint256) (#1241-1254) sends eth to arbitrary user
Dangerous calls:
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
EverestCoin.addLiquidity(uint256,uint256) (#1274-1287) sends eth to arbitrary user
Dangerous calls:
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
Ensure that an arbitrary user cannot withdraw unauthorized funds.

Additional information: link

Reentrancy in EverestCoin._transfer(address,address,uint256) (#1166-1213):
External calls:
- swapAndLiquify(contractTokenBalance) (#1208)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- dexRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1265-1271)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- lottery.addUserToLottery(to) (#1356)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1208)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
State variables written after the call(s):
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _liquidityHolders[from] = true (#1298)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _rOwned[burnAddress] = _rOwned[burnAddress] + rBurn (#1445)
- _rOwned[address(this)] = _rOwned[address(this)] + rLiquidity (#1436)
- _rOwned[from] = _rOwned[from] - values.rAmount (#1334)
- _rOwned[to] = _rOwned[to] + values.rTransferAmount (#1335)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _rTotal = _rTotal - rFee (#1429)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _tOwned[burnAddress] = _tOwned[burnAddress] + tBurn (#1447)
- _tOwned[address(this)] = _tOwned[address(this)] + tLiquidity (#1438)
- _tOwned[from] = _tOwned[from] - tAmount (#1338)
- _tOwned[to] = _tOwned[to] + values.tTransferAmount (#1340)
- _tOwned[from] = _tOwned[from] - tAmount (#1342)
- _tOwned[to] = _tOwned[to] + values.tTransferAmount (#1343)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- swapAndLiquifyEnabled = true (#1302)
Reentrancy in EverestCoin.transferOwner(address) (#810-828):
External calls:
- _transfer(_owner,newOwner,balanceOf(_owner)) (#822)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- dexRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1265-1271)
- lottery.addUserToLottery(to) (#1356)
External calls sending eth:
- _transfer(_owner,newOwner,balanceOf(_owner)) (#822)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
State variables written after the call(s):
- _owner = newOwner (#825)
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.

EverestCoin.swapAndLiquify(uint256) (#1215-1239) performs a multiplication on the result of a division:
-half = toLiquify / 2 (#1222)
-liquidityBalance = (fromSwap * half) / toSwapForEth (#1231)
Consider ordering multiplication before division.

Additional information: link

EverestCoin.setLpPair(address,bool) (#889-899) compares to a boolean constant:
-enabled == false (#890)
EverestCoin.setExcludedFromReward(address,bool) (#1054-1074) compares to a boolean constant:
-enabled == true (#1055)
EverestCoin.setExcludedFromReward(address,bool) (#1054-1074) compares to a boolean constant:
-enabled == false (#1062)
Remove the equality to the boolean constant.

Additional information: link

EverestCoin.setExcludedFromReward(address,bool) (#1054-1074) has costly operations inside a loop:
- _excluded.pop() (#1069)
Use a local variable to hold the loop computation result.

Additional information: link

Reentrancy in EverestCoin._finalizeTransfer(address,address,uint256,bool) (#1307-1376):
External calls:
- lottery.addUserToLottery(to) (#1356)
State variables written after the call(s):
- _takeLiquidity(from,values.tLiquidity) (#1368)
- _rOwned[address(this)] = _rOwned[address(this)] + rLiquidity (#1436)
- _takeBurn(from,values.tBurn) (#1372)
- _rOwned[burnAddress] = _rOwned[burnAddress] + rBurn (#1445)
- _takeReflect(values.rFee,values.tFee) (#1370)
- _rTotal = _rTotal - rFee (#1429)
- _takeLiquidity(from,values.tLiquidity) (#1368)
- _tOwned[address(this)] = _tOwned[address(this)] + tLiquidity (#1438)
- _takeBurn(from,values.tBurn) (#1372)
- _tOwned[burnAddress] = _tOwned[burnAddress] + tBurn (#1447)
- nextBuyLottery = false (#1357)
Reentrancy in EverestCoin.finishAndCloseLottery() (#1469-1473):
External calls:
- lottery.finishAndCloseLottery(address(_owner)) (#1471)
State variables written after the call(s):
- lotteryRunning = false (#1472)
Apply the check-effects-interactions pattern.

Additional information: link

EverestCoin._getValues(uint256,bool).values (#1383) is a local variable never initialized
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

EverestCoin.addLiquidity(uint256,uint256) (#1274-1287) ignores return value by dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
Ensure that all the return values of the function calls are used.

Additional information: link

EverestCoin.isProtected(uint256,uint256) (#917-921) should emit an event for:
- _liqAddStatus = rInitializer (#919)
- _initialLiquidityAmount = tInitalizer (#920)
EverestCoin.setStartingProtections(uint8,uint256) (#934-938) should emit an event for:
- snipeBlockAmt = _block (#936)
- gasPriceLimit = _gas * 1000000000 (#937)
EverestCoin.setBuyTaxes(uint256,uint256,uint256,uint256,uint256,uint256,uint256,uint256) (#940-968) should emit an event for:
- _buyLiquidityFee = liquidityFee (#960)
- _buyReflectFee = reflectFee (#961)
- _buyMarketingFee = marketingFee (#962)
- _buyDevFee = devFee (#963)
- _buyModFee = modFee (#964)
- _buyGameAdvFee = gameAdvFee (#965)
- _buyCharityFee = charityFee (#966)
- _buyBurnFee = burnFee (#967)
EverestCoin.setSellTaxes(uint256,uint256,uint256,uint256,uint256,uint256,uint256,uint256) (#970-998) should emit an event for:
- _sellLiquidityFee = liquidityFee (#990)
- _sellReflectFee = reflectFee (#991)
- _sellMarketingFee = marketingFee (#992)
- _sellDevFee = devFee (#993)
- _sellModFee = modFee (#994)
- _sellGameAdvFee = gameAdvFee (#995)
- _sellCharityFee = charityFee (#996)
- _sellBurnFee = burnFee (#997)
EverestCoin.setTransferTaxes(uint256,uint256,uint256,uint256,uint256,uint256,uint256,uint256) (#1000-1028) should emit an event for:
- _transferLiquidityFee = liquidityFee (#1020)
- _transferReflectFee = reflectFee (#1021)
- _transferMarketingFee = marketingFee (#1022)
- _transferDevFee = devFee (#1023)
- _transferModFee = modFee (#1024)
- _transferGameAdvFee = gameAdvFee (#1025)
- _transferCharityFee = charityFee (#1026)
- _transferBurnFee = burnFee (#1027)
EverestCoin.setMaxTxPercent(uint256,uint256) (#1030-1035) should emit an event for:
- _maxTxAmount = check (#1033)
EverestCoin.setMaxWalletSize(uint256,uint256) (#1037-1042) should emit an event for:
- _maxWalletSize = check (#1040)
EverestCoin.setMinimumHoldForLottery(uint256,uint256) (#1475-1479) should emit an event for:
- minimumHoldForLottery = (_tTotal * minHoldPercent) / divisor (#1477)
- minHoldForLotteryUI = (startingSupply * minHoldPercent) / divisor (#1478)
EverestCoin.setMinBNBBuyNeeded(uint256,uint256) (#1497-1500) should emit an event for:
- minBNBBuy = amount * 10 ** divisor (#1499)
Emit an event for critical parameter changes.

Additional information: link

LotteryArray.constructor(uint256,address)._token (#543) lacks a zero-check on :
- token = _token (#545)
Check that the address is not zero.

Additional information: link

Reentrancy in EverestCoin._finalizeTransfer(address,address,uint256,bool) (#1307-1376):
External calls:
- lottery.addUserToLottery(to) (#1356)
State variables written after the call(s):
- _takeReflect(values.rFee,values.tFee) (#1370)
- _tFeeTotal = _tFeeTotal + tFee (#1430)
Reentrancy in EverestCoin._transfer(address,address,uint256) (#1166-1213):
External calls:
- swapAndLiquify(contractTokenBalance) (#1208)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- dexRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1265-1271)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- lottery.addUserToLottery(to) (#1356)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1208)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
State variables written after the call(s):
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _liqAddStamp = block.timestamp (#1300)
Reentrancy in EverestCoin.constructor() (#756-797):
External calls:
- lpPair = IUniswapV2Factory(dexRouter.factory()).createPair(dexRouter.WETH(),address(this)) (#764)
State variables written after the call(s):
- _allowances[address(this)][address(dexRouter)] = type()(uint256).max (#767)
- _approve(_msgSender(),_routerAddress,_tTotal) (#786)
- _allowances[sender][spender] = amount (#1116)
- _excluded.push(address(this)) (#773)
- _excluded.push(owner()) (#775)
- _excluded.push(burnAddress) (#777)
- _excluded.push(lpPair) (#779)
- _excluded.push(0x2D045410f002A95EFcEE67759A92518fA3FcE677) (#783)
- _isExcluded[address(this)] = true (#772)
- _isExcluded[owner()] = true (#774)
- _isExcluded[burnAddress] = true (#776)
- _isExcluded[lpPair] = true (#778)
- _isExcluded[0x2D045410f002A95EFcEE67759A92518fA3FcE677] = true (#782)
- _isExcludedFromFee[owner()] = true (#769)
- _isExcludedFromFee[address(this)] = true (#770)
- _isExcludedFromFee[0x2D045410f002A95EFcEE67759A92518fA3FcE677] = true (#781)
- _isSniper[0xE4882975f933A199C92b5A925C9A8fE65d599Aa8] = true (#789)
- _isSniper[0x86C70C4a3BC775FB4030448c9fdb73Dc09dd8444] = true (#790)
- _isSniper[0xa4A25AdcFCA938aa030191C297321323C57148Bd] = true (#791)
- _isSniper[0x20C00AFf15Bb04cC631DB07ee9ce361ae91D12f8] = true (#792)
- _isSniper[0x0538856b6d0383cde1709c6531B9a0437185462b] = true (#793)
- _isSniper[0x6e44DdAb5c29c9557F275C9DB6D12d670125FE17] = true (#794)
- _liquidityHolders[owner()] = true (#771)
- lpPairObj = IUniswapV2Pair(lpPair) (#765)
- lpPairs[lpPair] = true (#766)
Reentrancy in EverestCoin.setNewRouter(address) (#877-887):
External calls:
- lpPair = IUniswapV2Factory(_newRouter.factory()).createPair(address(this),_newRouter.WETH()) (#881)
State variables written after the call(s):
- dexRouter = _newRouter (#886)
Reentrancy in EverestCoin.swapAndLiquify(uint256) (#1215-1239):
External calls:
- swapTokensForEth(toSwapForEth) (#1228)
- dexRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1265-1271)
- addLiquidity(otherHalf,liquidityBalance) (#1234)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
External calls sending eth:
- addLiquidity(otherHalf,liquidityBalance) (#1234)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
State variables written after the call(s):
- addLiquidity(otherHalf,liquidityBalance) (#1234)
- _allowances[sender][spender] = amount (#1116)
Reentrancy in EverestCoin.transferFrom(address,address,uint256) (#861-865):
External calls:
- _transfer(sender,recipient,amount) (#862)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- dexRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1265-1271)
- lottery.addUserToLottery(to) (#1356)
External calls sending eth:
- _transfer(sender,recipient,amount) (#862)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#863)
- _allowances[sender][spender] = amount (#1116)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in EverestCoin._finalizeTransfer(address,address,uint256,bool) (#1307-1376):
External calls:
- lottery.addUserToLottery(to) (#1356)
Event emitted after the call(s):
- Transfer(sender,burnAddress,tBurn) (#1448)
- _takeBurn(from,values.tBurn) (#1372)
- Transfer(sender,address(this),tLiquidity) (#1439)
- _takeLiquidity(from,values.tLiquidity) (#1368)
- Transfer(from,to,values.tTransferAmount) (#1374)
Reentrancy in EverestCoin._transfer(address,address,uint256) (#1166-1213):
External calls:
- swapAndLiquify(contractTokenBalance) (#1208)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- dexRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1265-1271)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- lottery.addUserToLottery(to) (#1356)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1208)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
Event emitted after the call(s):
- SniperCaught(to) (#1326)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- SwapAndLiquifyEnabledUpdated(true) (#1303)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- Transfer(sender,address(this),tLiquidity) (#1439)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- Transfer(sender,burnAddress,tBurn) (#1448)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- Transfer(from,to,values.tTransferAmount) (#1374)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
Reentrancy in EverestCoin.constructor() (#756-797):
External calls:
- lpPair = IUniswapV2Factory(dexRouter.factory()).createPair(dexRouter.WETH(),address(this)) (#764)
Event emitted after the call(s):
- Approval(sender,spender,amount) (#1117)
- _approve(_msgSender(),_routerAddress,_tTotal) (#786)
- Transfer(address(0),_msgSender(),_tTotal) (#796)
Reentrancy in EverestCoin.swapAndLiquify(uint256) (#1215-1239):
External calls:
- swapTokensForEth(toSwapForEth) (#1228)
- dexRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1265-1271)
- addLiquidity(otherHalf,liquidityBalance) (#1234)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
External calls sending eth:
- addLiquidity(otherHalf,liquidityBalance) (#1234)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
Event emitted after the call(s):
- Approval(sender,spender,amount) (#1117)
- addLiquidity(otherHalf,liquidityBalance) (#1234)
- SwapAndLiquify(half,liquidityBalance,otherHalf) (#1235)
Reentrancy in EverestCoin.transferFrom(address,address,uint256) (#861-865):
External calls:
- _transfer(sender,recipient,amount) (#862)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- dexRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1265-1271)
- lottery.addUserToLottery(to) (#1356)
External calls sending eth:
- _transfer(sender,recipient,amount) (#862)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
Event emitted after the call(s):
- Approval(sender,spender,amount) (#1117)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#863)
Reentrancy in EverestCoin.transferOwner(address) (#810-828):
External calls:
- _transfer(_owner,newOwner,balanceOf(_owner)) (#822)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- dexRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1265-1271)
- lottery.addUserToLottery(to) (#1356)
External calls sending eth:
- _transfer(_owner,newOwner,balanceOf(_owner)) (#822)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
Event emitted after the call(s):
- OwnershipTransferred(_owner,newOwner) (#826)
Apply the check-effects-interactions pattern.

Additional information: link

LotteryArray.addUserToLottery(address) (#556-561) uses timestamp for comparisons
Dangerous comparisons:
- block.timestamp < lotteryEndStamp (#557)
LotteryArray.finishAndCloseLottery(address) (#563-566) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp >= lotteryEndStamp,Lottery must be over.) (#564)
LotteryArray.getRemainingLotteryTime() (#572-574) uses timestamp for comparisons
Dangerous comparisons:
- (lotteryEndStamp > block.timestamp) (#573)
EverestCoin.setLpPair(address,bool) (#889-899) uses timestamp for comparisons
Dangerous comparisons:
- timeSinceLastPair != 0 (#893)
- require(bool,string)(block.timestamp - timeSinceLastPair > 604800,Cannot set a new pair this week!) (#894)
EverestCoin.startNewLottery(uint256) (#1451-1456) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(endTime > block.timestamp,Cannot end in the past.) (#1453)
Avoid relying on block.timestamp.

Additional information: link

Address.isContract(address) (#281-290) uses assembly
- INLINE ASM (#288)
Address._functionCallWithValue(address,bytes,uint256,string) (#318-339) uses assembly
- INLINE ASM (#331-334)
Do not use evm assembly.

Additional information: link

Address._functionCallWithValue(address,bytes,uint256,string) (#318-339) is never used and should be removed
Address.functionCall(address,bytes) (#300-302) is never used and should be removed
Address.functionCall(address,bytes,string) (#304-306) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#309-311) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#313-316) is never used and should be removed
Address.isContract(address) (#281-290) is never used and should be removed
Address.sendValue(address,uint256) (#292-298) is never used and should be removed
Context._msgData() (#26-29) is never used and should be removed
SafeMath.mod(uint256,uint256) (#258-260) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#274-277) is never used and should be removed
SafeMath.mul(uint256,uint256) (#196-208) is never used and should be removed
SafeMath.sub(uint256,uint256) (#165-167) is never used and should be removed
Remove unused functions.

Additional information: link

EverestCoin._decimalsMul (#603) is set pre-construction with a non-constant function or state variable:
- _decimals
EverestCoin._tTotal (#604) is set pre-construction with a non-constant function or state variable:
- startingSupply * 10 ** _decimalsMul
EverestCoin._rTotal (#605) is set pre-construction with a non-constant function or state variable:
- (MAX - (MAX % _tTotal))
EverestCoin._sellReflectFee (#629) is set pre-construction with a non-constant function or state variable:
- _buyReflectFee
EverestCoin._sellLiquidityFee (#630) is set pre-construction with a non-constant function or state variable:
- _buyLiquidityFee
EverestCoin._sellMarketingFee (#631) is set pre-construction with a non-constant function or state variable:
- _buyMarketingFee
EverestCoin._sellDevFee (#632) is set pre-construction with a non-constant function or state variable:
- _buyDevFee
EverestCoin._sellModFee (#633) is set pre-construction with a non-constant function or state variable:
- _buyModFee
EverestCoin._sellGameAdvFee (#634) is set pre-construction with a non-constant function or state variable:
- _buyGameAdvFee
EverestCoin._sellCharityFee (#635) is set pre-construction with a non-constant function or state variable:
- _buyCharityFee
EverestCoin._sellBurnFee (#636) is set pre-construction with a non-constant function or state variable:
- _buyBurnFee
EverestCoin._transferReflectFee (#638) is set pre-construction with a non-constant function or state variable:
- _buyReflectFee
EverestCoin._transferLiquidityFee (#639) is set pre-construction with a non-constant function or state variable:
- _buyLiquidityFee
EverestCoin._transferMarketingFee (#640) is set pre-construction with a non-constant function or state variable:
- _buyMarketingFee
EverestCoin._transferDevFee (#641) is set pre-construction with a non-constant function or state variable:
- _buyDevFee
EverestCoin._transferModFee (#642) is set pre-construction with a non-constant function or state variable:
- _buyModFee
EverestCoin._transferGameAdvFee (#643) is set pre-construction with a non-constant function or state variable:
- _buyGameAdvFee
EverestCoin._transferCharityFee (#644) is set pre-construction with a non-constant function or state variable:
- _buyCharityFee
EverestCoin._transferBurnFee (#645) is set pre-construction with a non-constant function or state variable:
- _buyBurnFee
EverestCoin._previousReflectFee (#658) is set pre-construction with a non-constant function or state variable:
- _reflectFee
EverestCoin._previousLiquidityFee (#659) is set pre-construction with a non-constant function or state variable:
- _liquidityFee
EverestCoin._previousMarketingFee (#660) is set pre-construction with a non-constant function or state variable:
- _marketingFee
EverestCoin._previousDevFee (#661) is set pre-construction with a non-constant function or state variable:
- _devFee
EverestCoin._previousModFee (#662) is set pre-construction with a non-constant function or state variable:
- _modFee
EverestCoin._previousGameAdvFee (#663) is set pre-construction with a non-constant function or state variable:
- _gameAdvFee
EverestCoin._previousCharityFee (#664) is set pre-construction with a non-constant function or state variable:
- _charityFee
EverestCoin._previousBurnFee (#665) is set pre-construction with a non-constant function or state variable:
- _burnFee
EverestCoin._maxTxAmount (#688) is set pre-construction with a non-constant function or state variable:
- (_tTotal * maxTxPercent) / maxTxDivisor
EverestCoin._previousMaxTxAmount (#689) is set pre-construction with a non-constant function or state variable:
- _maxTxAmount
EverestCoin.maxTxAmountUI (#690) is set pre-construction with a non-constant function or state variable:
- (startingSupply * maxTxPercent) / maxTxDivisor
EverestCoin._maxWalletSize (#694) is set pre-construction with a non-constant function or state variable:
- (_tTotal * maxWalletPercent) / maxWalletDivisor
EverestCoin._previousMaxWalletSize (#695) is set pre-construction with a non-constant function or state variable:
- _maxWalletSize
EverestCoin.maxWalletSizeUI (#696) is set pre-construction with a non-constant function or state variable:
- (startingSupply * maxWalletPercent) / maxWalletDivisor
EverestCoin.numTokensSellToAddToLiquidity (#698) is set pre-construction with a non-constant function or state variable:
- (_tTotal * 5) / 10000
EverestCoin.minimumHoldForLottery (#716) is set pre-construction with a non-constant function or state variable:
- minHoldForLotteryUI * 10 ** 9
Remove any initialization of state variables via non-constant state variables or function calls. If variables must be set upon contract deployment, locate initialization in the constructor instead.

Additional information: link

Pragma version>=0.6.0<0.9.0 (#2) is too complex
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) (#292-298):
- (success) = recipient.call{value: amount}() (#296)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#318-339):
- (success,returndata) = target.call{value: weiValue}(data) (#322)
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() (#367) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#368) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#382) is not in mixedCase
Function IUniswapV2Router01.WETH() (#400) is not in mixedCase
Parameter EverestCoin.setStartingProtections(uint8,uint256)._block (#934) is not in mixedCase
Parameter EverestCoin.setStartingProtections(uint8,uint256)._gas (#934) is not in mixedCase
Parameter EverestCoin.setSwapAndLiquifyEnabled(bool)._enabled (#1049) is not in mixedCase
Variable EverestCoin._reflectFee (#611) is not in mixedCase
Variable EverestCoin._liquidityFee (#612) is not in mixedCase
Variable EverestCoin._marketingFee (#613) is not in mixedCase
Variable EverestCoin._devFee (#614) is not in mixedCase
Variable EverestCoin._modFee (#615) is not in mixedCase
Variable EverestCoin._gameAdvFee (#616) is not in mixedCase
Variable EverestCoin._charityFee (#617) is not in mixedCase
Variable EverestCoin._burnFee (#618) is not in mixedCase
Variable EverestCoin._buyReflectFee (#620) is not in mixedCase
Variable EverestCoin._buyLiquidityFee (#621) is not in mixedCase
Variable EverestCoin._buyMarketingFee (#622) is not in mixedCase
Variable EverestCoin._buyDevFee (#623) is not in mixedCase
Variable EverestCoin._buyModFee (#624) is not in mixedCase
Variable EverestCoin._buyGameAdvFee (#625) is not in mixedCase
Variable EverestCoin._buyCharityFee (#626) is not in mixedCase
Variable EverestCoin._buyBurnFee (#627) is not in mixedCase
Variable EverestCoin._sellReflectFee (#629) is not in mixedCase
Variable EverestCoin._sellLiquidityFee (#630) is not in mixedCase
Variable EverestCoin._sellMarketingFee (#631) is not in mixedCase
Variable EverestCoin._sellDevFee (#632) is not in mixedCase
Variable EverestCoin._sellModFee (#633) is not in mixedCase
Variable EverestCoin._sellGameAdvFee (#634) is not in mixedCase
Variable EverestCoin._sellCharityFee (#635) is not in mixedCase
Variable EverestCoin._sellBurnFee (#636) is not in mixedCase
Variable EverestCoin._transferReflectFee (#638) is not in mixedCase
Variable EverestCoin._transferLiquidityFee (#639) is not in mixedCase
Variable EverestCoin._transferMarketingFee (#640) is not in mixedCase
Variable EverestCoin._transferDevFee (#641) is not in mixedCase
Variable EverestCoin._transferModFee (#642) is not in mixedCase
Variable EverestCoin._transferGameAdvFee (#643) is not in mixedCase
Variable EverestCoin._transferCharityFee (#644) is not in mixedCase
Variable EverestCoin._transferBurnFee (#645) is not in mixedCase
Variable EverestCoin._hasLiqBeenAdded (#701) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#27)" inContext (#21-30)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Reentrancy in EverestCoin._transfer(address,address,uint256) (#1166-1213):
External calls:
- swapAndLiquify(contractTokenBalance) (#1208)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1208)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
State variables written after the call(s):
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _hasLiqBeenAdded = true (#1299)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _isSniper[to] = true (#1324)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _liqAddBlock = block.number + 500 (#1293)
- _liqAddBlock = block.number (#1295)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _liqAddStamp = block.timestamp (#1300)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _liquidityHolders[from] = true (#1298)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _rOwned[burnAddress] = _rOwned[burnAddress] + rBurn (#1445)
- _rOwned[address(this)] = _rOwned[address(this)] + rLiquidity (#1436)
- _rOwned[from] = _rOwned[from] - values.rAmount (#1334)
- _rOwned[to] = _rOwned[to] + values.rTransferAmount (#1335)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _rTotal = _rTotal - rFee (#1429)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _tFeeTotal = _tFeeTotal + tFee (#1430)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- _tOwned[burnAddress] = _tOwned[burnAddress] + tBurn (#1447)
- _tOwned[address(this)] = _tOwned[address(this)] + tLiquidity (#1438)
- _tOwned[from] = _tOwned[from] - tAmount (#1338)
- _tOwned[to] = _tOwned[to] + values.tTransferAmount (#1340)
- _tOwned[from] = _tOwned[from] - tAmount (#1342)
- _tOwned[to] = _tOwned[to] + values.tTransferAmount (#1343)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- lastSell = from (#1362)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- nextBuyLottery = false (#1357)
- nextBuyLottery = true (#1361)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- snipersCaught ++ (#1325)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- swapAndLiquifyEnabled = true (#1302)
Event emitted after the call(s):
- SniperCaught(to) (#1326)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- SwapAndLiquifyEnabledUpdated(true) (#1303)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- Transfer(sender,address(this),tLiquidity) (#1439)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- Transfer(sender,burnAddress,tBurn) (#1448)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
- Transfer(from,to,values.tTransferAmount) (#1374)
- _finalizeTransfer(from,to,amount,takeFee) (#1212)
Reentrancy in EverestCoin.transferFrom(address,address,uint256) (#861-865):
External calls:
- _transfer(sender,recipient,amount) (#862)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
External calls sending eth:
- _transfer(sender,recipient,amount) (#862)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#863)
- _allowances[sender][spender] = amount (#1116)
Event emitted after the call(s):
- Approval(sender,spender,amount) (#1117)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#863)
Reentrancy in EverestCoin.transferOwner(address) (#810-828):
External calls:
- _transfer(_owner,newOwner,balanceOf(_owner)) (#822)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
External calls sending eth:
- _transfer(_owner,newOwner,balanceOf(_owner)) (#822)
- dexRouter.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,burnAddress,block.timestamp) (#1279-1286)
- _marketingWallet.transfer(amountMarketingBNB) (#1249)
- _devWallet.transfer(amountDevBNB) (#1250)
- _modWallet.transfer(amountModBNB) (#1251)
- _gameAdvWallet.transfer(amountGameAdvBNB) (#1252)
- _charityWallet.transfer(amountCharityBNB) (#1253)
State variables written after the call(s):
- _owner = newOwner (#825)
Event emitted after the call(s):
- OwnershipTransferred(_owner,newOwner) (#826)
Apply the check-effects-interactions pattern.

Additional information: link

Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#404) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#405)
Prevent variables from having similar names.

Additional information: link

EverestCoin.slitherConstructorVariables() (#578-1502) uses literals with too many digits:
- burnAddress = 0x000000000000000000000000000000000000dEaD (#674)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

EverestCoin._previousReflectFee (#658) is never used in EverestCoin (#578-1502)
EverestCoin._previousLiquidityFee (#659) is never used in EverestCoin (#578-1502)
EverestCoin._previousMarketingFee (#660) is never used in EverestCoin (#578-1502)
EverestCoin._previousDevFee (#661) is never used in EverestCoin (#578-1502)
EverestCoin._previousModFee (#662) is never used in EverestCoin (#578-1502)
EverestCoin._previousGameAdvFee (#663) is never used in EverestCoin (#578-1502)
EverestCoin._previousCharityFee (#664) is never used in EverestCoin (#578-1502)
EverestCoin._previousBurnFee (#665) is never used in EverestCoin (#578-1502)
EverestCoin._previousMaxTxAmount (#689) is never used in EverestCoin (#578-1502)
EverestCoin._previousMaxWalletSize (#695) is never used in EverestCoin (#578-1502)
Remove unused state variables.

Additional information: link

EverestCoin._charityWallet (#679) should be constant
EverestCoin._decimals (#602) should be constant
EverestCoin._devWallet (#677) should be constant
EverestCoin._gameAdvWallet (#680) should be constant
EverestCoin._modWallet (#678) should be constant
EverestCoin._name (#608) should be constant
EverestCoin._routerAddress (#672) should be constant
EverestCoin._symbol (#609) should be constant
EverestCoin.burnAddress (#674) should be constant
EverestCoin.masterTaxDivisor (#656) should be constant
EverestCoin.maxBurnFee (#654) should be constant
EverestCoin.maxCharityFee (#653) should be constant
EverestCoin.maxDevFee (#650) should be constant
EverestCoin.maxGameAdvFee (#652) should be constant
EverestCoin.maxLiquidityFee (#648) should be constant
EverestCoin.maxMarketingFee (#649) should be constant
EverestCoin.maxModFee (#651) should be constant
EverestCoin.maxReflectFee (#647) should be constant
EverestCoin.maxTxDivisor (#687) should be constant
EverestCoin.maxTxPercent (#686) should be constant
EverestCoin.maxWalletDivisor (#693) should be constant
EverestCoin.maxWalletPercent (#692) should be constant
EverestCoin.startingSupply (#599) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

getLotteryUserLength() should be declared external:
- LotteryArray.getLotteryUserLength() (#568-570)
getRemainingLotteryTime() should be declared external:
- LotteryArray.getRemainingLotteryTime() (#572-574)
renounceOwnership() should be declared external:
- EverestCoin.renounceOwnership() (#830-834)
transfer(address,uint256) should be declared external:
- EverestCoin.transfer(address,uint256) (#851-854)
approve(address,uint256) should be declared external:
- EverestCoin.approve(address,uint256) (#856-859)
transferFrom(address,address,uint256) should be declared external:
- EverestCoin.transferFrom(address,address,uint256) (#861-865)
increaseAllowance(address,uint256) should be declared external:
- EverestCoin.increaseAllowance(address,uint256) (#867-870)
decreaseAllowance(address,uint256) should be declared external:
- EverestCoin.decreaseAllowance(address,uint256) (#872-875)
setNewRouter(address) should be declared external:
- EverestCoin.setNewRouter(address) (#877-887)
isExcludedFromReward(address) should be declared external:
- EverestCoin.isExcludedFromReward(address) (#901-903)
isExcludedFromFee(address) should be declared external:
- EverestCoin.isExcludedFromFee(address) (#905-907)
totalFees() should be declared external:
- EverestCoin.totalFees() (#1092-1094)
isUserInLottery(address) should be declared external:
- EverestCoin.isUserInLottery(address) (#1458-1462)
getRemainingLotteryTime() should be declared external:
- EverestCoin.getRemainingLotteryTime() (#1481-1484)
getUserAtIndex(uint256) should be declared external:
- EverestCoin.getUserAtIndex(uint256) (#1486-1495)
Use the external attribute for functions never called from the contract.

Additional information: link

Contract has 11% buy tax and 10% sell tax.
Taxes are high (over 10%) but contract ownership is renounced.

Average 30d number of PancakeSwap swaps is low.

Token is deployed only at one blockchain

Unable to find token on CoinHunt

Additional information: link

Young tokens have high risks of scam / price dump / death

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

Young tokens have high risks of price dump / death

Token has relatively low CoinGecko rank

Token has relatively low CoinMarketCap rank