Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
BNBK.addLiquidity(uint256,uint256) (bnbking.sol#447-462) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in BNBK._transfer(address,address,uint256) (bnbking.sol#300-371):
External calls:
- swapAndSendToFee(marketingTokens) (bnbking.sol#327)
- IERC20(BNBKing).transfer(_marketingWalletAddress,newBalance) (bnbking.sol#379)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- swapAndSendDividends(sellTokens) (bnbking.sol#333)
- success = IERC20(BNBKing).transfer(address(dividendTracker),dividends) (bnbking.sol#467)
- dividendTracker.distributeBNBKingDividends(dividends) (bnbking.sol#470)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
External calls sending eth:
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
State variables written after the call(s):
- super._transfer(from,address(this),fees) (bnbking.sol#353)
- _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#356)
- _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#335)
Apply the check-effects-interactions pattern.
Additional information: link
BNBK.swapAndSendToFee(uint256) (bnbking.sol#373-380) ignores return value by IERC20(BNBKing).transfer(_marketingWalletAddress,newBalance) (bnbking.sol#379)
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.
Reentrancy in DividendPayingToken._withdrawDividendOfUser(address) (DividendPayingToken.sol#76-92):
External calls:
- success = IERC20(BNBKing).transfer(user,_withdrawableDividend) (DividendPayingToken.sol#81)
State variables written after the call(s):
- withdrawnDividends[user] = withdrawnDividends[user].sub(_withdrawableDividend) (DividendPayingToken.sol#84)
Reentrancy in BNBK.updateDividendTracker(address) (bnbking.sol#131-146):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (bnbking.sol#138)
- newDividendTracker.excludeFromDividends(address(this)) (bnbking.sol#139)
- newDividendTracker.excludeFromDividends(owner()) (bnbking.sol#140)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (bnbking.sol#141)
State variables written after the call(s):
- dividendTracker = newDividendTracker (bnbking.sol#145)
Apply the check-effects-interactions pattern.
Additional information: link
BNBK._transfer(address,address,uint256).iterations (bnbking.sol#364) is a local variable never initialized
BNBK._transfer(address,address,uint256).claims (bnbking.sol#364) is a local variable never initialized
BNBK._transfer(address,address,uint256).lastProcessedIndex (bnbking.sol#364) 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
BNBK.claim() (bnbking.sol#287-289) ignores return value by dividendTracker.processAccount(msg.sender,false) (bnbking.sol#288)
BNBK._transfer(address,address,uint256) (bnbking.sol#300-371) ignores return value by dividendTracker.process(gas) (bnbking.sol#364-369)
BNBK.addLiquidity(uint256,uint256) (bnbking.sol#447-462) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
Ensure that all the return values of the function calls are used.
Additional information: link
DividendPayingToken.constructor(string,string)._name (DividendPayingToken.sol#50) shadows:
- ERC20._name (ERC20.sol#43) (state variable)
DividendPayingToken.constructor(string,string)._symbol (DividendPayingToken.sol#50) shadows:
- ERC20._symbol (ERC20.sol#44) (state variable)
DividendPayingToken.dividendOf(address)._owner (DividendPayingToken.sol#98) shadows:
- Ownable._owner (Ownable.sol#8) (state variable)
DividendPayingToken.withdrawableDividendOf(address)._owner (DividendPayingToken.sol#105) shadows:
- Ownable._owner (Ownable.sol#8) (state variable)
DividendPayingToken.withdrawnDividendOf(address)._owner (DividendPayingToken.sol#112) shadows:
- Ownable._owner (Ownable.sol#8) (state variable)
DividendPayingToken.accumulativeDividendOf(address)._owner (DividendPayingToken.sol#122) shadows:
- Ownable._owner (Ownable.sol#8) (state variable)
Rename the local variables that shadow another component.
Additional information: link
BNBK.setBNBKingRewardsFee(uint256) (bnbking.sol#180-183) should emit an event for:
- BNBKingRewardsFee = value (bnbking.sol#181)
- totalFees = BNBKingRewardsFee.add(liquidityFee).add(marketingFee) (bnbking.sol#182)
BNBK.setLiquiditFee(uint256) (bnbking.sol#185-188) should emit an event for:
- liquidityFee = value (bnbking.sol#186)
- totalFees = BNBKingRewardsFee.add(liquidityFee).add(marketingFee) (bnbking.sol#187)
BNBK.setMarketingFee(uint256) (bnbking.sol#190-194) should emit an event for:
- marketingFee = value (bnbking.sol#191)
- totalFees = BNBKingRewardsFee.add(liquidityFee).add(marketingFee) (bnbking.sol#192)
Emit an event for critical parameter changes.
Additional information: link
BNBK.updateUniswapV2Router(address)._uniswapV2Pair (bnbking.sol#152-153) lacks a zero-check on :
- uniswapV2Pair = _uniswapV2Pair (bnbking.sol#154)
BNBK.setMarketingWallet(address).wallet (bnbking.sol#176) lacks a zero-check on :
- _marketingWalletAddress = wallet (bnbking.sol#177)
Check that the address is not zero.
Additional information: link
DividendPayingToken._withdrawDividendOfUser(address) (DividendPayingToken.sol#76-92) has external calls inside a loop: success = IERC20(BNBKing).transfer(user,_withdrawableDividend) (DividendPayingToken.sol#81)
Favor pull over push strategy for external calls.
Additional information: link
Variable 'BNBK._transfer(address,address,uint256).lastProcessedIndex (bnbking.sol#364)' in BNBK._transfer(address,address,uint256) (bnbking.sol#300-371) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (bnbking.sol#365)
Variable 'BNBK._transfer(address,address,uint256).claims (bnbking.sol#364)' in BNBK._transfer(address,address,uint256) (bnbking.sol#300-371) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (bnbking.sol#365)
Variable 'BNBK._transfer(address,address,uint256).iterations (bnbking.sol#364)' in BNBK._transfer(address,address,uint256) (bnbking.sol#300-371) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (bnbking.sol#365)
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 BNBK._transfer(address,address,uint256) (bnbking.sol#300-371):
External calls:
- swapAndSendToFee(marketingTokens) (bnbking.sol#327)
- IERC20(BNBKing).transfer(_marketingWalletAddress,newBalance) (bnbking.sol#379)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
External calls sending eth:
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
State variables written after the call(s):
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Reentrancy in BNBK._transfer(address,address,uint256) (bnbking.sol#300-371):
External calls:
- swapAndSendToFee(marketingTokens) (bnbking.sol#327)
- IERC20(BNBKing).transfer(_marketingWalletAddress,newBalance) (bnbking.sol#379)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- swapAndSendDividends(sellTokens) (bnbking.sol#333)
- success = IERC20(BNBKing).transfer(address(dividendTracker),dividends) (bnbking.sol#467)
- dividendTracker.distributeBNBKingDividends(dividends) (bnbking.sol#470)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
External calls sending eth:
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
State variables written after the call(s):
- swapAndSendDividends(sellTokens) (bnbking.sol#333)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Reentrancy in BNBK.constructor() (bnbking.sol#92-125):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (bnbking.sol#100-101)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (bnbking.sol#104)
- uniswapV2Router = _uniswapV2Router (bnbking.sol#103)
Reentrancy in BNBK.constructor() (bnbking.sol#92-125):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (bnbking.sol#100-101)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (bnbking.sol#106)
- dividendTracker.excludeFromDividends(pair) (bnbking.sol#213)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (bnbking.sol#109)
- dividendTracker.excludeFromDividends(address(this)) (bnbking.sol#110)
- dividendTracker.excludeFromDividends(owner()) (bnbking.sol#111)
- dividendTracker.excludeFromDividends(deadWallet) (bnbking.sol#112)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (bnbking.sol#113)
State variables written after the call(s):
- _mint(owner(),1000000000000000 * (10 ** 18)) (bnbking.sol#124)
- _balances[account] = _balances[account].add(amount) (ERC20.sol#241)
- excludeFromFees(owner(),true) (bnbking.sol#116)
- _isExcludedFromFees[account] = excluded (bnbking.sol#163)
- excludeFromFees(address(this),true) (bnbking.sol#118)
- _isExcludedFromFees[account] = excluded (bnbking.sol#163)
- _mint(owner(),1000000000000000 * (10 ** 18)) (bnbking.sol#124)
- _totalSupply = _totalSupply.add(amount) (ERC20.sol#240)
Reentrancy in BNBKingDividendTracker.processAccount(address,bool) (bnbking.sol#673-683):
External calls:
- amount = _withdrawDividendOfUser(account) (bnbking.sol#674)
- success = IERC20(BNBKing).transfer(user,_withdrawableDividend) (DividendPayingToken.sol#81)
State variables written after the call(s):
- lastClaimTimes[account] = block.timestamp (bnbking.sol#677)
Reentrancy in BNBK.swapAndLiquify(uint256) (bnbking.sol#382-403):
External calls:
- swapTokensForEth(half) (bnbking.sol#394)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- addLiquidity(otherHalf,newBalance) (bnbking.sol#400)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (bnbking.sol#400)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (bnbking.sol#400)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Reentrancy in BNBK.updateUniswapV2Router(address) (bnbking.sol#148-155):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(uniswapV2Router.factory()).createPair(address(this),uniswapV2Router.WETH()) (bnbking.sol#152-153)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (bnbking.sol#154)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in BNBK._setAutomatedMarketMakerPair(address,bool) (bnbking.sol#208-217):
External calls:
- dividendTracker.excludeFromDividends(pair) (bnbking.sol#213)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (bnbking.sol#216)
Reentrancy in BNBK._transfer(address,address,uint256) (bnbking.sol#300-371):
External calls:
- swapAndSendToFee(marketingTokens) (bnbking.sol#327)
- IERC20(BNBKing).transfer(_marketingWalletAddress,newBalance) (bnbking.sol#379)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
External calls sending eth:
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- SwapAndLiquify(half,newBalance,otherHalf) (bnbking.sol#402)
- swapAndLiquify(swapTokens) (bnbking.sol#330)
Reentrancy in BNBK._transfer(address,address,uint256) (bnbking.sol#300-371):
External calls:
- swapAndSendToFee(marketingTokens) (bnbking.sol#327)
- IERC20(BNBKing).transfer(_marketingWalletAddress,newBalance) (bnbking.sol#379)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- swapAndSendDividends(sellTokens) (bnbking.sol#333)
- success = IERC20(BNBKing).transfer(address(dividendTracker),dividends) (bnbking.sol#467)
- dividendTracker.distributeBNBKingDividends(dividends) (bnbking.sol#470)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
External calls sending eth:
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapAndSendDividends(sellTokens) (bnbking.sol#333)
- SendDividends(tokens,dividends) (bnbking.sol#471)
- swapAndSendDividends(sellTokens) (bnbking.sol#333)
- Transfer(sender,recipient,amount) (ERC20.sol#223)
- super._transfer(from,address(this),fees) (bnbking.sol#353)
- Transfer(sender,recipient,amount) (ERC20.sol#223)
- super._transfer(from,to,amount) (bnbking.sol#356)
Reentrancy in BNBK._transfer(address,address,uint256) (bnbking.sol#300-371):
External calls:
- swapAndSendToFee(marketingTokens) (bnbking.sol#327)
- IERC20(BNBKing).transfer(_marketingWalletAddress,newBalance) (bnbking.sol#379)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- swapAndSendDividends(sellTokens) (bnbking.sol#333)
- success = IERC20(BNBKing).transfer(address(dividendTracker),dividends) (bnbking.sol#467)
- dividendTracker.distributeBNBKingDividends(dividends) (bnbking.sol#470)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- dividendTracker.setBalance(address(from),balanceOf(from)) (bnbking.sol#358)
- dividendTracker.setBalance(address(to),balanceOf(to)) (bnbking.sol#359)
- dividendTracker.process(gas) (bnbking.sol#364-369)
External calls sending eth:
- swapAndLiquify(swapTokens) (bnbking.sol#330)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (bnbking.sol#365)
Reentrancy in BNBK.constructor() (bnbking.sol#92-125):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (bnbking.sol#100-101)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (bnbking.sol#106)
- dividendTracker.excludeFromDividends(pair) (bnbking.sol#213)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (bnbking.sol#216)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (bnbking.sol#106)
Reentrancy in BNBK.constructor() (bnbking.sol#92-125):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (bnbking.sol#100-101)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (bnbking.sol#106)
- dividendTracker.excludeFromDividends(pair) (bnbking.sol#213)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (bnbking.sol#109)
- dividendTracker.excludeFromDividends(address(this)) (bnbking.sol#110)
- dividendTracker.excludeFromDividends(owner()) (bnbking.sol#111)
- dividendTracker.excludeFromDividends(deadWallet) (bnbking.sol#112)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (bnbking.sol#113)
Event emitted after the call(s):
- ExcludeFromFees(account,excluded) (bnbking.sol#165)
- excludeFromFees(address(this),true) (bnbking.sol#118)
- ExcludeFromFees(account,excluded) (bnbking.sol#165)
- excludeFromFees(owner(),true) (bnbking.sol#116)
- Transfer(address(0),account,amount) (ERC20.sol#242)
- _mint(owner(),1000000000000000 * (10 ** 18)) (bnbking.sol#124)
Reentrancy in BNBKingDividendTracker.processAccount(address,bool) (bnbking.sol#673-683):
External calls:
- amount = _withdrawDividendOfUser(account) (bnbking.sol#674)
- success = IERC20(BNBKing).transfer(user,_withdrawableDividend) (DividendPayingToken.sol#81)
Event emitted after the call(s):
- Claim(account,amount,automatic) (bnbking.sol#678)
Reentrancy in BNBK.processDividendTracker(uint256) (bnbking.sol#282-285):
External calls:
- (iterations,claims,lastProcessedIndex) = dividendTracker.process(gas) (bnbking.sol#283)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,false,gas,tx.origin) (bnbking.sol#284)
Reentrancy in BNBK.swapAndLiquify(uint256) (bnbking.sol#382-403):
External calls:
- swapTokensForEth(half) (bnbking.sol#394)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- addLiquidity(otherHalf,newBalance) (bnbking.sol#400)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (bnbking.sol#400)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityWallet,block.timestamp) (bnbking.sol#453-460)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- addLiquidity(otherHalf,newBalance) (bnbking.sol#400)
- SwapAndLiquify(half,newBalance,otherHalf) (bnbking.sol#402)
Reentrancy in BNBK.swapAndSendDividends(uint256) (bnbking.sol#464-473):
External calls:
- swapTokensForEth(tokens) (bnbking.sol#465)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (bnbking.sol#417-423)
- success = IERC20(BNBKing).transfer(address(dividendTracker),dividends) (bnbking.sol#467)
- dividendTracker.distributeBNBKingDividends(dividends) (bnbking.sol#470)
Event emitted after the call(s):
- SendDividends(tokens,dividends) (bnbking.sol#471)
Reentrancy in BNBK.updateDividendTracker(address) (bnbking.sol#131-146):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (bnbking.sol#138)
- newDividendTracker.excludeFromDividends(address(this)) (bnbking.sol#139)
- newDividendTracker.excludeFromDividends(owner()) (bnbking.sol#140)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (bnbking.sol#141)
Event emitted after the call(s):
- UpdateDividendTracker(newAddress,address(dividendTracker)) (bnbking.sol#143)
Apply the check-effects-interactions pattern.
Additional information: link
BNBKingDividendTracker.getAccount(address) (bnbking.sol#537-580) uses timestamp for comparisons
Dangerous comparisons:
- nextClaimTime > block.timestamp (bnbking.sol#577-579)
BNBKingDividendTracker.canAutoClaim(uint256) (bnbking.sol#601-607) uses timestamp for comparisons
Dangerous comparisons:
- lastClaimTime > block.timestamp (bnbking.sol#602)
- block.timestamp.sub(lastClaimTime) >= claimWait (bnbking.sol#606)
Avoid relying on block.timestamp.
Additional information: link
BNBK.swapTokensForBNBKing(uint256) (bnbking.sol#428-445) is never used and should be removed
Context._msgData() (Context.sol#20-23) is never used and should be removed
DividendPayingToken._transfer(address,address,uint256) (DividendPayingToken.sol#132-138) 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
BNBK.totalFees (bnbking.sol#41) is set pre-construction with a non-constant function or state variable:
- BNBKingRewardsFee.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 (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
Pragma version^0.6.2 (bnbking.sol#9) 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
Parameter DividendPayingToken.dividendOf(address)._owner (DividendPayingToken.sol#98) is not in mixedCase
Parameter DividendPayingToken.withdrawableDividendOf(address)._owner (DividendPayingToken.sol#105) is not in mixedCase
Parameter DividendPayingToken.withdrawnDividendOf(address)._owner (DividendPayingToken.sol#112) is not in mixedCase
Parameter DividendPayingToken.accumulativeDividendOf(address)._owner (DividendPayingToken.sol#122) is not in mixedCase
Variable DividendPayingToken.BNBKing (DividendPayingToken.sol#24) is not in mixedCase
Constant DividendPayingToken.magnitude (DividendPayingToken.sol#30) 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
Variable BNBK.BNBKing (bnbking.sol#32) is not in mixedCase
Variable BNBK._isBlacklisted (bnbking.sol#36) is not in mixedCase
Variable BNBK.BNBKingRewardsFee (bnbking.sol#38) is not in mixedCase
Variable BNBK._maxWalletToken (bnbking.sol#42) is not in mixedCase
Variable BNBK._marketingWalletAddress (bnbking.sol#44) is not in mixedCase
Parameter BNBKingDividendTracker.getAccount(address)._account (bnbking.sol#537) is not in mixedCase
Variable BNBKingDividendTracker._maxWalletToken (bnbking.sol#490) 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
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)
Variable DividendPayingToken._withdrawDividendOfUser(address)._withdrawableDividend (DividendPayingToken.sol#77) is too similar to BNBKingDividendTracker.getAccount(address).withdrawableDividends (bnbking.sol#542)
Prevent variables from having similar names.
Additional information: link
BNBK.constructor() (bnbking.sol#92-125) uses literals with too many digits:
- _mint(owner(),1000000000000000 * (10 ** 18)) (bnbking.sol#124)
BNBK.updateGasForProcessing(uint256) (bnbking.sol#220-225) uses literals with too many digits:
- require(bool,string)(newValue >= 200000 && newValue <= 500000,BNBKing: gasForProcessing must be between 200,000 and 500,000) (bnbking.sol#221)
BNBK.slitherConstructorVariables() (bnbking.sol#20-474) uses literals with too many digits:
- deadWallet = 0x000000000000000000000000000000000000dEaD (bnbking.sol#30)
BNBK.slitherConstructorVariables() (bnbking.sol#20-474) uses literals with too many digits:
- swapTokensAtAmount = 50000000000 * (10 ** 18) (bnbking.sol#34)
BNBK.slitherConstructorVariables() (bnbking.sol#20-474) uses literals with too many digits:
- _maxWalletToken = 100000000000000 * (10 ** 18) (bnbking.sol#42)
BNBK.slitherConstructorVariables() (bnbking.sol#20-474) uses literals with too many digits:
- gasForProcessing = 300000 (bnbking.sol#49)
BNBKingDividendTracker.constructor() (bnbking.sol#497-500) uses literals with too many digits:
- minimumTokenBalanceForDividends = 200000 * (10 ** 18) (bnbking.sol#499)
BNBKingDividendTracker.getAccountAtIndex(uint256) (bnbking.sol#582-599) uses literals with too many digits:
- (0x0000000000000000000000000000000000000000,- 1,- 1,0,0,0,0,0) (bnbking.sol#593)
BNBKingDividendTracker.slitherConstructorVariables() (bnbking.sol#476-685) uses literals with too many digits:
- _maxWalletToken = 1000000000000000 * (10 ** 18) (bnbking.sol#490)
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
BNBK.deadWallet (bnbking.sol#30) should be constant
BNBK.liquidityWallet (bnbking.sol#45) should be constant
BNBK.swapTokensAtAmount (bnbking.sol#34) should be constant
BNBKingDividendTracker._maxWalletToken (bnbking.sol#490) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
distributeBNBKingDividends(uint256) should be declared external:
- DividendPayingToken.distributeBNBKingDividends(uint256) (DividendPayingToken.sol#55-66)
withdrawDividend() should be declared external:
- BNBKingDividendTracker.withdrawDividend() (bnbking.sol#506-508)
- DividendPayingToken.withdrawDividend() (DividendPayingToken.sol#70-72)
dividendOf(address) should be declared external:
- DividendPayingToken.dividendOf(address) (DividendPayingToken.sol#98-100)
withdrawnDividendOf(address) should be declared external:
- DividendPayingToken.withdrawnDividendOf(address) (DividendPayingToken.sol#112-114)
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)
updateDividendTracker(address) should be declared external:
- BNBK.updateDividendTracker(address) (bnbking.sol#131-146)
updateUniswapV2Router(address) should be declared external:
- BNBK.updateUniswapV2Router(address) (bnbking.sol#148-155)
updateMaxWallet(uint256) should be declared external:
- BNBK.updateMaxWallet(uint256) (bnbking.sol#157-159)
excludeMultipleAccountsFromFees(address[],bool) should be declared external:
- BNBK.excludeMultipleAccountsFromFees(address[],bool) (bnbking.sol#168-174)
setAutomatedMarketMakerPair(address,bool) should be declared external:
- BNBK.setAutomatedMarketMakerPair(address,bool) (bnbking.sol#197-201)
updateGasForProcessing(uint256) should be declared external:
- BNBK.updateGasForProcessing(uint256) (bnbking.sol#220-225)
isExcludedFromFees(address) should be declared external:
- BNBK.isExcludedFromFees(address) (bnbking.sol#239-241)
withdrawableDividendOf(address) should be declared external:
- BNBK.withdrawableDividendOf(address) (bnbking.sol#243-245)
dividendTokenBalanceOf(address) should be declared external:
- BNBK.dividendTokenBalanceOf(address) (bnbking.sol#247-249)
getAccountAtIndex(uint256) should be declared external:
- BNBKingDividendTracker.getAccountAtIndex(uint256) (bnbking.sol#582-599)
process(uint256) should be declared external:
- BNBKingDividendTracker.process(uint256) (bnbking.sol#626-671)
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