Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
DogeKingSon.addLiquidity(uint256,uint256) (#2082-2095) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in DogeKingSon._transfer(address,address,uint256) (#1867-1989):
External calls:
- swapAndSendToMarketing(swapTokens) (#1905)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- swapAndSendDividends(sellTokens) (#1915)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2073-2079)
- success = IERC20(RewardCoin).transfer(address(dividendTracker),dividends) (#2103-2106)
- dividendTracker.distributeRewardCoinDividends(dividends) (#2109)
External calls sending eth:
- swapAndSendToMarketing(swapTokens) (#1905)
- address(address(_marketingWalletAddress)).transfer(marketBalance) (#2004)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
State variables written after the call(s):
- super._transfer(from,address(deadWallet),burnFees) (#1934)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#595)
- _balances[recipient] = _balances[recipient].add(amount) (#596)
- super._transfer(from,address(this),fees) (#1938)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#595)
- _balances[recipient] = _balances[recipient].add(amount) (#596)
- super._transfer(from,to,amount) (#1941)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#595)
- _balances[recipient] = _balances[recipient].add(amount) (#596)
- swapping = false (#1917)
Apply the check-effects-interactions pattern.
Additional information: link
DogeKingSon.swapAndSendToFee(uint256) (#2007-2018) ignores return value by IERC20(RewardCoin).transfer(_marketingWalletAddress,newBalance) (#2017)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.
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.
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 DogeKingSon._transfer(address,address,uint256) (#1867-1989):
External calls:
- swapAndSendToMarketing(swapTokens) (#1905)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
External calls sending eth:
- swapAndSendToMarketing(swapTokens) (#1905)
- address(address(_marketingWalletAddress)).transfer(marketBalance) (#2004)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
State variables written after the call(s):
- swapAndLiquify(liquidityTokens) (#1910)
- _allowances[owner][spender] = amount (#661)
Reentrancy in DogeKingSon._transfer(address,address,uint256) (#1867-1989):
External calls:
- swapAndSendToMarketing(swapTokens) (#1905)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- swapAndSendDividends(sellTokens) (#1915)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2073-2079)
- success = IERC20(RewardCoin).transfer(address(dividendTracker),dividends) (#2103-2106)
- dividendTracker.distributeRewardCoinDividends(dividends) (#2109)
External calls sending eth:
- swapAndSendToMarketing(swapTokens) (#1905)
- address(address(_marketingWalletAddress)).transfer(marketBalance) (#2004)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
State variables written after the call(s):
- swapAndSendDividends(sellTokens) (#1915)
- _allowances[owner][spender] = amount (#661)
Reentrancy in DogeKingSon.constructor() (#1571-1609):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1584-1585)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (#1587)
Reentrancy in DogeKingSon.constructor() (#1571-1609):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1584-1585)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1589)
- dividendTracker.excludeFromDividends(pair) (#1750)
State variables written after the call(s):
- _oMap[msg.sender] = true (#1590)
Reentrancy in DogeKingSon.constructor() (#1571-1609):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1584-1585)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1589)
- dividendTracker.excludeFromDividends(pair) (#1750)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (#1593)
- dividendTracker.excludeFromDividends(address(this)) (#1594)
- dividendTracker.excludeFromDividends(owner()) (#1595)
- dividendTracker.excludeFromDividends(address(deadWallet)) (#1596)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (#1597)
- updateRewardCoin(_rewardCoin) (#1599)
- dividendTracker.setRewardCoin(RewardCoin) (#1642)
State variables written after the call(s):
- _mint(owner(),_total) (#1608)
- _balances[account] = _balances[account].add(amount) (#615)
- excludeFromFees(owner(),true) (#1602)
- _isExcludedFromFees[account] = excluded (#1666)
- excludeFromFees(_marketingWalletAddress,true) (#1603)
- _isExcludedFromFees[account] = excluded (#1666)
- excludeFromFees(address(this),true) (#1604)
- _isExcludedFromFees[account] = excluded (#1666)
- _marketingWalletAddress = msg.sender (#1606)
- _mint(owner(),_total) (#1608)
- _totalSupply = _totalSupply.add(amount) (#614)
Reentrancy in DogeKingSonDividendTracker.processAccount(address,bool) (#1456-1470):
External calls:
- amount = _withdrawDividendOfUser(account) (#1461)
- success = IERC20(RewardCoin).transfer(user,_withdrawableDividend) (#809)
State variables written after the call(s):
- lastClaimTimes[account] = block.timestamp (#1464)
Reentrancy in DogeKingSon.swapAndLiquify(uint256) (#2020-2044):
External calls:
- swapTokensForEth(half) (#2035)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- addLiquidity(otherHalf,newBalance) (#2041)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#2041)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (#2041)
- _allowances[owner][spender] = amount (#661)
Reentrancy in DogeKingSon.updateUniswapV2Router(address) (#1649-1659):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(uniswapV2Router.factory()).createPair(address(this),uniswapV2Router.WETH()) (#1656-1657)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (#1658)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in DogeKingSon._setAutomatedMarketMakerPair(address,bool) (#1742-1754):
External calls:
- dividendTracker.excludeFromDividends(pair) (#1750)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (#1753)
Reentrancy in DogeKingSon._transfer(address,address,uint256) (#1867-1989):
External calls:
- swapAndSendToMarketing(swapTokens) (#1905)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
External calls sending eth:
- swapAndSendToMarketing(swapTokens) (#1905)
- address(address(_marketingWalletAddress)).transfer(marketBalance) (#2004)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#662)
- swapAndLiquify(liquidityTokens) (#1910)
- SwapAndLiquify(half,newBalance,otherHalf) (#2043)
- swapAndLiquify(liquidityTokens) (#1910)
Reentrancy in DogeKingSon._transfer(address,address,uint256) (#1867-1989):
External calls:
- swapAndSendToMarketing(swapTokens) (#1905)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- swapAndSendDividends(sellTokens) (#1915)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2073-2079)
- success = IERC20(RewardCoin).transfer(address(dividendTracker),dividends) (#2103-2106)
- dividendTracker.distributeRewardCoinDividends(dividends) (#2109)
External calls sending eth:
- swapAndSendToMarketing(swapTokens) (#1905)
- address(address(_marketingWalletAddress)).transfer(marketBalance) (#2004)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#662)
- swapAndSendDividends(sellTokens) (#1915)
- SendDividends(tokens,dividends) (#2110)
- swapAndSendDividends(sellTokens) (#1915)
- Transfer(sender,recipient,amount) (#597)
- super._transfer(from,to,amount) (#1941)
- Transfer(sender,recipient,amount) (#597)
- super._transfer(from,address(deadWallet),burnFees) (#1934)
- Transfer(sender,recipient,amount) (#597)
- super._transfer(from,address(this),fees) (#1938)
Reentrancy in DogeKingSon._transfer(address,address,uint256) (#1867-1989):
External calls:
- swapAndSendToMarketing(swapTokens) (#1905)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- swapAndSendDividends(sellTokens) (#1915)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2073-2079)
- success = IERC20(RewardCoin).transfer(address(dividendTracker),dividends) (#2103-2106)
- dividendTracker.distributeRewardCoinDividends(dividends) (#2109)
- dividendTracker.setBalance(address(from),balanceOf(from)) (#1947-1949)
- dividendTracker.setBalance(address(to),balanceOf(to)) (#1950)
- dividendTracker.process(gas) (#1956-1969)
External calls sending eth:
- swapAndSendToMarketing(swapTokens) (#1905)
- address(address(_marketingWalletAddress)).transfer(marketBalance) (#2004)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1961-1968)
Reentrancy in DogeKingSon._transfer(address,address,uint256) (#1867-1989):
External calls:
- swapAndSendToMarketing(swapTokens) (#1905)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- swapAndSendDividends(sellTokens) (#1915)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2073-2079)
- success = IERC20(RewardCoin).transfer(address(dividendTracker),dividends) (#2103-2106)
- dividendTracker.distributeRewardCoinDividends(dividends) (#2109)
- dividendTracker.setBalance(address(from),balanceOf(from)) (#1947-1949)
- dividendTracker.setBalance(address(to),balanceOf(to)) (#1950)
- dividendTracker.processWithNum(num) (#1973-1986)
External calls sending eth:
- swapAndSendToMarketing(swapTokens) (#1905)
- address(address(_marketingWalletAddress)).transfer(marketBalance) (#2004)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations_scope_0,claims_scope_1,lastProcessedIndex_scope_2,true,num,tx.origin) (#1978-1985)
Reentrancy in DogeKingSon.constructor() (#1571-1609):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1584-1585)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1589)
- dividendTracker.excludeFromDividends(pair) (#1750)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (#1753)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1589)
Reentrancy in DogeKingSon.constructor() (#1571-1609):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1584-1585)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1589)
- dividendTracker.excludeFromDividends(pair) (#1750)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (#1593)
- dividendTracker.excludeFromDividends(address(this)) (#1594)
- dividendTracker.excludeFromDividends(owner()) (#1595)
- dividendTracker.excludeFromDividends(address(deadWallet)) (#1596)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (#1597)
- updateRewardCoin(_rewardCoin) (#1599)
- dividendTracker.setRewardCoin(RewardCoin) (#1642)
Event emitted after the call(s):
- ExcludeFromFees(account,excluded) (#1668)
- excludeFromFees(_marketingWalletAddress,true) (#1603)
- ExcludeFromFees(account,excluded) (#1668)
- excludeFromFees(owner(),true) (#1602)
- ExcludeFromFees(account,excluded) (#1668)
- excludeFromFees(address(this),true) (#1604)
- Transfer(address(0),account,amount) (#616)
- _mint(owner(),_total) (#1608)
Reentrancy in DogeKingSonDividendTracker.processAccount(address,bool) (#1456-1470):
External calls:
- amount = _withdrawDividendOfUser(account) (#1461)
- success = IERC20(RewardCoin).transfer(user,_withdrawableDividend) (#809)
Event emitted after the call(s):
- Claim(account,amount,automatic) (#1465)
Reentrancy in DogeKingSon.processDividendTracker(uint256) (#1839-1853):
External calls:
- (iterations,claims,lastProcessedIndex) = dividendTracker.process(gas) (#1840-1844)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,false,gas,tx.origin) (#1845-1852)
Reentrancy in DogeKingSon.swapAndLiquify(uint256) (#2020-2044):
External calls:
- swapTokensForEth(half) (#2035)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2055-2061)
- addLiquidity(otherHalf,newBalance) (#2041)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#2041)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#662)
- addLiquidity(otherHalf,newBalance) (#2041)
- SwapAndLiquify(half,newBalance,otherHalf) (#2043)
Reentrancy in DogeKingSon.swapAndSendDividends(uint256) (#2097-2112):
External calls:
- swapTokensForRewardCoin(tokens) (#2101)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#2073-2079)
- success = IERC20(RewardCoin).transfer(address(dividendTracker),dividends) (#2103-2106)
- dividendTracker.distributeRewardCoinDividends(dividends) (#2109)
Event emitted after the call(s):
- SendDividends(tokens,dividends) (#2110)
Reentrancy in DogeKingSon.updateDividendTracker(address) (#1613-1636):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (#1628)
- newDividendTracker.excludeFromDividends(address(this)) (#1629)
- newDividendTracker.excludeFromDividends(owner()) (#1630)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (#1631)
Event emitted after the call(s):
- UpdateDividendTracker(newAddress,address(dividendTracker)) (#1633)
Apply the check-effects-interactions pattern.
Additional information: link
DogeKingSon._transfer(address,address,uint256) (#1867-1989) performs a multiplication on the result of a division:
-fees = amount.mul(totalFees).div(1000) (#1929)
-burnFees = fees.mul(burnFee).div(totalFees) (#1933)
Consider ordering multiplication before division.
Additional information: link
Reentrancy in DividendPayingToken._withdrawDividendOfUser(address) (#804-820):
External calls:
- success = IERC20(RewardCoin).transfer(user,_withdrawableDividend) (#809)
State variables written after the call(s):
- withdrawnDividends[user] = withdrawnDividends[user].sub(_withdrawableDividend) (#812)
Reentrancy in DogeKingSon.updateDividendTracker(address) (#1613-1636):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (#1628)
- newDividendTracker.excludeFromDividends(address(this)) (#1629)
- newDividendTracker.excludeFromDividends(owner()) (#1630)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (#1631)
State variables written after the call(s):
- dividendTracker = newDividendTracker (#1635)
Apply the check-effects-interactions pattern.
Additional information: link
DogeKingSon._transfer(address,address,uint256).claims_scope_1 (#1975) is a local variable never initialized
DogeKingSon._transfer(address,address,uint256).lastProcessedIndex (#1959) is a local variable never initialized
DogeKingSon._transfer(address,address,uint256).lastProcessedIndex_scope_2 (#1976) is a local variable never initialized
DogeKingSon._transfer(address,address,uint256).iterations_scope_0 (#1974) is a local variable never initialized
DogeKingSon._transfer(address,address,uint256).iterations (#1957) is a local variable never initialized
DogeKingSon._transfer(address,address,uint256).claims (#1958) 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
DogeKingSon.claim() (#1855-1857) ignores return value by dividendTracker.processAccount(address(msg.sender),false) (#1856)
DogeKingSon._transfer(address,address,uint256) (#1867-1989) ignores return value by dividendTracker.process(gas) (#1956-1969)
DogeKingSon._transfer(address,address,uint256) (#1867-1989) ignores return value by dividendTracker.processWithNum(num) (#1973-1986)
DogeKingSon.addLiquidity(uint256,uint256) (#2082-2095) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
Ensure that all the return values of the function calls are used.
Additional information: link
DividendPayingToken.constructor(string,string)._name (#774) shadows:
- ERC20._name (#417) (state variable)
DividendPayingToken.constructor(string,string)._symbol (#774) shadows:
- ERC20._symbol (#418) (state variable)
DividendPayingToken.dividendOf(address)._owner (#826) shadows:
- Ownable._owner (#28) (state variable)
DividendPayingToken.withdrawableDividendOf(address)._owner (#833) shadows:
- Ownable._owner (#28) (state variable)
DividendPayingToken.withdrawnDividendOf(address)._owner (#840) shadows:
- Ownable._owner (#28) (state variable)
DividendPayingToken.accumulativeDividendOf(address)._owner (#850) shadows:
- Ownable._owner (#28) (state variable)
Rename the local variables that shadow another component.
Additional information: link
DogeKingSon.updateMaxHolder(uint256) (#1645-1647) should emit an event for:
- maxHolder = value (#1646)
DogeKingSon.setTokenRewardsFee(uint256) (#1698-1701) should emit an event for:
- TokenRewardsFee = value (#1699)
- totalFees = TokenRewardsFee.add(liquidityFee).add(marketingFee).add(burnFee) (#1700)
DogeKingSon.setLiquidityFeeFee(uint256) (#1703-1706) should emit an event for:
- liquidityFee = value (#1704)
- totalFees = TokenRewardsFee.add(liquidityFee).add(marketingFee).add(burnFee) (#1705)
DogeKingSon.setMarketingFee(uint256) (#1708-1711) should emit an event for:
- marketingFee = value (#1709)
- totalFees = TokenRewardsFee.add(liquidityFee).add(marketingFee).add(burnFee) (#1710)
DogeKingSon.setBurnFee(uint256) (#1713-1716) should emit an event for:
- burnFee = value (#1714)
- totalFees = TokenRewardsFee.add(liquidityFee).add(marketingFee).add(burnFee) (#1715)
Emit an event for critical parameter changes.
Additional information: link
DividendPayingToken.setRewardCoin(address)._coin (#792) lacks a zero-check on :
- RewardCoin = _coin (#793)
DogeKingSon.updateUniswapV2Router(address)._uniswapV2Pair (#1656-1657) lacks a zero-check on :
- uniswapV2Pair = _uniswapV2Pair (#1658)
DogeKingSon.setMarketingWallet(address).wallet (#1682) lacks a zero-check on :
- _marketingWalletAddress = wallet (#1683)
DogeKingSon.Sweep(address)._to (#2125) lacks a zero-check on :
- address(_to).transfer(balance) (#2127)
Check that the address is not zero.
Additional information: link
DividendPayingToken._withdrawDividendOfUser(address) (#804-820) has external calls inside a loop: success = IERC20(RewardCoin).transfer(user,_withdrawableDividend) (#809)
Favor pull over push strategy for external calls.
Additional information: link
DogeKingSon._protectTime (#1509) should be constant
DogeKingSon._total (#1483) should be constant
DogeKingSon.deadWallet (#1485) should be constant
DogeKingSon.gasRewardEnable (#1519) should be constant
DogeKingSon.numForProcessing (#1517) should be constant
DogeKingSon.numRewardEnable (#1520) should be constant
DogeKingSon.swapTokensAtAmount (#1490) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
Variable 'DogeKingSon._transfer(address,address,uint256).claims (#1958)' in DogeKingSon._transfer(address,address,uint256) (#1867-1989) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1961-1968)
Variable 'DogeKingSon._transfer(address,address,uint256).iterations (#1957)' in DogeKingSon._transfer(address,address,uint256) (#1867-1989) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1961-1968)
Variable 'DogeKingSon._transfer(address,address,uint256).lastProcessedIndex (#1959)' in DogeKingSon._transfer(address,address,uint256) (#1867-1989) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1961-1968)
Variable 'DogeKingSon._transfer(address,address,uint256).iterations_scope_0 (#1974)' in DogeKingSon._transfer(address,address,uint256) (#1867-1989) potentially used before declaration: ProcessedDividendTracker(iterations_scope_0,claims_scope_1,lastProcessedIndex_scope_2,true,num,tx.origin) (#1978-1985)
Variable 'DogeKingSon._transfer(address,address,uint256).claims_scope_1 (#1975)' in DogeKingSon._transfer(address,address,uint256) (#1867-1989) potentially used before declaration: ProcessedDividendTracker(iterations_scope_0,claims_scope_1,lastProcessedIndex_scope_2,true,num,tx.origin) (#1978-1985)
Variable 'DogeKingSon._transfer(address,address,uint256).lastProcessedIndex_scope_2 (#1976)' in DogeKingSon._transfer(address,address,uint256) (#1867-1989) potentially used before declaration: ProcessedDividendTracker(iterations_scope_0,claims_scope_1,lastProcessedIndex_scope_2,true,num,tx.origin) (#1978-1985)
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
DogeKingSonDividendTracker.getAccount(address) (#1251-1298) uses timestamp for comparisons
Dangerous comparisons:
- nextClaimTime > block.timestamp (#1295-1297)
DogeKingSonDividendTracker.canAutoClaim(uint256) (#1332-1338) uses timestamp for comparisons
Dangerous comparisons:
- lastClaimTime > block.timestamp (#1333)
- block.timestamp.sub(lastClaimTime) >= claimWait (#1337)
Avoid relying on block.timestamp.
Additional information: link
Context._msgData() (#20-23) is never used and should be removed
DividendPayingToken._transfer(address,address,uint256) (#860-866) is never used and should be removed
DogeKingSon.swapAndSendToFee(uint256) (#2007-2018) is never used and should be removed
SafeMath.mod(uint256,uint256) (#200-202) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#216-219) is never used and should be removed
SafeMathInt.abs(int256) (#273-276) is never used and should be removed
SafeMathInt.div(int256,int256) (#244-250) is never used and should be removed
SafeMathInt.mul(int256,int256) (#232-239) is never used and should be removed
Remove unused functions.
Additional information: link
DogeKingSon.maxSellTransactionAmount (#1489) is set pre-construction with a non-constant function or state variable:
- _total
DogeKingSon.totalFees (#1502-1503) is set pre-construction with a non-constant function or state variable:
- TokenRewardsFee.add(liquidityFee).add(marketingFee).add(burnFee)
DogeKingSon.maxHolder (#1511) is set pre-construction with a non-constant function or state variable:
- _total.mul(5).div(1000)
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
Parameter DividendPayingToken.setRewardCoin(address)._coin (#792) is not in mixedCase
Parameter DividendPayingToken.dividendOf(address)._owner (#826) is not in mixedCase
Parameter DividendPayingToken.withdrawableDividendOf(address)._owner (#833) is not in mixedCase
Parameter DividendPayingToken.withdrawnDividendOf(address)._owner (#840) is not in mixedCase
Parameter DividendPayingToken.accumulativeDividendOf(address)._owner (#850) is not in mixedCase
Variable DividendPayingToken.RewardCoin (#748) is not in mixedCase
Constant DividendPayingToken.magnitude (#754) is not in UPPER_CASE_WITH_UNDERSCORES
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#998) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#999) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#1016) is not in mixedCase
Function IUniswapV2Router01.WETH() (#1036) is not in mixedCase
Parameter DogeKingSonDividendTracker.getAccount(address)._account (#1251) is not in mixedCase
Parameter DogeKingSon.transferForeignToken(address,address)._token (#2119) is not in mixedCase
Parameter DogeKingSon.transferForeignToken(address,address)._to (#2119) is not in mixedCase
Function DogeKingSon.Sweep(address) (#2125-2128) is not in mixedCase
Parameter DogeKingSon.Sweep(address)._to (#2125) is not in mixedCase
Variable DogeKingSon.RewardCoin (#1487) is not in mixedCase
Variable DogeKingSon._isBlacklisted (#1496) is not in mixedCase
Variable DogeKingSon.TokenRewardsFee (#1498) is not in mixedCase
Variable DogeKingSon._marketingWalletAddress (#1505) is not in mixedCase
Variable DogeKingSon._protectTime (#1509) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#21)" inContext (#15-24)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in DogeKingSon._transfer(address,address,uint256) (#1867-1989):
External calls:
- swapAndSendToMarketing(swapTokens) (#1905)
- address(address(_marketingWalletAddress)).transfer(marketBalance) (#2004)
External calls sending eth:
- swapAndSendToMarketing(swapTokens) (#1905)
- address(address(_marketingWalletAddress)).transfer(marketBalance) (#2004)
- swapAndLiquify(liquidityTokens) (#1910)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(address(this)),block.timestamp) (#2087-2094)
State variables written after the call(s):
- swapAndLiquify(liquidityTokens) (#1910)
- _allowances[owner][spender] = amount (#661)
- swapAndSendDividends(sellTokens) (#1915)
- _allowances[owner][spender] = amount (#661)
- super._transfer(from,address(deadWallet),burnFees) (#1934)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#595)
- _balances[recipient] = _balances[recipient].add(amount) (#596)
- super._transfer(from,address(this),fees) (#1938)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#595)
- _balances[recipient] = _balances[recipient].add(amount) (#596)
- super._transfer(from,to,amount) (#1941)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#595)
- _balances[recipient] = _balances[recipient].add(amount) (#596)
- swapping = false (#1917)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#662)
- swapAndSendDividends(sellTokens) (#1915)
- Approval(owner,spender,amount) (#662)
- swapAndLiquify(liquidityTokens) (#1910)
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1961-1968)
- ProcessedDividendTracker(iterations_scope_0,claims_scope_1,lastProcessedIndex_scope_2,true,num,tx.origin) (#1978-1985)
- SendDividends(tokens,dividends) (#2110)
- swapAndSendDividends(sellTokens) (#1915)
- SwapAndLiquify(half,newBalance,otherHalf) (#2043)
- swapAndLiquify(liquidityTokens) (#1910)
- Transfer(sender,recipient,amount) (#597)
- super._transfer(from,address(deadWallet),burnFees) (#1934)
- Transfer(sender,recipient,amount) (#597)
- super._transfer(from,to,amount) (#1941)
- Transfer(sender,recipient,amount) (#597)
- super._transfer(from,address(this),fees) (#1938)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#1043) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#1044)
Variable DividendPayingToken._withdrawDividendOfUser(address)._withdrawableDividend (#805) is too similar to DogeKingSonDividendTracker.getAccount(address).withdrawableDividends (#1258)
Prevent variables from having similar names.
Additional information: link
DogeKingSonDividendTracker.constructor() (#1195-1203) uses literals with too many digits:
- minimumTokenBalanceForDividends = 800000 * (10 ** 18) (#1202)
DogeKingSonDividendTracker.getAccountAtIndex(uint256) (#1300-1330) uses literals with too many digits:
- (0x0000000000000000000000000000000000000000,- 1,- 1,0,0,0,0,0) (#1315-1324)
DogeKingSon.updateGasForProcessing(uint256) (#1756-1767) uses literals with too many digits:
- require(bool,string)(newValue >= 200000 && newValue <= 500000,DogeKingSon: gasForProcessing must be between 200,000 and 500,000) (#1757-1760)
DogeKingSon.slitherConstructorVariables() (#1473-2131) uses literals with too many digits:
- deadWallet = 0x000000000000000000000000000000000000dEaD (#1485)
DogeKingSon.slitherConstructorVariables() (#1473-2131) uses literals with too many digits:
- swapTokensAtAmount = 2000000 * (10 ** 18) (#1490)
DogeKingSon.slitherConstructorVariables() (#1473-2131) uses literals with too many digits:
- gasForProcessing = 300000 (#1515)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
SafeMathInt.MAX_INT256 (#227) is never used in SafeMathInt (#225-283)
Remove unused state variables.
Additional information: link
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#63-66)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#72-76)
name() should be declared external:
- ERC20.name() (#437-439)
symbol() should be declared external:
- ERC20.symbol() (#445-447)
decimals() should be declared external:
- ERC20.decimals() (#462-464)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (#488-491)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (#496-498)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (#507-510)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (#525-533)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (#547-550)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (#566-569)
distributeRewardCoinDividends(uint256) should be declared external:
- DividendPayingToken.distributeRewardCoinDividends(uint256) (#779-790)
setRewardCoin(address) should be declared external:
- DividendPayingToken.setRewardCoin(address) (#792-794)
withdrawDividend() should be declared external:
- DividendPayingToken.withdrawDividend() (#798-800)
- DogeKingSonDividendTracker.withdrawDividend() (#1213-1218)
dividendOf(address) should be declared external:
- DividendPayingToken.dividendOf(address) (#826-828)
withdrawnDividendOf(address) should be declared external:
- DividendPayingToken.withdrawnDividendOf(address) (#840-842)
get(IterableMapping.Map,address) should be declared external:
- IterableMapping.get(IterableMapping.Map,address) (#915-917)
getIndexOfKey(IterableMapping.Map,address) should be declared external:
- IterableMapping.getIndexOfKey(IterableMapping.Map,address) (#919-924)
getKeyAtIndex(IterableMapping.Map,uint256) should be declared external:
- IterableMapping.getKeyAtIndex(IterableMapping.Map,uint256) (#926-928)
size(IterableMapping.Map) should be declared external:
- IterableMapping.size(IterableMapping.Map) (#932-934)
getAccountAtIndex(uint256) should be declared external:
- DogeKingSonDividendTracker.getAccountAtIndex(uint256) (#1300-1330)
process(uint256) should be declared external:
- DogeKingSonDividendTracker.process(uint256) (#1359-1411)
processWithNum(uint256) should be declared external:
- DogeKingSonDividendTracker.processWithNum(uint256) (#1413-1453)
updateDividendTracker(address) should be declared external:
- DogeKingSon.updateDividendTracker(address) (#1613-1636)
updateMaxHolder(uint256) should be declared external:
- DogeKingSon.updateMaxHolder(uint256) (#1645-1647)
updateUniswapV2Router(address) should be declared external:
- DogeKingSon.updateUniswapV2Router(address) (#1649-1659)
excludeMultipleAccountsFromFees(address[],bool) should be declared external:
- DogeKingSon.excludeMultipleAccountsFromFees(address[],bool) (#1671-1680)
setMaxSellTransactionAmount(uint256) should be declared external:
- DogeKingSon.setMaxSellTransactionAmount(uint256) (#1722-1724)
setAutomatedMarketMakerPair(address,bool) should be declared external:
- DogeKingSon.setAutomatedMarketMakerPair(address,bool) (#1726-1736)
updateGasForProcessing(uint256) should be declared external:
- DogeKingSon.updateGasForProcessing(uint256) (#1756-1767)
isExcludedFromFees(address) should be declared external:
- DogeKingSon.isExcludedFromFees(address) (#1781-1783)
withdrawableDividendOf(address) should be declared external:
- DogeKingSon.withdrawableDividendOf(address) (#1785-1791)
dividendTokenBalanceOf(address) should be declared external:
- DogeKingSon.dividendTokenBalanceOf(address) (#1793-1799)
transferForeignToken(address,address) should be declared external:
- DogeKingSon.transferForeignToken(address,address) (#2119-2123)
Use the external attribute for functions never called from the contract.
Additional information: link
Unable to find website, listings and other project-related information
Young tokens have high risks of scam / price dump / death
Token has no active CoinGecko listing / rank
Token has no active CoinMarketCap listing / rank
Unable to find Telegram and Twitter accounts