Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
RedDogeKing.addLiquidity(uint256,uint256) (BNBKing.sol#436-451) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in RedDogeKing._transfer(address,address,uint256) (BNBKing.sol#305-380):
External calls:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- (success) = address(dividendTracker).call{value: dividends}() (BNBKing.sol#457)
External calls sending eth:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- address(_marketingWalletAddress).transfer(newBalance) (BNBKing.sol#388)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- (success) = address(dividendTracker).call{value: dividends}() (BNBKing.sol#457)
State variables written after the call(s):
- super._transfer(from,address(this),fees) (BNBKing.sol#362)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (ERC20.sol#221)
- _balances[recipient] = _balances[recipient].add(amount) (ERC20.sol#222)
- super._transfer(from,to,amount) (BNBKing.sol#365)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (ERC20.sol#221)
- _balances[recipient] = _balances[recipient].add(amount) (ERC20.sol#222)
- swapping = false (BNBKing.sol#344)
Reentrancy in DividendPayingToken._withdrawDividendOfUser(address) (DividendPayingToken.sol#89-105):
External calls:
- (success) = user.call{gas: 3000,value: _withdrawableDividend}() (DividendPayingToken.sol#94)
State variables written after the call(s):
- withdrawnDividends[user] = withdrawnDividends[user].sub(_withdrawableDividend) (DividendPayingToken.sol#97)
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.
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.
RedDogeKing.updateUniswapV2Router(address)._uniswapV2Pair (BNBKing.sol#142-143) lacks a zero-check on :
- uniswapV2Pair = _uniswapV2Pair (BNBKing.sol#144)
RedDogeKing.setMarketingWallet(address).wallet (BNBKing.sol#162) lacks a zero-check on :
- _marketingWalletAddress = wallet (BNBKing.sol#163)
Check that the address is not zero.
Additional information: link
DividendPayingToken._withdrawDividendOfUser(address) (DividendPayingToken.sol#89-105) has external calls inside a loop: (success) = user.call{gas: 3000,value: _withdrawableDividend}() (DividendPayingToken.sol#94)
Favor pull over push strategy for external calls.
Additional information: link
Reentrancy in RedDogeKing.updateDividendTracker(address) (BNBKing.sol#121-136):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (BNBKing.sol#128)
- newDividendTracker.excludeFromDividends(address(this)) (BNBKing.sol#129)
- newDividendTracker.excludeFromDividends(owner()) (BNBKing.sol#130)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (BNBKing.sol#131)
State variables written after the call(s):
- dividendTracker = newDividendTracker (BNBKing.sol#135)
Apply the check-effects-interactions pattern.
Additional information: link
RedDogeKing._transfer(address,address,uint256).iterations (BNBKing.sol#373) is a local variable never initialized
RedDogeKing._transfer(address,address,uint256).claims (BNBKing.sol#373) is a local variable never initialized
RedDogeKing._transfer(address,address,uint256).lastProcessedIndex (BNBKing.sol#373) 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
RedDogeKing.claim() (BNBKing.sol#293-295) ignores return value by dividendTracker.processAccount(msg.sender,false) (BNBKing.sol#294)
RedDogeKing._transfer(address,address,uint256) (BNBKing.sol#305-380) ignores return value by dividendTracker.process(gas) (BNBKing.sol#373-378)
RedDogeKing.addLiquidity(uint256,uint256) (BNBKing.sol#436-451) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
Ensure that all the return values of the function calls are used.
Additional information: link
DividendPayingToken.constructor(string,string)._name (DividendPayingToken.sol#46) shadows:
- ERC20._name (ERC20.sol#43) (state variable)
DividendPayingToken.constructor(string,string)._symbol (DividendPayingToken.sol#46) shadows:
- ERC20._symbol (ERC20.sol#44) (state variable)
Rename the local variables that shadow another component.
Additional information: link
RedDogeKing.setBNBRewardsFee(uint256) (BNBKing.sol#167-170) should emit an event for:
- BNBRewardsFee = value (BNBKing.sol#168)
- totalFees = BNBRewardsFee.add(liquidityFee).add(marketingFee) (BNBKing.sol#169)
RedDogeKing.setLiquidityFee(uint256) (BNBKing.sol#172-175) should emit an event for:
- liquidityFee = value (BNBKing.sol#173)
- totalFees = BNBRewardsFee.add(liquidityFee).add(marketingFee) (BNBKing.sol#174)
RedDogeKing.setMarketingFee(uint256) (BNBKing.sol#177-180) should emit an event for:
- marketingFee = value (BNBKing.sol#178)
- totalFees = BNBRewardsFee.add(liquidityFee).add(marketingFee) (BNBKing.sol#179)
RedDogeKing.setExtraSellFee(uint256) (BNBKing.sol#182-184) should emit an event for:
- extraSellFee = value (BNBKing.sol#183)
RedDogeKing.setMaxTxAmount(uint256) (BNBKing.sol#186-188) should emit an event for:
- maxTxAmount = amount * 10 ** 18 (BNBKing.sol#187)
Emit an event for critical parameter changes.
Additional information: link
Variable 'RedDogeKing._transfer(address,address,uint256).iterations (BNBKing.sol#373)' in RedDogeKing._transfer(address,address,uint256) (BNBKing.sol#305-380) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (BNBKing.sol#374)
Variable 'RedDogeKing._transfer(address,address,uint256).lastProcessedIndex (BNBKing.sol#373)' in RedDogeKing._transfer(address,address,uint256) (BNBKing.sol#305-380) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (BNBKing.sol#374)
Variable 'RedDogeKing._transfer(address,address,uint256).claims (BNBKing.sol#373)' in RedDogeKing._transfer(address,address,uint256) (BNBKing.sol#305-380) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (BNBKing.sol#374)
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 RedDogeKing._transfer(address,address,uint256) (BNBKing.sol#305-380):
External calls:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
External calls sending eth:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- address(_marketingWalletAddress).transfer(newBalance) (BNBKing.sol#388)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
State variables written after the call(s):
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Reentrancy in RedDogeKing._transfer(address,address,uint256) (BNBKing.sol#305-380):
External calls:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- (success) = address(dividendTracker).call{value: dividends}() (BNBKing.sol#457)
External calls sending eth:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- address(_marketingWalletAddress).transfer(newBalance) (BNBKing.sol#388)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- (success) = address(dividendTracker).call{value: dividends}() (BNBKing.sol#457)
State variables written after the call(s):
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Reentrancy in RedDogeKing.constructor() (BNBKing.sol#82-115):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (BNBKing.sol#90-91)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (BNBKing.sol#94)
- uniswapV2Router = _uniswapV2Router (BNBKing.sol#93)
Reentrancy in RedDogeKing.constructor() (BNBKing.sol#82-115):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (BNBKing.sol#90-91)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (BNBKing.sol#96)
- dividendTracker.excludeFromDividends(pair) (BNBKing.sol#221)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (BNBKing.sol#99)
- dividendTracker.excludeFromDividends(address(this)) (BNBKing.sol#100)
- dividendTracker.excludeFromDividends(owner()) (BNBKing.sol#101)
- dividendTracker.excludeFromDividends(deadWallet) (BNBKing.sol#102)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (BNBKing.sol#103)
State variables written after the call(s):
- _mint(owner(),100000000 * (10 ** 18)) (BNBKing.sol#114)
- _balances[account] = _balances[account].add(amount) (ERC20.sol#241)
- excludeFromFees(owner(),true) (BNBKing.sol#106)
- _isExcludedFromFees[account] = excluded (BNBKing.sol#149)
- excludeFromFees(_marketingWalletAddress,true) (BNBKing.sol#107)
- _isExcludedFromFees[account] = excluded (BNBKing.sol#149)
- excludeFromFees(address(this),true) (BNBKing.sol#108)
- _isExcludedFromFees[account] = excluded (BNBKing.sol#149)
- _mint(owner(),100000000 * (10 ** 18)) (BNBKing.sol#114)
- _totalSupply = _totalSupply.add(amount) (ERC20.sol#240)
Reentrancy in DividendTracker.processAccount(address,bool) (BNBKing.sol#661-671):
External calls:
- amount = _withdrawDividendOfUser(account) (BNBKing.sol#662)
- (success) = user.call{gas: 3000,value: _withdrawableDividend}() (DividendPayingToken.sol#94)
State variables written after the call(s):
- lastClaimTimes[account] = block.timestamp (BNBKing.sol#665)
Reentrancy in RedDogeKing.swapAndLiquify(uint256) (BNBKing.sol#391-412):
External calls:
- swapTokensForEth(half) (BNBKing.sol#403)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- addLiquidity(otherHalf,newBalance) (BNBKing.sol#409)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (BNBKing.sol#409)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (BNBKing.sol#409)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Reentrancy in RedDogeKing.updateUniswapV2Router(address) (BNBKing.sol#138-145):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(uniswapV2Router.factory()).createPair(address(this),uniswapV2Router.WETH()) (BNBKing.sol#142-143)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (BNBKing.sol#144)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in RedDogeKing._setAutomatedMarketMakerPair(address,bool) (BNBKing.sol#216-225):
External calls:
- dividendTracker.excludeFromDividends(pair) (BNBKing.sol#221)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (BNBKing.sol#224)
Reentrancy in RedDogeKing._transfer(address,address,uint256) (BNBKing.sol#305-380):
External calls:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
External calls sending eth:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- address(_marketingWalletAddress).transfer(newBalance) (BNBKing.sol#388)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- SwapAndLiquify(half,newBalance,otherHalf) (BNBKing.sol#411)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
Reentrancy in RedDogeKing._transfer(address,address,uint256) (BNBKing.sol#305-380):
External calls:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- (success) = address(dividendTracker).call{value: dividends}() (BNBKing.sol#457)
External calls sending eth:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- address(_marketingWalletAddress).transfer(newBalance) (BNBKing.sol#388)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- (success) = address(dividendTracker).call{value: dividends}() (BNBKing.sol#457)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- SendDividends(tokens,dividends) (BNBKing.sol#460)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- Transfer(sender,recipient,amount) (ERC20.sol#223)
- super._transfer(from,address(this),fees) (BNBKing.sol#362)
- Transfer(sender,recipient,amount) (ERC20.sol#223)
- super._transfer(from,to,amount) (BNBKing.sol#365)
Reentrancy in RedDogeKing._transfer(address,address,uint256) (BNBKing.sol#305-380):
External calls:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- (success) = address(dividendTracker).call{value: dividends}() (BNBKing.sol#457)
- dividendTracker.setBalance(address(from),balanceOf(from)) (BNBKing.sol#367)
- dividendTracker.setBalance(address(to),balanceOf(to)) (BNBKing.sol#368)
- dividendTracker.process(gas) (BNBKing.sol#373-378)
External calls sending eth:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- address(_marketingWalletAddress).transfer(newBalance) (BNBKing.sol#388)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- (success) = address(dividendTracker).call{value: dividends}() (BNBKing.sol#457)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (BNBKing.sol#374)
Reentrancy in RedDogeKing.constructor() (BNBKing.sol#82-115):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (BNBKing.sol#90-91)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (BNBKing.sol#96)
- dividendTracker.excludeFromDividends(pair) (BNBKing.sol#221)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (BNBKing.sol#224)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (BNBKing.sol#96)
Reentrancy in RedDogeKing.constructor() (BNBKing.sol#82-115):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (BNBKing.sol#90-91)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (BNBKing.sol#96)
- dividendTracker.excludeFromDividends(pair) (BNBKing.sol#221)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (BNBKing.sol#99)
- dividendTracker.excludeFromDividends(address(this)) (BNBKing.sol#100)
- dividendTracker.excludeFromDividends(owner()) (BNBKing.sol#101)
- dividendTracker.excludeFromDividends(deadWallet) (BNBKing.sol#102)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (BNBKing.sol#103)
Event emitted after the call(s):
- ExcludeFromFees(account,excluded) (BNBKing.sol#151)
- excludeFromFees(address(this),true) (BNBKing.sol#108)
- ExcludeFromFees(account,excluded) (BNBKing.sol#151)
- excludeFromFees(_marketingWalletAddress,true) (BNBKing.sol#107)
- ExcludeFromFees(account,excluded) (BNBKing.sol#151)
- excludeFromFees(owner(),true) (BNBKing.sol#106)
- Transfer(address(0),account,amount) (ERC20.sol#242)
- _mint(owner(),100000000 * (10 ** 18)) (BNBKing.sol#114)
Reentrancy in DividendTracker.processAccount(address,bool) (BNBKing.sol#661-671):
External calls:
- amount = _withdrawDividendOfUser(account) (BNBKing.sol#662)
- (success) = user.call{gas: 3000,value: _withdrawableDividend}() (DividendPayingToken.sol#94)
Event emitted after the call(s):
- Claim(account,amount,automatic) (BNBKing.sol#666)
Reentrancy in RedDogeKing.processDividendTracker(uint256) (BNBKing.sol#288-291):
External calls:
- (iterations,claims,lastProcessedIndex) = dividendTracker.process(gas) (BNBKing.sol#289)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,false,gas,tx.origin) (BNBKing.sol#290)
Reentrancy in RedDogeKing.swapAndLiquify(uint256) (BNBKing.sol#391-412):
External calls:
- swapTokensForEth(half) (BNBKing.sol#403)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- addLiquidity(otherHalf,newBalance) (BNBKing.sol#409)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (BNBKing.sol#409)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- addLiquidity(otherHalf,newBalance) (BNBKing.sol#409)
- SwapAndLiquify(half,newBalance,otherHalf) (BNBKing.sol#411)
Reentrancy in RedDogeKing.swapAndSendDividends(uint256) (BNBKing.sol#453-462):
External calls:
- swapTokensForEth(tokens) (BNBKing.sol#455)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (BNBKing.sol#426-432)
- (success) = address(dividendTracker).call{value: dividends}() (BNBKing.sol#457)
External calls sending eth:
- (success) = address(dividendTracker).call{value: dividends}() (BNBKing.sol#457)
Event emitted after the call(s):
- SendDividends(tokens,dividends) (BNBKing.sol#460)
Reentrancy in RedDogeKing.updateDividendTracker(address) (BNBKing.sol#121-136):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (BNBKing.sol#128)
- newDividendTracker.excludeFromDividends(address(this)) (BNBKing.sol#129)
- newDividendTracker.excludeFromDividends(owner()) (BNBKing.sol#130)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (BNBKing.sol#131)
Event emitted after the call(s):
- UpdateDividendTracker(newAddress,address(dividendTracker)) (BNBKing.sol#133)
Apply the check-effects-interactions pattern.
Additional information: link
DividendTracker.getAccount(address) (BNBKing.sol#525-568) uses timestamp for comparisons
Dangerous comparisons:
- nextClaimTime > block.timestamp (BNBKing.sol#565-567)
DividendTracker.canAutoClaim(uint256) (BNBKing.sol#589-595) uses timestamp for comparisons
Dangerous comparisons:
- lastClaimTime > block.timestamp (BNBKing.sol#590)
- block.timestamp.sub(lastClaimTime) >= claimWait (BNBKing.sol#594)
Avoid relying on block.timestamp.
Additional information: link
Context._msgData() (Context.sol#20-23) is never used and should be removed
DividendPayingToken._transfer(address,address,uint256) (DividendPayingToken.sol#145-151) is never used and should be removed
SafeMath.mod(uint256,uint256) (SafeMath.sol#126-128) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (SafeMath.sol#142-145) is never used and should be removed
SafeMathInt.abs(int256) (SafeMathInt.sol#82-85) is never used and should be removed
SafeMathInt.div(int256,int256) (SafeMathInt.sol#53-59) is never used and should be removed
SafeMathInt.mul(int256,int256) (SafeMathInt.sol#41-48) is never used and should be removed
Remove unused functions.
Additional information: link
RedDogeKing.totalFees (BNBKing.sol#33) is set pre-construction with a non-constant function or state variable:
- BNBRewardsFee.add(liquidityFee).add(marketingFee)
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.2 (BNBKing.sol#3) allows old versions
Pragma version^0.6.2 (Context.sol#3) allows old versions
Pragma version^0.6.2 (DividendPayingToken.sol#3) allows old versions
Pragma version^0.6.2 (DividendPayingTokenInterface.sol#3) allows old versions
Pragma version^0.6.2 (DividendPayingTokenOptionalInterface.sol#3) allows old versions
Pragma version^0.6.2 (ERC20.sol#3) allows old versions
Pragma version^0.6.2 (IERC20.sol#3) allows old versions
Pragma version^0.6.2 (IERC20Metadata.sol#3) allows old versions
Pragma version^0.6.2 (IUniswapV2Factory.sol#3) allows old versions
Pragma version^0.6.2 (IUniswapV2Pair.sol#3) allows old versions
Pragma version^0.6.2 (IUniswapV2Router.sol#3) allows old versions
Pragma version^0.6.2 (IterableMapping.sol#2) allows old versions
Pragma version^0.6.2 (Ownable.sol#1) allows old versions
Pragma version^0.6.2 (SafeMath.sol#3) allows old versions
Pragma version^0.6.2 (SafeMathInt.sol#28) allows old versions
Pragma version^0.6.2 (SafeMathUint.sol#3) 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
Low level call in RedDogeKing.swapAndSendDividends(uint256) (BNBKing.sol#453-462):
- (success) = address(dividendTracker).call{value: dividends}() (BNBKing.sol#457)
Low level call in DividendPayingToken._withdrawDividendOfUser(address) (DividendPayingToken.sol#89-105):
- (success) = user.call{gas: 3000,value: _withdrawableDividend}() (DividendPayingToken.sol#94)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Variable RedDogeKing._isBlacklisted (BNBKing.sol#27) is not in mixedCase
Variable RedDogeKing.BNBRewardsFee (BNBKing.sol#29) is not in mixedCase
Variable RedDogeKing._marketingWalletAddress (BNBKing.sol#36) is not in mixedCase
Parameter DividendTracker.getAccount(address)._account (BNBKing.sol#525) is not in mixedCase
Parameter DividendPayingToken.dividendOf(address)._owner (DividendPayingToken.sol#111) is not in mixedCase
Parameter DividendPayingToken.withdrawableDividendOf(address)._owner (DividendPayingToken.sol#118) is not in mixedCase
Parameter DividendPayingToken.withdrawnDividendOf(address)._owner (DividendPayingToken.sol#125) is not in mixedCase
Parameter DividendPayingToken.accumulativeDividendOf(address)._owner (DividendPayingToken.sol#135) is not in mixedCase
Constant DividendPayingToken.magnitude (DividendPayingToken.sol#26) is not in UPPER_CASE_WITH_UNDERSCORES
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (IUniswapV2Pair.sol#20) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (IUniswapV2Pair.sol#21) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (IUniswapV2Pair.sol#38) is not in mixedCase
Function IUniswapV2Router01.WETH() (IUniswapV2Router.sol#7) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (Context.sol#21)" inContext (Context.sol#15-24)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in RedDogeKing._transfer(address,address,uint256) (BNBKing.sol#305-380):
External calls:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- address(_marketingWalletAddress).transfer(newBalance) (BNBKing.sol#388)
External calls sending eth:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- address(_marketingWalletAddress).transfer(newBalance) (BNBKing.sol#388)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
State variables written after the call(s):
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- SwapAndLiquify(half,newBalance,otherHalf) (BNBKing.sol#411)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
Reentrancy in RedDogeKing._transfer(address,address,uint256) (BNBKing.sol#305-380):
External calls:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- address(_marketingWalletAddress).transfer(newBalance) (BNBKing.sol#388)
External calls sending eth:
- swapAndSendToFee(marketingTokens) (BNBKing.sol#336)
- address(_marketingWalletAddress).transfer(newBalance) (BNBKing.sol#388)
- swapAndLiquify(swapTokens) (BNBKing.sol#339)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (BNBKing.sol#442-449)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- (success) = address(dividendTracker).call{value: dividends}() (BNBKing.sol#457)
State variables written after the call(s):
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- _allowances[owner][spender] = amount (ERC20.sol#287)
- super._transfer(from,address(this),fees) (BNBKing.sol#362)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (ERC20.sol#221)
- _balances[recipient] = _balances[recipient].add(amount) (ERC20.sol#222)
- super._transfer(from,to,amount) (BNBKing.sol#365)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (ERC20.sol#221)
- _balances[recipient] = _balances[recipient].add(amount) (ERC20.sol#222)
- swapping = false (BNBKing.sol#344)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (BNBKing.sol#374)
- SendDividends(tokens,dividends) (BNBKing.sol#460)
- swapAndSendDividends(sellTokens) (BNBKing.sol#342)
- Transfer(sender,recipient,amount) (ERC20.sol#223)
- super._transfer(from,address(this),fees) (BNBKing.sol#362)
- Transfer(sender,recipient,amount) (ERC20.sol#223)
- super._transfer(from,to,amount) (BNBKing.sol#365)
Apply the check-effects-interactions pattern.
Additional information: link
Variable DividendPayingToken._withdrawDividendOfUser(address)._withdrawableDividend (DividendPayingToken.sol#90) is too similar to DividendTracker.getAccount(address).withdrawableDividends (BNBKing.sol#530)
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (IUniswapV2Router.sol#12) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (IUniswapV2Router.sol#13)
Prevent variables from having similar names.
Additional information: link
RedDogeKing.constructor() (BNBKing.sol#82-115) uses literals with too many digits:
- _mint(owner(),100000000 * (10 ** 18)) (BNBKing.sol#114)
RedDogeKing.updateGasForProcessing(uint256) (BNBKing.sol#227-232) uses literals with too many digits:
- require(bool,string)(newValue >= 200000 && newValue <= 500000,gasForProcessing must be between 200,000 and 500,000) (BNBKing.sol#228)
RedDogeKing.slitherConstructorVariables() (BNBKing.sol#14-463) uses literals with too many digits:
- deadWallet = 0x000000000000000000000000000000000000dEaD (BNBKing.sol#24)
RedDogeKing.slitherConstructorVariables() (BNBKing.sol#14-463) uses literals with too many digits:
- swapTokensAtAmount = 1000000000000 * (10 ** 18) (BNBKing.sol#26)
RedDogeKing.slitherConstructorVariables() (BNBKing.sol#14-463) uses literals with too many digits:
- maxTxAmount = 1000000000000000 * (10 ** 18) (BNBKing.sol#34)
RedDogeKing.slitherConstructorVariables() (BNBKing.sol#14-463) uses literals with too many digits:
- gasForProcessing = 300000 (BNBKing.sol#40)
DividendTracker.constructor() (BNBKing.sol#485-488) uses literals with too many digits:
- minimumTokenBalanceForDividends = 200000 * (10 ** 18) (BNBKing.sol#487)
DividendTracker.getAccountAtIndex(uint256) (BNBKing.sol#570-587) uses literals with too many digits:
- (0x0000000000000000000000000000000000000000,- 1,- 1,0,0,0,0,0) (BNBKing.sol#581)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
SafeMathInt.MAX_INT256 (SafeMathInt.sol#36) is never used in SafeMathInt (SafeMathInt.sol#34-92)
Remove unused state variables.
Additional information: link
RedDogeKing.deadWallet (BNBKing.sol#24) should be constant
RedDogeKing.swapTokensAtAmount (BNBKing.sol#26) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
updateDividendTracker(address) should be declared external:
- RedDogeKing.updateDividendTracker(address) (BNBKing.sol#121-136)
updateUniswapV2Router(address) should be declared external:
- RedDogeKing.updateUniswapV2Router(address) (BNBKing.sol#138-145)
excludeMultipleAccountsFromFees(address[],bool) should be declared external:
- RedDogeKing.excludeMultipleAccountsFromFees(address[],bool) (BNBKing.sol#154-160)
setAutomatedMarketMakerPair(address,bool) should be declared external:
- RedDogeKing.setAutomatedMarketMakerPair(address,bool) (BNBKing.sol#190-194)
blacklistMultipleAccounts(address[],bool) should be declared external:
- RedDogeKing.blacklistMultipleAccounts(address[],bool) (BNBKing.sol#200-206)
transferToMultipleAccounts(address[],uint256) should be declared external:
- RedDogeKing.transferToMultipleAccounts(address[],uint256) (BNBKing.sol#208-214)
updateGasForProcessing(uint256) should be declared external:
- RedDogeKing.updateGasForProcessing(uint256) (BNBKing.sol#227-232)
isExcludedFromFees(address) should be declared external:
- RedDogeKing.isExcludedFromFees(address) (BNBKing.sol#246-248)
withdrawableDividendOf(address) should be declared external:
- RedDogeKing.withdrawableDividendOf(address) (BNBKing.sol#250-252)
dividendTokenBalanceOf(address) should be declared external:
- RedDogeKing.dividendTokenBalanceOf(address) (BNBKing.sol#254-256)
withdrawDividend() should be declared external:
- DividendPayingToken.withdrawDividend() (DividendPayingToken.sol#83-85)
- DividendTracker.withdrawDividend() (BNBKing.sol#494-496)
getAccountAtIndex(uint256) should be declared external:
- DividendTracker.getAccountAtIndex(uint256) (BNBKing.sol#570-587)
process(uint256) should be declared external:
- DividendTracker.process(uint256) (BNBKing.sol#614-659)
dividendOf(address) should be declared external:
- DividendPayingToken.dividendOf(address) (DividendPayingToken.sol#111-113)
withdrawnDividendOf(address) should be declared external:
- DividendPayingToken.withdrawnDividendOf(address) (DividendPayingToken.sol#125-127)
name() should be declared external:
- ERC20.name() (ERC20.sol#63-65)
symbol() should be declared external:
- ERC20.symbol() (ERC20.sol#71-73)
decimals() should be declared external:
- ERC20.decimals() (ERC20.sol#88-90)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (ERC20.sol#114-117)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (ERC20.sol#122-124)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (ERC20.sol#133-136)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (ERC20.sol#151-159)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (ERC20.sol#173-176)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (ERC20.sol#192-195)
get(IterableMapping.Map,address) should be declared external:
- IterableMapping.get(IterableMapping.Map,address) (IterableMapping.sol#13-15)
getIndexOfKey(IterableMapping.Map,address) should be declared external:
- IterableMapping.getIndexOfKey(IterableMapping.Map,address) (IterableMapping.sol#17-22)
getKeyAtIndex(IterableMapping.Map,uint256) should be declared external:
- IterableMapping.getKeyAtIndex(IterableMapping.Map,uint256) (IterableMapping.sol#24-26)
size(IterableMapping.Map) should be declared external:
- IterableMapping.size(IterableMapping.Map) (IterableMapping.sol#30-32)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (Ownable.sol#43-46)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (Ownable.sol#52-56)
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