Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
DDefault.addLiquidity(uint256,uint256) (#737-750) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in DDefault._transfer(address,address,uint256) (#646-684):
External calls:
- swapAndLiquify(contractTokenBalance) (#668)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#726-732)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#668)
- recipient.transfer(amount) (#610)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#671)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#679)
- finalAmount = takeFee(sender,recipient,amount) (#673-674)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#764)
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.
DDefault.addLiquidity(uint256,uint256) (#737-750) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
Ensure that all the return values of the function calls are used.
Additional information: link
DDefault.allowance(address,address).owner (#504) shadows:
- Ownable.owner() (#161-163) (function)
DDefault._approve(address,address,uint256).owner (#523) shadows:
- Ownable.owner() (#161-163) (function)
Rename the local variables that shadow another component.
Additional information: link
DDefault.setBuyTaxes(uint256,uint256,uint256) (#543-549) should emit an event for:
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyBuyBackFee) (#548)
DDefault.setSellTaxes(uint256,uint256,uint256) (#551-557) should emit an event for:
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellBuyBackFee) (#556)
DDefault.setDistributionSettings(uint256,uint256,uint256) (#559-565) should emit an event for:
- _liquidityShare = newLiquidityShare (#560)
- _BuyBackShare = newBuyBackShare (#562)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_BuyBackShare) (#564)
DDefault.setMaxTxAmount(uint256) (#567-570) should emit an event for:
- _maxTxAmount = maxTxAmount (#569)
DDefault.setWalletLimit(uint256) (#580-582) should emit an event for:
- _walletMax = newLimit (#581)
DDefault.setNumTokensBeforeSwap(uint256) (#584-586) should emit an event for:
- minimumTokensBeforeSwap = newLimit (#585)
Emit an event for critical parameter changes.
Additional information: link
DDefault.setMarketingWalletAddress(address).newAddress (#588) lacks a zero-check on :
- marketingWalletAddress = address(newAddress) (#589)
DDefault.setBuyBackWalletAddress(address).newAddress (#592) lacks a zero-check on :
- BuyBackWalletAddress = address(newAddress) (#593)
Check that the address is not zero.
Additional information: link
Reentrancy in DDefault.changeRouterVersion(address) (#613-630):
External calls:
- newPairAddress = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#621-622)
State variables written after the call(s):
- isMarketPair[address(uniswapPair)] = true (#629)
- isWalletLimitExempt[address(uniswapPair)] = true (#628)
- uniswapPair = newPairAddress (#625)
- uniswapV2Router = _uniswapV2Router (#626)
Reentrancy in DDefault.constructor() (#454-482):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#458-459)
State variables written after the call(s):
- _allowances[address(this)][address(uniswapV2Router)] = _totalSupply (#462)
- _balances[_msgSender()] = _totalSupply (#480)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_BuyBackShare) (#469)
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyBuyBackFee) (#467)
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellBuyBackFee) (#468)
- isExcludedFromFee[owner()] = true (#464)
- isExcludedFromFee[address(this)] = true (#465)
- isMarketPair[address(uniswapPair)] = true (#478)
- isTxLimitExempt[owner()] = true (#475)
- isTxLimitExempt[address(this)] = true (#476)
- isWalletLimitExempt[owner()] = true (#471)
- isWalletLimitExempt[address(uniswapPair)] = true (#472)
- isWalletLimitExempt[address(this)] = true (#473)
- uniswapV2Router = _uniswapV2Router (#461)
Reentrancy in DDefault.swapAndLiquify(uint256) (#693-715):
External calls:
- swapTokensForEth(tokensForSwap) (#698)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#726-732)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#714)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#708)
- recipient.transfer(amount) (#610)
- transferToAddressETH(BuyBackWalletAddress,amountBNBBuyBack) (#711)
- recipient.transfer(amount) (#610)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#714)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#714)
- _allowances[owner][spender] = amount (#527)
Reentrancy in DDefault.transferFrom(address,address,uint256) (#640-644):
External calls:
- _transfer(sender,recipient,amount) (#641)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#726-732)
External calls sending eth:
- _transfer(sender,recipient,amount) (#641)
- recipient.transfer(amount) (#610)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#642)
- _allowances[owner][spender] = amount (#527)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in DDefault._transfer(address,address,uint256) (#646-684):
External calls:
- swapAndLiquify(contractTokenBalance) (#668)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#726-732)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#668)
- recipient.transfer(amount) (#610)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#765)
- finalAmount = takeFee(sender,recipient,amount) (#673-674)
- Transfer(sender,recipient,finalAmount) (#681)
Reentrancy in DDefault.constructor() (#454-482):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#458-459)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_totalSupply) (#481)
Reentrancy in DDefault.swapAndLiquify(uint256) (#693-715):
External calls:
- swapTokensForEth(tokensForSwap) (#698)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#726-732)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#714)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#708)
- recipient.transfer(amount) (#610)
- transferToAddressETH(BuyBackWalletAddress,amountBNBBuyBack) (#711)
- recipient.transfer(amount) (#610)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#714)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#528)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#714)
Reentrancy in DDefault.swapTokensForEth(uint256) (#717-735):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#726-732)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#734)
Reentrancy in DDefault.transferFrom(address,address,uint256) (#640-644):
External calls:
- _transfer(sender,recipient,amount) (#641)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#726-732)
External calls sending eth:
- _transfer(sender,recipient,amount) (#641)
- recipient.transfer(amount) (#610)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#528)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#642)
Apply the check-effects-interactions pattern.
Additional information: link
Address.isContract(address) (#93-102) uses assembly
- INLINE ASM (#100)
Address._functionCallWithValue(address,bytes,uint256,string) (#129-146) uses assembly
- INLINE ASM (#138-141)
Do not use evm assembly.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#129-146) is never used and should be removed
Address.functionCall(address,bytes) (#112-114) is never used and should be removed
Address.functionCall(address,bytes,string) (#116-118) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#120-122) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#124-127) is never used and should be removed
Address.isContract(address) (#93-102) is never used and should be removed
Address.sendValue(address,uint256) (#104-110) is never used and should be removed
Context._msgData() (#20-23) is never used and should be removed
SafeMath.mod(uint256,uint256) (#81-83) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#85-88) is never used and should be removed
Remove unused functions.
Additional information: link
Low level call in Address.sendValue(address,uint256) (#104-110):
- (success) = recipient.call{value: amount}() (#108)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#129-146):
- (success,returndata) = target.call{value: weiValue}(data) (#132)
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() (#213) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#214) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#230) is not in mixedCase
Function IUniswapV2Router01.WETH() (#249) is not in mixedCase
Parameter DDefault.setSwapAndLiquifyEnabled(bool)._enabled (#596) is not in mixedCase
Variable DDefault.BuyBackWalletAddress (#392) is not in mixedCase
Variable DDefault._balances (#395) is not in mixedCase
Variable DDefault._buyLiquidityFee (#403) is not in mixedCase
Variable DDefault._buyMarketingFee (#404) is not in mixedCase
Variable DDefault._buyBuyBackFee (#405) is not in mixedCase
Variable DDefault._sellLiquidityFee (#406) is not in mixedCase
Variable DDefault._sellMarketingFee (#407) is not in mixedCase
Variable DDefault._sellBuyBackFee (#408) is not in mixedCase
Variable DDefault._liquidityShare (#410) is not in mixedCase
Variable DDefault._marketingShare (#411) is not in mixedCase
Variable DDefault._BuyBackShare (#412) is not in mixedCase
Variable DDefault._totalTaxIfBuying (#414) is not in mixedCase
Variable DDefault._totalTaxIfSelling (#415) is not in mixedCase
Variable DDefault._totalDistributionShares (#416) is not in mixedCase
Variable DDefault._maxTxAmount (#419) is not in mixedCase
Variable DDefault._walletMax (#420) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#21)" inContext (#14-24)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in DDefault._transfer(address,address,uint256) (#646-684):
External calls:
- swapAndLiquify(contractTokenBalance) (#668)
- recipient.transfer(amount) (#610)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#668)
- recipient.transfer(amount) (#610)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#671)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#679)
- finalAmount = takeFee(sender,recipient,amount) (#673-674)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#764)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#765)
- finalAmount = takeFee(sender,recipient,amount) (#673-674)
- Transfer(sender,recipient,finalAmount) (#681)
Reentrancy in DDefault.swapAndLiquify(uint256) (#693-715):
External calls:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#708)
- recipient.transfer(amount) (#610)
- transferToAddressETH(BuyBackWalletAddress,amountBNBBuyBack) (#711)
- recipient.transfer(amount) (#610)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#708)
- recipient.transfer(amount) (#610)
- transferToAddressETH(BuyBackWalletAddress,amountBNBBuyBack) (#711)
- recipient.transfer(amount) (#610)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#714)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#714)
- _allowances[owner][spender] = amount (#527)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#528)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#714)
Reentrancy in DDefault.transferFrom(address,address,uint256) (#640-644):
External calls:
- _transfer(sender,recipient,amount) (#641)
- recipient.transfer(amount) (#610)
External calls sending eth:
- _transfer(sender,recipient,amount) (#641)
- recipient.transfer(amount) (#610)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#742-749)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#642)
- _allowances[owner][spender] = amount (#527)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#528)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#642)
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
DDefault.slitherConstructorVariables() (#382-772) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#393)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
Ownable._previousOwner (#151) is never used in DDefault (#382-772)
Remove unused state variables.
Additional information: link
DDefault._decimals (#389) should be constant
DDefault._name (#387) should be constant
DDefault._symbol (#388) should be constant
DDefault._totalSupply (#418) should be constant
Ownable._previousOwner (#151) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
waiveOwnership() should be declared external:
- Ownable.waiveOwnership() (#170-173)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#175-179)
name() should be declared external:
- DDefault.name() (#484-486)
symbol() should be declared external:
- DDefault.symbol() (#488-490)
decimals() should be declared external:
- DDefault.decimals() (#492-494)
totalSupply() should be declared external:
- DDefault.totalSupply() (#496-498)
allowance(address,address) should be declared external:
- DDefault.allowance(address,address) (#504-506)
increaseAllowance(address,uint256) should be declared external:
- DDefault.increaseAllowance(address,uint256) (#508-511)
decreaseAllowance(address,uint256) should be declared external:
- DDefault.decreaseAllowance(address,uint256) (#513-516)
approve(address,uint256) should be declared external:
- DDefault.approve(address,uint256) (#518-521)
addMarketPair(address) should be declared external:
- DDefault.addMarketPair(address) (#531-533)
setIsExcludedFromFee(address,bool) should be declared external:
- DDefault.setIsExcludedFromFee(address,bool) (#539-541)
setSwapAndLiquifyEnabled(bool) should be declared external:
- DDefault.setSwapAndLiquifyEnabled(bool) (#596-599)
setSwapAndLiquifyByLimitOnly(bool) should be declared external:
- DDefault.setSwapAndLiquifyByLimitOnly(bool) (#601-603)
getCirculatingSupply() should be declared external:
- DDefault.getCirculatingSupply() (#605-607)
changeRouterVersion(address) should be declared external:
- DDefault.changeRouterVersion(address) (#613-630)
transfer(address,uint256) should be declared external:
- DDefault.transfer(address,uint256) (#635-638)
transferFrom(address,address,uint256) should be declared external:
- DDefault.transferFrom(address,address,uint256) (#640-644)
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