Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
WINU.addLiquidity(uint256,uint256) (#783-796) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in WINU._transfer(address,address,uint256) (#692-730):
External calls:
- swapAndLiquify(contractTokenBalance) (#714)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#772-778)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#714)
- recipient.transfer(amount) (#656)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#717)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#725)
- finalAmount = takeFee(sender,recipient,amount) (#719-720)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#810)
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.
WINU.addLiquidity(uint256,uint256) (#783-796) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
Ensure that all the return values of the function calls are used.
Additional information: link
WINU.allowance(address,address).owner (#547) shadows:
- Ownable.owner() (#181-183) (function)
WINU._approve(address,address,uint256).owner (#570) shadows:
- Ownable.owner() (#181-183) (function)
Rename the local variables that shadow another component.
Additional information: link
WINU.setBuyTaxes(uint256,uint256,uint256) (#590-596) should emit an event for:
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#595)
WINU.setSellTaxes(uint256,uint256,uint256) (#598-604) should emit an event for:
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#603)
WINU.setDistributionSettings(uint256,uint256,uint256) (#606-612) should emit an event for:
- _liquidityShare = newLiquidityShare (#607)
- _teamShare = newTeamShare (#609)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#611)
WINU.setMaxTxAmount(uint256) (#614-616) should emit an event for:
- _maxTxAmount = maxTxAmount (#615)
WINU.setWalletLimit(uint256) (#626-628) should emit an event for:
- _walletMax = newLimit (#627)
WINU.setNumTokensBeforeSwap(uint256) (#630-632) should emit an event for:
- minimumTokensBeforeSwap = newLimit (#631)
Emit an event for critical parameter changes.
Additional information: link
WINU.setMarketingWalletAddress(address).newAddress (#634) lacks a zero-check on :
- marketingWalletAddress = address(newAddress) (#635)
WINU.setTeamWalletAddress(address).newAddress (#638) lacks a zero-check on :
- teamWalletAddress = address(newAddress) (#639)
Check that the address is not zero.
Additional information: link
Reentrancy in WINU.changeRouterVersion(address) (#659-676):
External calls:
- newPairAddress = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#667-668)
State variables written after the call(s):
- isMarketPair[address(uniswapPair)] = true (#675)
- isWalletLimitExempt[address(uniswapPair)] = true (#674)
- uniswapPair = newPairAddress (#671)
- uniswapV2Router = _uniswapV2Router (#672)
Reentrancy in WINU.constructor() (#497-525):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#501-502)
State variables written after the call(s):
- _allowances[address(this)][address(uniswapV2Router)] = _totalSupply (#505)
- _balances[_msgSender()] = _totalSupply (#523)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#512)
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#510)
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#511)
- isExcludedFromFee[owner()] = true (#507)
- isExcludedFromFee[address(this)] = true (#508)
- isMarketPair[address(uniswapPair)] = true (#521)
- isTxLimitExempt[owner()] = true (#518)
- isTxLimitExempt[address(this)] = true (#519)
- isWalletLimitExempt[owner()] = true (#514)
- isWalletLimitExempt[address(uniswapPair)] = true (#515)
- isWalletLimitExempt[address(this)] = true (#516)
- uniswapV2Router = _uniswapV2Router (#504)
Reentrancy in WINU.swapAndLiquify(uint256) (#739-761):
External calls:
- swapTokensForEth(tokensForSwap) (#744)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#772-778)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#760)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#754)
- recipient.transfer(amount) (#656)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#757)
- recipient.transfer(amount) (#656)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#760)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#760)
- _allowances[owner][spender] = amount (#574)
Reentrancy in WINU.transferFrom(address,address,uint256) (#686-690):
External calls:
- _transfer(sender,recipient,amount) (#687)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#772-778)
External calls sending eth:
- _transfer(sender,recipient,amount) (#687)
- recipient.transfer(amount) (#656)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#688)
- _allowances[owner][spender] = amount (#574)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in WINU._transfer(address,address,uint256) (#692-730):
External calls:
- swapAndLiquify(contractTokenBalance) (#714)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#772-778)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#714)
- recipient.transfer(amount) (#656)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#811)
- finalAmount = takeFee(sender,recipient,amount) (#719-720)
- Transfer(sender,recipient,finalAmount) (#727)
Reentrancy in WINU.constructor() (#497-525):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#501-502)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_totalSupply) (#524)
Reentrancy in WINU.swapAndLiquify(uint256) (#739-761):
External calls:
- swapTokensForEth(tokensForSwap) (#744)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#772-778)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#760)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#754)
- recipient.transfer(amount) (#656)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#757)
- recipient.transfer(amount) (#656)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#760)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#575)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#760)
Reentrancy in WINU.swapTokensForEth(uint256) (#763-781):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#772-778)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#780)
Reentrancy in WINU.transferFrom(address,address,uint256) (#686-690):
External calls:
- _transfer(sender,recipient,amount) (#687)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#772-778)
External calls sending eth:
- _transfer(sender,recipient,amount) (#687)
- recipient.transfer(amount) (#656)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#575)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#688)
Apply the check-effects-interactions pattern.
Additional information: link
Ownable.unlock() (#216-221) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp > _lockTime,Contract is locked until 7 days) (#218)
Avoid relying on block.timestamp.
Additional information: link
Address.isContract(address) (#112-121) uses assembly
- INLINE ASM (#119)
Address._functionCallWithValue(address,bytes,uint256,string) (#148-165) uses assembly
- INLINE ASM (#157-160)
Do not use evm assembly.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#148-165) is never used and should be removed
Address.functionCall(address,bytes) (#131-133) is never used and should be removed
Address.functionCall(address,bytes,string) (#135-137) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#139-141) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#143-146) is never used and should be removed
Address.isContract(address) (#112-121) is never used and should be removed
Address.sendValue(address,uint256) (#123-129) is never used and should be removed
Context._msgData() (#39-42) is never used and should be removed
SafeMath.mod(uint256,uint256) (#100-102) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#104-107) is never used and should be removed
Remove unused functions.
Additional information: link
WINU._totalSupply (#461) is set pre-construction with a non-constant function or state variable:
- 83838383838383 * 10 ** 0 * 10 ** 0 * 10 ** _decimals
WINU._maxTxAmount (#462) is set pre-construction with a non-constant function or state variable:
- 83838383838383 * 10 ** 0 * 10 ** 0 * 10 ** _decimals
WINU._walletMax (#463) is set pre-construction with a non-constant function or state variable:
- 83838383838383 * 10 ** 0 * 10 ** 0 * 10 ** _decimals
WINU.minimumTokensBeforeSwap (#464) is set pre-construction with a non-constant function or state variable:
- 83838383838383 * 10 ** 0 * 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.sendValue(address,uint256) (#123-129):
- (success) = recipient.call{value: amount}() (#127)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#148-165):
- (success,returndata) = target.call{value: weiValue}(data) (#151)
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() (#255) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#256) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#272) is not in mixedCase
Function IUniswapV2Router01.WETH() (#291) is not in mixedCase
Parameter WINU.setSwapAndLiquifyEnabled(bool)._enabled (#642) is not in mixedCase
Variable WINU._balances (#437) is not in mixedCase
Variable WINU._buyLiquidityFee (#445) is not in mixedCase
Variable WINU._buyMarketingFee (#446) is not in mixedCase
Variable WINU._buyTeamFee (#447) is not in mixedCase
Variable WINU._sellLiquidityFee (#449) is not in mixedCase
Variable WINU._sellMarketingFee (#450) is not in mixedCase
Variable WINU._sellTeamFee (#451) is not in mixedCase
Variable WINU._liquidityShare (#453) is not in mixedCase
Variable WINU._marketingShare (#454) is not in mixedCase
Variable WINU._teamShare (#455) is not in mixedCase
Variable WINU._totalTaxIfBuying (#457) is not in mixedCase
Variable WINU._totalTaxIfSelling (#458) is not in mixedCase
Variable WINU._totalDistributionShares (#459) is not in mixedCase
Variable WINU._maxTxAmount (#462) is not in mixedCase
Variable WINU._walletMax (#463) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#40)" inContext (#33-43)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in WINU._transfer(address,address,uint256) (#692-730):
External calls:
- swapAndLiquify(contractTokenBalance) (#714)
- recipient.transfer(amount) (#656)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#714)
- recipient.transfer(amount) (#656)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#717)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#725)
- finalAmount = takeFee(sender,recipient,amount) (#719-720)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#810)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#811)
- finalAmount = takeFee(sender,recipient,amount) (#719-720)
- Transfer(sender,recipient,finalAmount) (#727)
Reentrancy in WINU.swapAndLiquify(uint256) (#739-761):
External calls:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#754)
- recipient.transfer(amount) (#656)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#757)
- recipient.transfer(amount) (#656)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#754)
- recipient.transfer(amount) (#656)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#757)
- recipient.transfer(amount) (#656)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#760)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#760)
- _allowances[owner][spender] = amount (#574)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#575)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#760)
Reentrancy in WINU.transferFrom(address,address,uint256) (#686-690):
External calls:
- _transfer(sender,recipient,amount) (#687)
- recipient.transfer(amount) (#656)
External calls sending eth:
- _transfer(sender,recipient,amount) (#687)
- recipient.transfer(amount) (#656)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#788-795)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#688)
- _allowances[owner][spender] = amount (#574)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#575)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#688)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#296) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#297)
Prevent variables from having similar names.
Additional information: link
WINU.slitherConstructorVariables() (#424-818) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#435)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
WINU._decimals (#431) should be constant
WINU._name (#429) should be constant
WINU._symbol (#430) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
waiveOwnership() should be declared external:
- Ownable.waiveOwnership() (#190-193)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#195-199)
getUnlockTime() should be declared external:
- Ownable.getUnlockTime() (#201-203)
getTime() should be declared external:
- Ownable.getTime() (#205-207)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#209-214)
unlock() should be declared external:
- Ownable.unlock() (#216-221)
name() should be declared external:
- WINU.name() (#527-529)
symbol() should be declared external:
- WINU.symbol() (#531-533)
decimals() should be declared external:
- WINU.decimals() (#535-537)
totalSupply() should be declared external:
- WINU.totalSupply() (#539-541)
allowance(address,address) should be declared external:
- WINU.allowance(address,address) (#547-549)
increaseAllowance(address,uint256) should be declared external:
- WINU.increaseAllowance(address,uint256) (#551-554)
decreaseAllowance(address,uint256) should be declared external:
- WINU.decreaseAllowance(address,uint256) (#556-559)
minimumTokensBeforeSwapAmount() should be declared external:
- WINU.minimumTokensBeforeSwapAmount() (#561-563)
approve(address,uint256) should be declared external:
- WINU.approve(address,uint256) (#565-568)
setMarketPairStatus(address,bool) should be declared external:
- WINU.setMarketPairStatus(address,bool) (#578-580)
setIsExcludedFromFee(address,bool) should be declared external:
- WINU.setIsExcludedFromFee(address,bool) (#586-588)
setSwapAndLiquifyEnabled(bool) should be declared external:
- WINU.setSwapAndLiquifyEnabled(bool) (#642-645)
setSwapAndLiquifyByLimitOnly(bool) should be declared external:
- WINU.setSwapAndLiquifyByLimitOnly(bool) (#647-649)
getCirculatingSupply() should be declared external:
- WINU.getCirculatingSupply() (#651-653)
changeRouterVersion(address) should be declared external:
- WINU.changeRouterVersion(address) (#659-676)
transfer(address,uint256) should be declared external:
- WINU.transfer(address,uint256) (#681-684)
transferFrom(address,address,uint256) should be declared external:
- WINU.transferFrom(address,address,uint256) (#686-690)
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