Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
Olympicicedance.addLiquidity(uint256,uint256) (#770-783) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in Olympicicedance._transfer(address,address,uint256) (#668-717):
External calls:
- swapAndLiquify(contractTokenBalance) (#701)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#759-765)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#701)
- recipient.transfer(amount) (#632)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#704)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#712)
- finalAmount = takeFee(sender,recipient,amount) (#706-707)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#797)
Apply the check-effects-interactions pattern.
Additional information: link
Combination 1: Reentrancy vulnerabilities + Functions that send Ether to arbitraty destination. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.
Not a direct threat, but may indicate unreliable intentions of developer. Widespread names (e.g. Elon, King, Moon, Doge) are common among meme-tokens and scams. The allow to gain free hype and attract unexperienced investors.
Olympicicedance.addLiquidity(uint256,uint256) (#770-783) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
Ensure that all the return values of the function calls are used.
Additional information: link
Olympicicedance.allowance(address,address).owner (#523) shadows:
- Ownable.owner() (#152-154) (function)
Olympicicedance._approve(address,address,uint256).owner (#546) shadows:
- Ownable.owner() (#152-154) (function)
Rename the local variables that shadow another component.
Additional information: link
Olympicicedance.setBuyTaxes(uint256,uint256,uint256) (#566-572) should emit an event for:
- _buyLiquidityFee = newLiquidityTax (#567)
- _buyMarketingFee = newMarketingTax (#568)
- _buyTeamFee = newTeamTax (#569)
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#571)
Olympicicedance.setSellTaxes(uint256,uint256,uint256) (#574-580) should emit an event for:
- _sellLiquidityFee = newLiquidityTax (#575)
- _sellMarketingFee = newMarketingTax (#576)
- _sellTeamFee = newTeamTax (#577)
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#579)
Olympicicedance.setDistributionSettings(uint256,uint256,uint256) (#582-588) should emit an event for:
- _liquidityShare = newLiquidityShare (#583)
- _teamShare = newTeamShare (#585)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#587)
Olympicicedance.setMaxTxAmount(uint256) (#590-592) should emit an event for:
- _maxTxAmount = maxTxAmount (#591)
Olympicicedance.setWalletLimit(uint256) (#602-604) should emit an event for:
- _walletMax = newLimit (#603)
Olympicicedance.setNumTokensBeforeSwap(uint256) (#606-608) should emit an event for:
- minimumTokensBeforeSwap = newLimit (#607)
Olympicicedance.UPfeeTXtime(uint256) (#812-814) should emit an event for:
- feeTXtime = _feeTXtime (#813)
Olympicicedance.UPendtime(uint256) (#816-818) should emit an event for:
- endtime = _endtime (#817)
Emit an event for critical parameter changes.
Additional information: link
Olympicicedance.setMarketingWalletAddress(address).newAddress (#610) lacks a zero-check on :
- marketingWalletAddress = address(newAddress) (#611)
Olympicicedance.setBuybackWalletAddress(address).newAddress (#614) lacks a zero-check on :
- BuybackWalletAddress = address(newAddress) (#615)
Check that the address is not zero.
Additional information: link
Reentrancy in Olympicicedance.changeRouterVersion(address) (#635-652):
External calls:
- newPairAddress = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#643-644)
State variables written after the call(s):
- isMarketPair[address(uniswapPair)] = true (#651)
- isWalletLimitExempt[address(uniswapPair)] = true (#650)
- uniswapPair = newPairAddress (#647)
- uniswapV2Router = _uniswapV2Router (#648)
Reentrancy in Olympicicedance.constructor() (#473-501):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#477-478)
State variables written after the call(s):
- _allowances[address(this)][address(uniswapV2Router)] = _totalSupply (#481)
- _balances[_msgSender()] = _totalSupply (#499)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#488)
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#486)
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#487)
- isExcludedFromFee[owner()] = true (#483)
- isExcludedFromFee[address(this)] = true (#484)
- isMarketPair[address(uniswapPair)] = true (#497)
- isTxLimitExempt[owner()] = true (#494)
- isTxLimitExempt[address(this)] = true (#495)
- isWalletLimitExempt[owner()] = true (#490)
- isWalletLimitExempt[address(uniswapPair)] = true (#491)
- isWalletLimitExempt[address(this)] = true (#492)
- uniswapV2Router = _uniswapV2Router (#480)
Reentrancy in Olympicicedance.swapAndLiquify(uint256) (#726-748):
External calls:
- swapTokensForEth(tokensForSwap) (#731)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#759-765)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#747)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#741)
- recipient.transfer(amount) (#632)
- transferToAddressETH(BuybackWalletAddress,amountBNBTeam) (#744)
- recipient.transfer(amount) (#632)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#747)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#747)
- _allowances[owner][spender] = amount (#550)
Reentrancy in Olympicicedance.transferFrom(address,address,uint256) (#662-666):
External calls:
- _transfer(sender,recipient,amount) (#663)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#759-765)
External calls sending eth:
- _transfer(sender,recipient,amount) (#663)
- recipient.transfer(amount) (#632)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#664)
- _allowances[owner][spender] = amount (#550)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in Olympicicedance._transfer(address,address,uint256) (#668-717):
External calls:
- swapAndLiquify(contractTokenBalance) (#701)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#759-765)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#701)
- recipient.transfer(amount) (#632)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#798)
- finalAmount = takeFee(sender,recipient,amount) (#706-707)
- Transfer(sender,recipient,finalAmount) (#714)
Reentrancy in Olympicicedance.constructor() (#473-501):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#477-478)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_totalSupply) (#500)
Reentrancy in Olympicicedance.swapAndLiquify(uint256) (#726-748):
External calls:
- swapTokensForEth(tokensForSwap) (#731)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#759-765)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#747)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#741)
- recipient.transfer(amount) (#632)
- transferToAddressETH(BuybackWalletAddress,amountBNBTeam) (#744)
- recipient.transfer(amount) (#632)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#747)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#551)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#747)
Reentrancy in Olympicicedance.swapTokensForEth(uint256) (#750-768):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#759-765)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#767)
Reentrancy in Olympicicedance.transferFrom(address,address,uint256) (#662-666):
External calls:
- _transfer(sender,recipient,amount) (#663)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#759-765)
External calls sending eth:
- _transfer(sender,recipient,amount) (#663)
- recipient.transfer(amount) (#632)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#551)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#664)
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)
Olympicicedance._transfer(address,address,uint256) (#668-717) uses timestamp for comparisons
Dangerous comparisons:
- endtime == 0 (#673)
- endtime > block.timestamp (#676)
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
Olympicicedance._liquidityShare (#427) is set pre-construction with a non-constant function or state variable:
- _buyLiquidityFee.add(_sellLiquidityFee)
Olympicicedance._marketingShare (#428) is set pre-construction with a non-constant function or state variable:
- _buyMarketingFee.add(_sellMarketingFee)
Olympicicedance._teamShare (#429) is set pre-construction with a non-constant function or state variable:
- _buyTeamFee.add(_sellTeamFee)
Olympicicedance._totalSupply (#435) is set pre-construction with a non-constant function or state variable:
- 10000 * 10 ** 4 * 10 ** _decimals
Olympicicedance._maxTxAmount (#436) is set pre-construction with a non-constant function or state variable:
- _totalSupply.div(10)
Olympicicedance._walletMax (#437) is set pre-construction with a non-constant function or state variable:
- _totalSupply.div(10)
Olympicicedance.minimumTokensBeforeSwap (#438) is set pre-construction with a non-constant function or state variable:
- _totalSupply.div(10000)
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.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 Olympicicedance.setSwapAndLiquifyEnabled(bool)._enabled (#618) is not in mixedCase
Parameter Olympicicedance.setblocklist(address)._account (#804) is not in mixedCase
Function Olympicicedance.UPfeeTXtime(uint256) (#812-814) is not in mixedCase
Parameter Olympicicedance.UPfeeTXtime(uint256)._feeTXtime (#812) is not in mixedCase
Function Olympicicedance.UPendtime(uint256) (#816-818) is not in mixedCase
Parameter Olympicicedance.UPendtime(uint256)._endtime (#816) is not in mixedCase
Variable Olympicicedance.BuybackWalletAddress (#405) is not in mixedCase
Variable Olympicicedance._balances (#408) is not in mixedCase
Variable Olympicicedance._buyLiquidityFee (#419) is not in mixedCase
Variable Olympicicedance._buyMarketingFee (#420) is not in mixedCase
Variable Olympicicedance._buyTeamFee (#421) is not in mixedCase
Variable Olympicicedance._sellLiquidityFee (#423) is not in mixedCase
Variable Olympicicedance._sellMarketingFee (#424) is not in mixedCase
Variable Olympicicedance._sellTeamFee (#425) is not in mixedCase
Variable Olympicicedance._liquidityShare (#427) is not in mixedCase
Variable Olympicicedance._marketingShare (#428) is not in mixedCase
Variable Olympicicedance._teamShare (#429) is not in mixedCase
Variable Olympicicedance._totalTaxIfBuying (#431) is not in mixedCase
Variable Olympicicedance._totalTaxIfSelling (#432) is not in mixedCase
Variable Olympicicedance._totalDistributionShares (#433) is not in mixedCase
Variable Olympicicedance._maxTxAmount (#436) is not in mixedCase
Variable Olympicicedance._walletMax (#437) 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 Olympicicedance._transfer(address,address,uint256) (#668-717):
External calls:
- swapAndLiquify(contractTokenBalance) (#701)
- recipient.transfer(amount) (#632)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#701)
- recipient.transfer(amount) (#632)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#704)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#712)
- finalAmount = takeFee(sender,recipient,amount) (#706-707)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#797)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#798)
- finalAmount = takeFee(sender,recipient,amount) (#706-707)
- Transfer(sender,recipient,finalAmount) (#714)
Reentrancy in Olympicicedance.swapAndLiquify(uint256) (#726-748):
External calls:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#741)
- recipient.transfer(amount) (#632)
- transferToAddressETH(BuybackWalletAddress,amountBNBTeam) (#744)
- recipient.transfer(amount) (#632)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#741)
- recipient.transfer(amount) (#632)
- transferToAddressETH(BuybackWalletAddress,amountBNBTeam) (#744)
- recipient.transfer(amount) (#632)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#747)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#747)
- _allowances[owner][spender] = amount (#550)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#551)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#747)
Reentrancy in Olympicicedance.transferFrom(address,address,uint256) (#662-666):
External calls:
- _transfer(sender,recipient,amount) (#663)
- recipient.transfer(amount) (#632)
External calls sending eth:
- _transfer(sender,recipient,amount) (#663)
- recipient.transfer(amount) (#632)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#775-782)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#664)
- _allowances[owner][spender] = amount (#550)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#551)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#664)
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
Olympicicedance.slitherConstructorVariables() (#395-820) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#406)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
Olympicicedance._decimals (#402) should be constant
Olympicicedance._name (#400) should be constant
Olympicicedance._symbol (#401) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
waiveOwnership() should be declared external:
- Ownable.waiveOwnership() (#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:
- Olympicicedance.name() (#503-505)
symbol() should be declared external:
- Olympicicedance.symbol() (#507-509)
decimals() should be declared external:
- Olympicicedance.decimals() (#511-513)
totalSupply() should be declared external:
- Olympicicedance.totalSupply() (#515-517)
allowance(address,address) should be declared external:
- Olympicicedance.allowance(address,address) (#523-525)
increaseAllowance(address,uint256) should be declared external:
- Olympicicedance.increaseAllowance(address,uint256) (#527-530)
decreaseAllowance(address,uint256) should be declared external:
- Olympicicedance.decreaseAllowance(address,uint256) (#532-535)
minimumTokensBeforeSwapAmount() should be declared external:
- Olympicicedance.minimumTokensBeforeSwapAmount() (#537-539)
approve(address,uint256) should be declared external:
- Olympicicedance.approve(address,uint256) (#541-544)
setMarketPairStatus(address,bool) should be declared external:
- Olympicicedance.setMarketPairStatus(address,bool) (#554-556)
setIsExcludedFromFee(address,bool) should be declared external:
- Olympicicedance.setIsExcludedFromFee(address,bool) (#562-564)
setSwapAndLiquifyEnabled(bool) should be declared external:
- Olympicicedance.setSwapAndLiquifyEnabled(bool) (#618-621)
setSwapAndLiquifyByLimitOnly(bool) should be declared external:
- Olympicicedance.setSwapAndLiquifyByLimitOnly(bool) (#623-625)
getCirculatingSupply() should be declared external:
- Olympicicedance.getCirculatingSupply() (#627-629)
changeRouterVersion(address) should be declared external:
- Olympicicedance.changeRouterVersion(address) (#635-652)
transfer(address,uint256) should be declared external:
- Olympicicedance.transfer(address,uint256) (#657-660)
transferFrom(address,address,uint256) should be declared external:
- Olympicicedance.transferFrom(address,address,uint256) (#662-666)
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