BirdNestToken.addLiquidity(uint256,uint256) (#713-726) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in BirdNestToken._transfer(address,address,uint256) (#625-665):
External calls:
- swapAndLiquify(contractTokenBalance) (#649)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#649)
- recipient.transfer(amount) (#592)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#652)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#660)
- finalAmount = takeFee(sender,recipient,amount) (#654-655)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#733)
Apply the check-effects-interactions pattern.
Additional information: link
Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
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.
BirdNestToken.swapAndLiquify(uint256) (#674-691) performs a multiplication on the result of a division:
-tokensForLP = tAmount.div(_totalTaxIfBuying).mul(_liquidityFee).div(2) (#676)
Consider ordering multiplication before division.
Additional information: link
BirdNestToken.addLiquidity(uint256,uint256) (#713-726) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
Ensure that all the return values of the function calls are used.
Additional information: link
BirdNestToken.allowance(address,address).owner (#501) shadows:
- Ownable.owner() (#152-154) (function)
BirdNestToken._approve(address,address,uint256).owner (#524) shadows:
- Ownable.owner() (#152-154) (function)
Rename the local variables that shadow another component.
Additional information: link
BirdNestToken.setTaxes(uint256,uint256,uint256,uint256) (#540-548) should emit an event for:
- _liquidityFee = newLiquidityTax (#541)
- _buyBackFee = newBuyBackTax (#543)
- _totalTaxIfBuying = _liquidityFee.add(_marketingFee).add(_buyBackFee) (#546)
- _totalTaxIfSelling = _totalTaxIfBuying.add(_extraFeeOnSell) (#547)
BirdNestToken.setMaxTxAmount(uint256) (#550-552) should emit an event for:
- _maxTxAmount = maxTxAmount (#551)
BirdNestToken.setWalletLimit(uint256) (#562-564) should emit an event for:
- _walletMax = newLimit (#563)
BirdNestToken.setNumTokensBeforeSwap(uint256) (#566-568) should emit an event for:
- minimumTokensBeforeSwap = newLimit (#567)
Emit an event for critical parameter changes.
Additional information: link
BirdNestToken.setMarketingWalletAddress(address).newAddress (#570) lacks a zero-check on :
- marketingWalletAddress = address(newAddress) (#571)
BirdNestToken.setbuyBackWalletAddress(address).newAddress (#574) lacks a zero-check on :
- buyBackWalletAddress = address(newAddress) (#575)
Check that the address is not zero.
Additional information: link
Reentrancy in BirdNestToken.changeRouterVersion(address) (#595-609):
External calls:
- newPairAddress = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#603-604)
State variables written after the call(s):
- uniswapV2Pair = newPairAddress (#607)
- uniswapV2Router = _uniswapV2Router (#608)
Reentrancy in BirdNestToken.constructor() (#458-479):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#462-463)
State variables written after the call(s):
- _allowances[address(this)][address(uniswapV2Router)] = _totalSupply (#466)
- _balances[_msgSender()] = _totalSupply (#477)
- _totalTaxIfBuying = _liquidityFee.add(_marketingFee).add(_buyBackFee) (#471)
- _totalTaxIfSelling = _totalTaxIfBuying.add(_extraFeeOnSell) (#472)
- isExcludedFromFee[owner()] = true (#468)
- isExcludedFromFee[address(this)] = true (#469)
- isWalletLimitExempt[owner()] = true (#474)
- isWalletLimitExempt[address(uniswapV2Pair)] = true (#475)
- uniswapV2Router = _uniswapV2Router (#465)
Reentrancy in BirdNestToken.swapAndLiquify(uint256) (#674-691):
External calls:
- swapTokensForEth(tokensForSwap) (#679)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#690)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#688)
- recipient.transfer(amount) (#592)
- transferToAddressETH(buyBackWalletAddress,amountBNBBuyBack) (#689)
- recipient.transfer(amount) (#592)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#690)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#690)
- _allowances[owner][spender] = amount (#528)
Reentrancy in BirdNestToken.transferFrom(address,address,uint256) (#619-623):
External calls:
- _transfer(sender,recipient,amount) (#620)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
External calls sending eth:
- _transfer(sender,recipient,amount) (#620)
- recipient.transfer(amount) (#592)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#621)
- _allowances[owner][spender] = amount (#528)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in BirdNestToken._transfer(address,address,uint256) (#625-665):
External calls:
- swapAndLiquify(contractTokenBalance) (#649)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#649)
- recipient.transfer(amount) (#592)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#734)
- finalAmount = takeFee(sender,recipient,amount) (#654-655)
- Transfer(sender,recipient,finalAmount) (#662)
Reentrancy in BirdNestToken.constructor() (#458-479):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#462-463)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_totalSupply) (#478)
Reentrancy in BirdNestToken.swapAndLiquify(uint256) (#674-691):
External calls:
- swapTokensForEth(tokensForSwap) (#679)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#690)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#688)
- recipient.transfer(amount) (#592)
- transferToAddressETH(buyBackWalletAddress,amountBNBBuyBack) (#689)
- recipient.transfer(amount) (#592)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#690)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#529)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#690)
Reentrancy in BirdNestToken.swapTokensForEth(uint256) (#693-711):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#710)
Reentrancy in BirdNestToken.transferFrom(address,address,uint256) (#619-623):
External calls:
- _transfer(sender,recipient,amount) (#620)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
External calls sending eth:
- _transfer(sender,recipient,amount) (#620)
- recipient.transfer(amount) (#592)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#529)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#621)
Apply the check-effects-interactions pattern.
Additional information: link
Ownable.unlock() (#187-192) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp > _lockTime,Contract is locked until 7 days) (#189)
Avoid relying on block.timestamp.
Additional information: link
Address.isContract(address) (#83-92) uses assembly
- INLINE ASM (#90)
Address._functionCallWithValue(address,bytes,uint256,string) (#119-136) uses assembly
- INLINE ASM (#128-131)
Do not use evm assembly.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#119-136) is never used and should be removed
Address.functionCall(address,bytes) (#102-104) is never used and should be removed
Address.functionCall(address,bytes,string) (#106-108) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#110-112) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#114-117) is never used and should be removed
Address.isContract(address) (#83-92) is never used and should be removed
Address.sendValue(address,uint256) (#94-100) is never used and should be removed
Context._msgData() (#10-13) is never used and should be removed
SafeMath.mod(uint256,uint256) (#71-73) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#75-78) is never used and should be removed
Remove unused functions.
Additional information: link
Pragma version>=0.6.0<=0.8.7 (#1) is too complex
Deploy with any of the following Solidity versions: 0.5.16 - 0.5.17, 0.6.11 - 0.6.12, 0.7.5 - 0.7.6 Use a simple pragma version that allows any of these versions. Consider using the latest version of Solidity for testing.
Additional information: link
Low level call in Address.sendValue(address,uint256) (#94-100):
- (success) = recipient.call{value: amount}() (#98)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#119-136):
- (success,returndata) = target.call{value: weiValue}(data) (#122)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#226) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#227) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#243) is not in mixedCase
Function IUniswapV2Router01.WETH() (#262) is not in mixedCase
Parameter BirdNestToken.setSwapAndLiquifyEnabled(bool)._enabled (#578) is not in mixedCase
Variable BirdNestToken._balances (#408) is not in mixedCase
Variable BirdNestToken._liquidityFee (#415) is not in mixedCase
Variable BirdNestToken._marketingFee (#416) is not in mixedCase
Variable BirdNestToken._buyBackFee (#417) is not in mixedCase
Variable BirdNestToken._extraFeeOnSell (#418) is not in mixedCase
Variable BirdNestToken._totalTaxIfBuying (#419) is not in mixedCase
Variable BirdNestToken._totalTaxIfSelling (#420) is not in mixedCase
Variable BirdNestToken._maxTxAmount (#423) is not in mixedCase
Variable BirdNestToken._walletMax (#424) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#11)" inContext (#4-14)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in BirdNestToken._transfer(address,address,uint256) (#625-665):
External calls:
- swapAndLiquify(contractTokenBalance) (#649)
- recipient.transfer(amount) (#592)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#649)
- recipient.transfer(amount) (#592)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#652)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#660)
- finalAmount = takeFee(sender,recipient,amount) (#654-655)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#733)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#734)
- finalAmount = takeFee(sender,recipient,amount) (#654-655)
- Transfer(sender,recipient,finalAmount) (#662)
Reentrancy in BirdNestToken.swapAndLiquify(uint256) (#674-691):
External calls:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#688)
- recipient.transfer(amount) (#592)
- transferToAddressETH(buyBackWalletAddress,amountBNBBuyBack) (#689)
- recipient.transfer(amount) (#592)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#688)
- recipient.transfer(amount) (#592)
- transferToAddressETH(buyBackWalletAddress,amountBNBBuyBack) (#689)
- recipient.transfer(amount) (#592)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#690)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#690)
- _allowances[owner][spender] = amount (#528)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#529)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#690)
Reentrancy in BirdNestToken.transferFrom(address,address,uint256) (#619-623):
External calls:
- _transfer(sender,recipient,amount) (#620)
- recipient.transfer(amount) (#592)
External calls sending eth:
- _transfer(sender,recipient,amount) (#620)
- recipient.transfer(amount) (#592)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#621)
- _allowances[owner][spender] = amount (#528)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#529)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#621)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#267) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#268)
Prevent variables from having similar names.
Additional information: link
BirdNestToken.slitherConstructorVariables() (#395-740) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#406)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
BirdNestToken._decimals (#402) should be constant
BirdNestToken._name (#400) should be constant
BirdNestToken._symbol (#401) should be constant
BirdNestToken._totalSupply (#422) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#161-164)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#166-170)
getUnlockTime() should be declared external:
- Ownable.getUnlockTime() (#172-174)
getTime() should be declared external:
- Ownable.getTime() (#176-178)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#180-185)
unlock() should be declared external:
- Ownable.unlock() (#187-192)
name() should be declared external:
- BirdNestToken.name() (#481-483)
symbol() should be declared external:
- BirdNestToken.symbol() (#485-487)
decimals() should be declared external:
- BirdNestToken.decimals() (#489-491)
totalSupply() should be declared external:
- BirdNestToken.totalSupply() (#493-495)
allowance(address,address) should be declared external:
- BirdNestToken.allowance(address,address) (#501-503)
increaseAllowance(address,uint256) should be declared external:
- BirdNestToken.increaseAllowance(address,uint256) (#505-508)
decreaseAllowance(address,uint256) should be declared external:
- BirdNestToken.decreaseAllowance(address,uint256) (#510-513)
minimumTokensBeforeSwapAmount() should be declared external:
- BirdNestToken.minimumTokensBeforeSwapAmount() (#515-517)
approve(address,uint256) should be declared external:
- BirdNestToken.approve(address,uint256) (#519-522)
blacklistAddress(address,bool) should be declared external:
- BirdNestToken.blacklistAddress(address,bool) (#532-534)
setIsExcludedFromFee(address,bool) should be declared external:
- BirdNestToken.setIsExcludedFromFee(address,bool) (#536-538)
setSwapAndLiquifyEnabled(bool) should be declared external:
- BirdNestToken.setSwapAndLiquifyEnabled(bool) (#578-581)
setSwapAndLiquifyByLimitOnly(bool) should be declared external:
- BirdNestToken.setSwapAndLiquifyByLimitOnly(bool) (#583-585)
getCirculatingSupply() should be declared external:
- BirdNestToken.getCirculatingSupply() (#587-589)
changeRouterVersion(address) should be declared external:
- BirdNestToken.changeRouterVersion(address) (#595-609)
transfer(address,uint256) should be declared external:
- BirdNestToken.transfer(address,uint256) (#614-617)
transferFrom(address,address,uint256) should be declared external:
- BirdNestToken.transferFrom(address,address,uint256) (#619-623)
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 Twitter account
Unable to find Blog account (Reddit or Medium)
Unable to find Youtube account
Unable to find Discord account