FenDouDog.addLiquidity(uint256,uint256) (#741-754) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in FenDouDog._transfer(address,address,uint256) (#650-688):
External calls:
- swapAndLiquify(contractTokenBalance) (#672)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#730-736)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#672)
- recipient.transfer(amount) (#614)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#675)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#683)
- finalAmount = takeFee(sender,recipient,amount) (#677-678)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#768)
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.
FenDouDog.addLiquidity(uint256,uint256) (#741-754) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
Ensure that all the return values of the function calls are used.
Additional information: link
FenDouDog._approve(address,address,uint256).owner (#528) shadows:
- Ownable.owner() (#156-158) (function)
FenDouDog.allowance(address,address).owner (#505) shadows:
- Ownable.owner() (#156-158) (function)
Rename the local variables that shadow another component.
Additional information: link
FenDouDog.setNumTokensBeforeSwap(uint256) (#588-590) should emit an event for:
- minimumTokensBeforeSwap = newLimit (#589)
FenDouDog.setSelTaxes(uint256,uint256,uint256) (#556-562) should emit an event for:
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#561)
FenDouDog.setMaxTxAmount(uint256) (#572-574) should emit an event for:
- _maxTxAmount = maxTxAmount (#573)
FenDouDog.setBuyTaxes(uint256,uint256,uint256) (#548-554) should emit an event for:
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#553)
FenDouDog.setWalletLimit(uint256) (#584-586) should emit an event for:
- _walletMax = newLimit (#585)
FenDouDog.setDistributionSettings(uint256,uint256,uint256) (#564-570) should emit an event for:
- _liquidityShare = newLiquidityShare (#565)
- _teamShare = newTeamShare (#567)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#569)
Emit an event for critical parameter changes.
Additional information: link
FenDouDog.setMarketingWalletAddress(address).newAddress (#592) lacks a zero-check on :
- marketingWalletAddress = address(newAddress) (#593)
FenDouDog.setTeamWalletAddress(address).newAddress (#596) lacks a zero-check on :
- teamWalletAddress = address(newAddress) (#597)
Check that the address is not zero.
Additional information: link
Reentrancy in FenDouDog.changeRouterVersion(address) (#617-634):
External calls:
- newPairAddress = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#625-626)
State variables written after the call(s):
- isMarketPair[address(uniswapPair)] = true (#633)
- isWalletLimitExempt[address(uniswapPair)] = true (#632)
- uniswapPair = newPairAddress (#629)
- uniswapV2Router = _uniswapV2Router (#630)
Reentrancy in FenDouDog.swapAndLiquify(uint256) (#697-719):
External calls:
- swapTokensForEth(tokensForSwap) (#702)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#730-736)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#718)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#712)
- recipient.transfer(amount) (#614)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#715)
- recipient.transfer(amount) (#614)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#718)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#718)
- _allowances[owner][spender] = amount (#532)
Reentrancy in FenDouDog.transferFrom(address,address,uint256) (#644-648):
External calls:
- _transfer(sender,recipient,amount) (#645)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#730-736)
External calls sending eth:
- _transfer(sender,recipient,amount) (#645)
- recipient.transfer(amount) (#614)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#646)
- _allowances[owner][spender] = amount (#532)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in FenDouDog._transfer(address,address,uint256) (#650-688):
External calls:
- swapAndLiquify(contractTokenBalance) (#672)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#730-736)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#672)
- recipient.transfer(amount) (#614)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#769)
- finalAmount = takeFee(sender,recipient,amount) (#677-678)
- Transfer(sender,recipient,finalAmount) (#685)
Reentrancy in FenDouDog.swapAndLiquify(uint256) (#697-719):
External calls:
- swapTokensForEth(tokensForSwap) (#702)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#730-736)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#718)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#712)
- recipient.transfer(amount) (#614)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#715)
- recipient.transfer(amount) (#614)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#718)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#533)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#718)
Reentrancy in FenDouDog.swapTokensForEth(uint256) (#721-739):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#730-736)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#738)
Reentrancy in FenDouDog.transferFrom(address,address,uint256) (#644-648):
External calls:
- _transfer(sender,recipient,amount) (#645)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#730-736)
External calls sending eth:
- _transfer(sender,recipient,amount) (#645)
- recipient.transfer(amount) (#614)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#533)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#646)
Apply the check-effects-interactions pattern.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#123-140) uses assembly
- INLINE ASM (#132-135)
Address.isContract(address) (#87-96) uses assembly
- INLINE ASM (#94)
Do not use evm assembly.
Additional information: link
Address.sendValue(address,uint256) (#98-104) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#114-116) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#79-82) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#118-121) is never used and should be removed
Context._msgData() (#14-17) is never used and should be removed
SafeMath.mod(uint256,uint256) (#75-77) is never used and should be removed
Address._functionCallWithValue(address,bytes,uint256,string) (#123-140) is never used and should be removed
Address.isContract(address) (#87-96) is never used and should be removed
Address.functionCall(address,bytes,string) (#110-112) is never used and should be removed
Address.functionCall(address,bytes) (#106-108) is never used and should be removed
Remove unused functions.
Additional information: link
FenDouDog._totalSupply (#419) is set pre-construction with a non-constant function or state variable:
- 28888 * 10 ** _decimals
FenDouDog.minimumTokensBeforeSwap (#422) is set pre-construction with a non-constant function or state variable:
- 3 * 10 ** _decimals
FenDouDog._maxTxAmount (#420) is set pre-construction with a non-constant function or state variable:
- 3 * 10 ** _decimals
FenDouDog._walletMax (#421) is set pre-construction with a non-constant function or state variable:
- 3 * 10 ** _decimals
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
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#123-140):
- (success,returndata) = target.call{value: weiValue}(data) (#126)
Low level call in Address.sendValue(address,uint256) (#98-104):
- (success) = recipient.call{value: amount}() (#102)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Variable FenDouDog._balances (#395) is not in mixedCase
Variable FenDouDog._buyTeamFee (#405) is not in mixedCase
Variable FenDouDog._walletMax (#421) is not in mixedCase
Variable FenDouDog._sellLiquidityFee (#407) is not in mixedCase
Variable FenDouDog._marketingShare (#412) is not in mixedCase
Variable FenDouDog._totalDistributionShares (#417) is not in mixedCase
Variable FenDouDog._buyLiquidityFee (#403) is not in mixedCase
Parameter FenDouDog.setSwapAndLiquifyEnabled(bool)._enabled (#600) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#214) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#230) is not in mixedCase
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#213) is not in mixedCase
Variable FenDouDog._teamShare (#413) is not in mixedCase
Variable FenDouDog._liquidityShare (#411) is not in mixedCase
Variable FenDouDog._buyMarketingFee (#404) is not in mixedCase
Variable FenDouDog._totalTaxIfBuying (#415) is not in mixedCase
Variable FenDouDog._sellMarketingFee (#408) is not in mixedCase
Function IUniswapV2Router01.WETH() (#249) is not in mixedCase
Variable FenDouDog._sellTeamFee (#409) is not in mixedCase
Variable FenDouDog._totalTaxIfSelling (#416) is not in mixedCase
Variable FenDouDog._maxTxAmount (#420) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#15)" inContext (#8-18)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in FenDouDog.transferFrom(address,address,uint256) (#644-648):
External calls:
- _transfer(sender,recipient,amount) (#645)
- recipient.transfer(amount) (#614)
External calls sending eth:
- _transfer(sender,recipient,amount) (#645)
- recipient.transfer(amount) (#614)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#646)
- _allowances[owner][spender] = amount (#532)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#533)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#646)
Reentrancy in FenDouDog._transfer(address,address,uint256) (#650-688):
External calls:
- swapAndLiquify(contractTokenBalance) (#672)
- recipient.transfer(amount) (#614)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#672)
- recipient.transfer(amount) (#614)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#675)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#683)
- finalAmount = takeFee(sender,recipient,amount) (#677-678)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#768)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#769)
- finalAmount = takeFee(sender,recipient,amount) (#677-678)
- Transfer(sender,recipient,finalAmount) (#685)
Reentrancy in FenDouDog.swapAndLiquify(uint256) (#697-719):
External calls:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#712)
- recipient.transfer(amount) (#614)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#715)
- recipient.transfer(amount) (#614)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#712)
- recipient.transfer(amount) (#614)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#715)
- recipient.transfer(amount) (#614)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#718)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#746-753)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#718)
- _allowances[owner][spender] = amount (#532)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#533)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#718)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#254) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#255)
Prevent variables from having similar names.
Additional information: link
FenDouDog.slitherConstructorVariables() (#382-776) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#393)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
Ownable._lockTime (#146) is never used in FenDouDog (#382-776)
Ownable.asdasd (#145) is never used in FenDouDog (#382-776)
Remove unused state variables.
Additional information: link
FenDouDog._symbol (#388) should be constant
Ownable.asdasd (#145) should be constant
FenDouDog._name (#387) should be constant
Ownable._lockTime (#146) should be constant
FenDouDog._decimals (#389) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
setSwapAndLiquifyEnabled(bool) should be declared external:
- FenDouDog.setSwapAndLiquifyEnabled(bool) (#600-603)
setSwapAndLiquifyByLimitOnly(bool) should be declared external:
- FenDouDog.setSwapAndLiquifyByLimitOnly(bool) (#605-607)
setIsExcludedFromFee(address,bool) should be declared external:
- FenDouDog.setIsExcludedFromFee(address,bool) (#544-546)
setMarketPairStatus(address,bool) should be declared external:
- FenDouDog.setMarketPairStatus(address,bool) (#536-538)
minimumTokensBeforeSwapAmount() should be declared external:
- FenDouDog.minimumTokensBeforeSwapAmount() (#519-521)
waiveOwnership() should be declared external:
- Ownable.waiveOwnership() (#165-168)
transferFrom(address,address,uint256) should be declared external:
- FenDouDog.transferFrom(address,address,uint256) (#644-648)
transfer(address,uint256) should be declared external:
- FenDouDog.transfer(address,uint256) (#639-642)
changeRouterVersion(address) should be declared external:
- FenDouDog.changeRouterVersion(address) (#617-634)
getTime() should be declared external:
- Ownable.getTime() (#176-178)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#170-174)
increaseAllowance(address,uint256) should be declared external:
- FenDouDog.increaseAllowance(address,uint256) (#509-512)
allowance(address,address) should be declared external:
- FenDouDog.allowance(address,address) (#505-507)
name() should be declared external:
- FenDouDog.name() (#485-487)
symbol() should be declared external:
- FenDouDog.symbol() (#489-491)
decreaseAllowance(address,uint256) should be declared external:
- FenDouDog.decreaseAllowance(address,uint256) (#514-517)
approve(address,uint256) should be declared external:
- FenDouDog.approve(address,uint256) (#523-526)
getCirculatingSupply() should be declared external:
- FenDouDog.getCirculatingSupply() (#609-611)
decimals() should be declared external:
- FenDouDog.decimals() (#493-495)
totalSupply() should be declared external:
- FenDouDog.totalSupply() (#497-499)
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