🐒🚀💎We are at the beginning of a revolution. I will raise an army of monkeys and we will take over the world.💎🚀🐒
Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
ElonsApeRevenge.addLiquidity(uint256,uint256) (#1684-1699) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in ElonsApeRevenge._transfer(address,address,uint256) (#1524-1609):
External calls:
- swapAndSendToFee(marketingTokens) (#1557)
- IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1617)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1675-1681)
- swapAndLiquify(swapTokens) (#1560)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1655-1661)
- swapAndSendDividends(sellTokens) (#1563)
- success = IERC20(BUSD).transfer(address(dividendTracker),dividends) (#1704)
- dividendTracker.distributeETHDividends(dividends) (#1707)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1675-1681)
External calls sending eth:
- swapAndLiquify(swapTokens) (#1560)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
State variables written after the call(s):
- super._transfer(from,address(this),fees) (#1591)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#927)
- _balances[recipient] = _balances[recipient].add(amount) (#928)
- super._transfer(from,to,amount) (#1594)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#927)
- _balances[recipient] = _balances[recipient].add(amount) (#928)
- swapping = false (#1565)
Apply the check-effects-interactions pattern.
Additional information: link
ElonsApeRevenge.swapAndSendToFee(uint256) (#1611-1618) ignores return value by IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1617)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.
Additional information: link
ElonsApeRevenge._transfer(address,address,uint256) (#1524-1609) uses a dangerous strict equality:
- launchedAt == 0 && from == owner() && automatedMarketMakerPairs[to] (#1533)
Don't use strict equality to determine if an account has enough Ether or tokens.
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.
Combination 2: Unchecked transfer + 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.
Combination 3: Reentrancy vulnerabilities + Unchecked transfer vulnerability. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.
Contract ticker ($EAR) contains non-alphanumeric characters.
Not a direct threat, but may indicate unreliable intentions of developer. Non-alphanumeric chars (,.;!#*&") are extremely rare among low risk tokens.
Not a direct threat, but may indicate unreliable intentions of developer. Widespread names (e.g. Elon, King, Moon, Doge) are common among meme-tokens and scams. The allow to gain free hype and attract unexperienced investors.
Reentrancy in DividendPayingToken._withdrawDividendOfUser(address) (#1145-1161):
External calls:
- success = IERC20(BUSD).transfer(user,_withdrawableDividend) (#1150)
State variables written after the call(s):
- withdrawnDividends[user] = withdrawnDividends[user].sub(_withdrawableDividend) (#1153)
Reentrancy in ElonsApeRevenge.constructor() (#1318-1357):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1327-1328)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1333)
- dividendTracker.excludeFromDividends(pair) (#1438)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (#1338)
- dividendTracker.excludeFromDividends(address(this)) (#1339)
- dividendTracker.excludeFromDividends(_newOwner) (#1340)
- dividendTracker.excludeFromDividends(deadWallet) (#1341)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (#1342)
State variables written after the call(s):
- excludeFromFees(_newOwner,true) (#1345)
- _isExcludedFromFees[account] = excluded (#1391)
- excludeFromFees(address(this),true) (#1346)
- _isExcludedFromFees[account] = excluded (#1391)
- transferOwnership(_newOwner) (#1354)
- _owner = newOwner (#330)
Reentrancy in ElonsApeRevenge.updateDividendTracker(address) (#1363-1378):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (#1370)
- newDividendTracker.excludeFromDividends(address(this)) (#1371)
- newDividendTracker.excludeFromDividends(owner()) (#1372)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (#1373)
State variables written after the call(s):
- dividendTracker = newDividendTracker (#1377)
Apply the check-effects-interactions pattern.
Additional information: link
ElonsApeRevenge._transfer(address,address,uint256).iterations (#1602) is a local variable never initialized
ElonsApeRevenge._transfer(address,address,uint256).claims (#1602) is a local variable never initialized
ElonsApeRevenge._transfer(address,address,uint256).lastProcessedIndex (#1602) 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
ElonsApeRevenge.claim() (#1511-1513) ignores return value by dividendTracker.processAccount(msg.sender,false) (#1512)
ElonsApeRevenge._transfer(address,address,uint256) (#1524-1609) ignores return value by dividendTracker.process(gas) (#1602-1607)
ElonsApeRevenge.addLiquidity(uint256,uint256) (#1684-1699) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
Ensure that all the return values of the function calls are used.
Additional information: link
DividendPayingToken.constructor(string,string)._name (#1119) shadows:
- ERC20._name (#749) (state variable)
DividendPayingToken.constructor(string,string)._symbol (#1119) shadows:
- ERC20._symbol (#750) (state variable)
DividendPayingToken.dividendOf(address)._owner (#1167) shadows:
- Ownable._owner (#283) (state variable)
DividendPayingToken.withdrawableDividendOf(address)._owner (#1174) shadows:
- Ownable._owner (#283) (state variable)
DividendPayingToken.withdrawnDividendOf(address)._owner (#1181) shadows:
- Ownable._owner (#283) (state variable)
DividendPayingToken.accumulativeDividendOf(address)._owner (#1191) shadows:
- Ownable._owner (#283) (state variable)
Rename the local variables that shadow another component.
Additional information: link
ElonsApeRevenge.setETHRewardsFee(uint256) (#1410-1412) should emit an event for:
- BUSDRewardsFee = value (#1411)
ElonsApeRevenge.setLiquiditFee(uint256) (#1414-1416) should emit an event for:
- liquidityFee = value (#1415)
ElonsApeRevenge.setMarketingFee(uint256) (#1418-1420) should emit an event for:
- marketingFee = value (#1419)
Emit an event for critical parameter changes.
Additional information: link
ElonsApeRevenge.updateUniswapV2Router(address)._uniswapV2Pair (#1384-1385) lacks a zero-check on :
- uniswapV2Pair = _uniswapV2Pair (#1386)
Check that the address is not zero.
Additional information: link
DividendPayingToken._withdrawDividendOfUser(address) (#1145-1161) has external calls inside a loop: success = IERC20(BUSD).transfer(user,_withdrawableDividend) (#1150)
Favor pull over push strategy for external calls.
Additional information: link
Variable 'ElonsApeRevenge._transfer(address,address,uint256).claims (#1602)' in ElonsApeRevenge._transfer(address,address,uint256) (#1524-1609) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1603)
Variable 'ElonsApeRevenge._transfer(address,address,uint256).lastProcessedIndex (#1602)' in ElonsApeRevenge._transfer(address,address,uint256) (#1524-1609) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1603)
Variable 'ElonsApeRevenge._transfer(address,address,uint256).iterations (#1602)' in ElonsApeRevenge._transfer(address,address,uint256) (#1524-1609) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1603)
Move all variable declarations prior to any usage of the variable, and ensure that reaching a variable declaration does not depend on some conditional if it is used unconditionally.
Additional information: link
Reentrancy in ElonsApeRevenge._transfer(address,address,uint256) (#1524-1609):
External calls:
- swapAndSendToFee(marketingTokens) (#1557)
- IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1617)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1675-1681)
- swapAndLiquify(swapTokens) (#1560)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1655-1661)
External calls sending eth:
- swapAndLiquify(swapTokens) (#1560)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
State variables written after the call(s):
- swapAndLiquify(swapTokens) (#1560)
- _allowances[owner][spender] = amount (#993)
Reentrancy in ElonsApeRevenge._transfer(address,address,uint256) (#1524-1609):
External calls:
- swapAndSendToFee(marketingTokens) (#1557)
- IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1617)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1675-1681)
- swapAndLiquify(swapTokens) (#1560)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1655-1661)
- swapAndSendDividends(sellTokens) (#1563)
- success = IERC20(BUSD).transfer(address(dividendTracker),dividends) (#1704)
- dividendTracker.distributeETHDividends(dividends) (#1707)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1675-1681)
External calls sending eth:
- swapAndLiquify(swapTokens) (#1560)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
State variables written after the call(s):
- swapAndSendDividends(sellTokens) (#1563)
- _allowances[owner][spender] = amount (#993)
Reentrancy in ElonsApeRevenge.constructor() (#1318-1357):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1327-1328)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (#1331)
- uniswapV2Router = _uniswapV2Router (#1330)
Reentrancy in ElonsApeRevenge.constructor() (#1318-1357):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1327-1328)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1333)
- dividendTracker.excludeFromDividends(pair) (#1438)
State variables written after the call(s):
- setMarketingWallet(0x1b30dC3FF02AEfaA22016E99c0658fF3ed69465a) (#1334)
- _isExcludedFromFees[account] = excluded (#1391)
- setMarketingWallet(0x1b30dC3FF02AEfaA22016E99c0658fF3ed69465a) (#1334)
- _marketingWalletAddress = wallet (#1406)
Reentrancy in ElonsApeRevenge.constructor() (#1318-1357):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1327-1328)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1333)
- dividendTracker.excludeFromDividends(pair) (#1438)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (#1338)
- dividendTracker.excludeFromDividends(address(this)) (#1339)
- dividendTracker.excludeFromDividends(_newOwner) (#1340)
- dividendTracker.excludeFromDividends(deadWallet) (#1341)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (#1342)
State variables written after the call(s):
- _mint(owner(),1000000000000 * (10 ** 18)) (#1356)
- _balances[account] = _balances[account].add(amount) (#947)
- _mint(owner(),1000000000000 * (10 ** 18)) (#1356)
- _totalSupply = _totalSupply.add(amount) (#946)
Reentrancy in ElonsApeRevengeDividendTracker.processAccount(address,bool) (#1909-1919):
External calls:
- amount = _withdrawDividendOfUser(account) (#1910)
- success = IERC20(BUSD).transfer(user,_withdrawableDividend) (#1150)
State variables written after the call(s):
- lastClaimTimes[account] = block.timestamp (#1913)
Reentrancy in ElonsApeRevenge.swapAndLiquify(uint256) (#1620-1641):
External calls:
- swapTokensForEth(half) (#1632)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1655-1661)
- addLiquidity(otherHalf,newBalance) (#1638)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1638)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (#1638)
- _allowances[owner][spender] = amount (#993)
Reentrancy in ElonsApeRevenge.updateUniswapV2Router(address) (#1380-1387):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(uniswapV2Router.factory()).createPair(address(this),uniswapV2Router.WETH()) (#1384-1385)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (#1386)
Apply the check-effects-interactions pattern.
Additional information: link
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#318-321)
get(IterableMapping.Map,address) should be declared external:
- IterableMapping.get(IterableMapping.Map,address) (#560-562)
getIndexOfKey(IterableMapping.Map,address) should be declared external:
- IterableMapping.getIndexOfKey(IterableMapping.Map,address) (#564-569)
getKeyAtIndex(IterableMapping.Map,uint256) should be declared external:
- IterableMapping.getKeyAtIndex(IterableMapping.Map,uint256) (#571-573)
size(IterableMapping.Map) should be declared external:
- IterableMapping.size(IterableMapping.Map) (#577-579)
name() should be declared external:
- ERC20.name() (#769-771)
symbol() should be declared external:
- ERC20.symbol() (#777-779)
decimals() should be declared external:
- ERC20.decimals() (#794-796)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (#820-823)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (#828-830)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (#839-842)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (#857-865)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (#879-882)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (#898-901)
distributeETHDividends(uint256) should be declared external:
- DividendPayingToken.distributeETHDividends(uint256) (#1124-1135)
withdrawDividend() should be declared external:
- DividendPayingToken.withdrawDividend() (#1139-1141)
- ElonsApeRevengeDividendTracker.withdrawDividend() (#1742-1744)
dividendOf(address) should be declared external:
- DividendPayingToken.dividendOf(address) (#1167-1169)
withdrawnDividendOf(address) should be declared external:
- DividendPayingToken.withdrawnDividendOf(address) (#1181-1183)
updateDividendTracker(address) should be declared external:
- ElonsApeRevenge.updateDividendTracker(address) (#1363-1378)
updateUniswapV2Router(address) should be declared external:
- ElonsApeRevenge.updateUniswapV2Router(address) (#1380-1387)
excludeMultipleAccountsFromFees(address[],bool) should be declared external:
- ElonsApeRevenge.excludeMultipleAccountsFromFees(address[],bool) (#1396-1402)
setAutomatedMarketMakerPair(address,bool) should be declared external:
- ElonsApeRevenge.setAutomatedMarketMakerPair(address,bool) (#1422-1426)
updateGasForProcessing(uint256) should be declared external:
- ElonsApeRevenge.updateGasForProcessing(uint256) (#1445-1450)
isExcludedFromFees(address) should be declared external:
- ElonsApeRevenge.isExcludedFromFees(address) (#1464-1466)
withdrawableDividendOf(address) should be declared external:
- ElonsApeRevenge.withdrawableDividendOf(address) (#1468-1470)
dividendTokenBalanceOf(address) should be declared external:
- ElonsApeRevenge.dividendTokenBalanceOf(address) (#1472-1474)
getAccountAtIndex(uint256) should be declared external:
- ElonsApeRevengeDividendTracker.getAccountAtIndex(uint256) (#1818-1835)
process(uint256) should be declared external:
- ElonsApeRevengeDividendTracker.process(uint256) (#1862-1907)
Use the external attribute for functions never called from the contract.
Additional information: link
Reentrancy in ElonsApeRevenge._setAutomatedMarketMakerPair(address,bool) (#1433-1442):
External calls:
- dividendTracker.excludeFromDividends(pair) (#1438)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (#1441)
Reentrancy in ElonsApeRevenge._transfer(address,address,uint256) (#1524-1609):
External calls:
- swapAndSendToFee(marketingTokens) (#1557)
- IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1617)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1675-1681)
- swapAndLiquify(swapTokens) (#1560)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1655-1661)
External calls sending eth:
- swapAndLiquify(swapTokens) (#1560)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#994)
- swapAndLiquify(swapTokens) (#1560)
- SwapAndLiquify(half,newBalance,otherHalf) (#1640)
- swapAndLiquify(swapTokens) (#1560)
Reentrancy in ElonsApeRevenge._transfer(address,address,uint256) (#1524-1609):
External calls:
- swapAndSendToFee(marketingTokens) (#1557)
- IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1617)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1675-1681)
- swapAndLiquify(swapTokens) (#1560)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1655-1661)
- swapAndSendDividends(sellTokens) (#1563)
- success = IERC20(BUSD).transfer(address(dividendTracker),dividends) (#1704)
- dividendTracker.distributeETHDividends(dividends) (#1707)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1675-1681)
External calls sending eth:
- swapAndLiquify(swapTokens) (#1560)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#994)
- swapAndSendDividends(sellTokens) (#1563)
- SendDividends(tokens,dividends) (#1708)
- swapAndSendDividends(sellTokens) (#1563)
- Transfer(sender,recipient,amount) (#929)
- super._transfer(from,address(this),fees) (#1591)
- Transfer(sender,recipient,amount) (#929)
- super._transfer(from,to,amount) (#1594)
Reentrancy in ElonsApeRevenge._transfer(address,address,uint256) (#1524-1609):
External calls:
- swapAndSendToFee(marketingTokens) (#1557)
- IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1617)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1675-1681)
- swapAndLiquify(swapTokens) (#1560)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1655-1661)
- swapAndSendDividends(sellTokens) (#1563)
- success = IERC20(BUSD).transfer(address(dividendTracker),dividends) (#1704)
- dividendTracker.distributeETHDividends(dividends) (#1707)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1675-1681)
- dividendTracker.setBalance(address(from),balanceOf(from)) (#1596)
- dividendTracker.setBalance(address(to),balanceOf(to)) (#1597)
- dividendTracker.process(gas) (#1602-1607)
External calls sending eth:
- swapAndLiquify(swapTokens) (#1560)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1603)
Reentrancy in ElonsApeRevenge.constructor() (#1318-1357):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1327-1328)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1333)
- dividendTracker.excludeFromDividends(pair) (#1438)
Event emitted after the call(s):
- ExcludeFromFees(account,excluded) (#1393)
- setMarketingWallet(0x1b30dC3FF02AEfaA22016E99c0658fF3ed69465a) (#1334)
- SetAutomatedMarketMakerPair(pair,value) (#1441)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1333)
Reentrancy in ElonsApeRevenge.constructor() (#1318-1357):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1327-1328)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1333)
- dividendTracker.excludeFromDividends(pair) (#1438)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (#1338)
- dividendTracker.excludeFromDividends(address(this)) (#1339)
- dividendTracker.excludeFromDividends(_newOwner) (#1340)
- dividendTracker.excludeFromDividends(deadWallet) (#1341)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (#1342)
Event emitted after the call(s):
- ExcludeFromFees(account,excluded) (#1393)
- excludeFromFees(address(this),true) (#1346)
- ExcludeFromFees(account,excluded) (#1393)
- excludeFromFees(_newOwner,true) (#1345)
- OwnershipTransferred(_owner,newOwner) (#329)
- transferOwnership(_newOwner) (#1354)
- Transfer(address(0),account,amount) (#948)
- _mint(owner(),1000000000000 * (10 ** 18)) (#1356)
Reentrancy in ElonsApeRevengeDividendTracker.processAccount(address,bool) (#1909-1919):
External calls:
- amount = _withdrawDividendOfUser(account) (#1910)
- success = IERC20(BUSD).transfer(user,_withdrawableDividend) (#1150)
Event emitted after the call(s):
- Claim(account,amount,automatic) (#1914)
Reentrancy in ElonsApeRevenge.processDividendTracker(uint256) (#1506-1509):
External calls:
- (iterations,claims,lastProcessedIndex) = dividendTracker.process(gas) (#1507)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,false,gas,tx.origin) (#1508)
Reentrancy in ElonsApeRevenge.swapAndLiquify(uint256) (#1620-1641):
External calls:
- swapTokensForEth(half) (#1632)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1655-1661)
- addLiquidity(otherHalf,newBalance) (#1638)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1638)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1690-1697)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#994)
- addLiquidity(otherHalf,newBalance) (#1638)
- SwapAndLiquify(half,newBalance,otherHalf) (#1640)
Reentrancy in ElonsApeRevenge.swapAndSendDividends(uint256) (#1701-1710):
External calls:
- swapTokensForETH(tokens) (#1702)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1675-1681)
- success = IERC20(BUSD).transfer(address(dividendTracker),dividends) (#1704)
- dividendTracker.distributeETHDividends(dividends) (#1707)
Event emitted after the call(s):
- SendDividends(tokens,dividends) (#1708)
Reentrancy in ElonsApeRevenge.updateDividendTracker(address) (#1363-1378):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (#1370)
- newDividendTracker.excludeFromDividends(address(this)) (#1371)
- newDividendTracker.excludeFromDividends(owner()) (#1372)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (#1373)
Event emitted after the call(s):
- UpdateDividendTracker(newAddress,address(dividendTracker)) (#1375)
Apply the check-effects-interactions pattern.
Additional information: link
ElonsApeRevengeDividendTracker.getAccount(address) (#1773-1816) uses timestamp for comparisons
Dangerous comparisons:
- nextClaimTime > block.timestamp (#1813-1815)
ElonsApeRevengeDividendTracker.canAutoClaim(uint256) (#1837-1843) uses timestamp for comparisons
Dangerous comparisons:
- lastClaimTime > block.timestamp (#1838)
- block.timestamp.sub(lastClaimTime) >= claimWait (#1842)
Avoid relying on block.timestamp.
Additional information: link
Context._msgData() (#274-277) is never used and should be removed
DividendPayingToken._transfer(address,address,uint256) (#1201-1207) is never used and should be removed
SafeMath.mod(uint256,uint256) (#233-235) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#249-252) is never used and should be removed
SafeMathInt.abs(int256) (#97-100) is never used and should be removed
SafeMathInt.div(int256,int256) (#68-74) is never used and should be removed
SafeMathInt.mul(int256,int256) (#56-63) is never used and should be removed
Remove unused functions.
Additional information: link
Pragma version^0.6.2 (#3) allows old versions
Pragma version^0.6.2 (#43) allows old versions
Pragma version^0.6.2 (#110) allows old versions
Pragma version^0.6.2 (#257) allows old versions
Pragma version^0.6.2 (#335) allows old versions
Pragma version^0.6.2 (#477) allows old versions
Pragma version^0.6.2 (#531) allows old versions
Pragma version^0.6.2 (#549) allows old versions
Pragma version^0.6.2 (#613) allows old versions
Pragma version^0.6.2 (#1019) allows old versions
Pragma version^0.6.2 (#1044) allows old versions
Pragma version^0.6.2 (#1080) allows old versions
Pragma version^0.6.2 (#1245) allows old versions
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
Function IUniswapV2Router01.WETH() (#339) is not in mixedCase
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#494) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#495) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#512) is not in mixedCase
Parameter DividendPayingToken.dividendOf(address)._owner (#1167) is not in mixedCase
Parameter DividendPayingToken.withdrawableDividendOf(address)._owner (#1174) is not in mixedCase
Parameter DividendPayingToken.withdrawnDividendOf(address)._owner (#1181) is not in mixedCase
Parameter DividendPayingToken.accumulativeDividendOf(address)._owner (#1191) is not in mixedCase
Variable DividendPayingToken.BUSD (#1092) is not in mixedCase
Constant DividendPayingToken.magnitude (#1099) is not in UPPER_CASE_WITH_UNDERSCORES
Variable ElonsApeRevenge.BUSD (#1261) is not in mixedCase
Variable ElonsApeRevenge._isBlacklisted (#1265) is not in mixedCase
Variable ElonsApeRevenge.BUSDRewardsFee (#1267) is not in mixedCase
Variable ElonsApeRevenge._marketingWalletAddress (#1271) is not in mixedCase
Parameter ElonsApeRevengeDividendTracker.getAccount(address)._account (#1773) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#275)" inContext (#269-278)
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 (#344) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#345)
Variable DividendPayingToken._withdrawDividendOfUser(address)._withdrawableDividend (#1146) is too similar to ElonsApeRevengeDividendTracker.getAccount(address).withdrawableDividends (#1778)
Prevent variables from having similar names.
Additional information: link
ElonsApeRevenge.constructor() (#1318-1357) uses literals with too many digits:
- _mint(owner(),1000000000000 * (10 ** 18)) (#1356)
ElonsApeRevenge.updateGasForProcessing(uint256) (#1445-1450) uses literals with too many digits:
- require(bool,string)(newValue >= 200000 && newValue <= 500000,ElonsApeRevenge: gasForProcessing must be between 200,000 and 500,000) (#1446)
ElonsApeRevenge.slitherConstructorVariables() (#1247-1711) uses literals with too many digits:
- deadWallet = 0x000000000000000000000000000000000000dEaD (#1258)
ElonsApeRevenge.slitherConstructorVariables() (#1247-1711) uses literals with too many digits:
- swapTokensAtAmount = 200000 * (10 ** 18) (#1263)
ElonsApeRevenge.slitherConstructorVariables() (#1247-1711) uses literals with too many digits:
- gasForProcessing = 400000 (#1275)
ElonsApeRevengeDividendTracker.constructor() (#1733-1736) uses literals with too many digits:
- minimumTokenBalanceForDividends = 100000 * (10 ** 18) (#1735)
ElonsApeRevengeDividendTracker.getAccountAtIndex(uint256) (#1818-1835) uses literals with too many digits:
- (0x0000000000000000000000000000000000000000,- 1,- 1,0,0,0,0,0) (#1829)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
SafeMathInt.MAX_INT256 (#51) is never used in SafeMathInt (#49-107)
Remove unused state variables.
Additional information: link
DividendPayingToken.BUSD (#1092) should be constant
ElonsApeRevenge.BUSD (#1261) should be constant
ElonsApeRevenge.deadWallet (#1258) should be constant
ElonsApeRevenge.swapTokensAtAmount (#1263) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
Unable to find website, listings and other project-related information
Young tokens have high risks of scam / price dump / death
Token has no active CoinGecko listing / rank
Token has no active CoinMarketCap listing / rank
Unable to find Twitter account
Telegram account has relatively few subscribers
Unable to find Blog account (Reddit or Medium)
Unable to find Youtube account
Unable to find Discord account