Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
M31YachtClub.swapBack() (#747-790) sends eth to arbitrary user
Dangerous calls:
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#774)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in M31YachtClub._transferFrom(address,address,uint256) (#549-600):
External calls:
- swapBack() (#579)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#758-764)
- distributor.deposit{value: amountBNBReflection}() (#773)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#774)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#775)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#780-787)
External calls sending eth:
- swapBack() (#579)
- distributor.deposit{value: amountBNBReflection}() (#773)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#774)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#775)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#780-787)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#582)
- _balances[recipient] = _balances[recipient].add(amountReceived) (#585)
- amountReceived = takeFee(sender,amount,isSell,GREEDMode) (#584)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#704)
Apply the check-effects-interactions pattern.
Additional information: link
DividendDistributor.distributeDividend(address) (#326-337) ignores return value by BUSD.transfer(shareholder,amount) (#332)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.
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.
Combination 2: Unchecked transfer + 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.
Combination 3: Reentrancy vulnerabilities + Unchecked transfer vulnerability. 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.
M31YachtClub.launched() (#863-865) is never used and should be removed
M31YachtClub.shouldAutoBuyback() (#793-799) is never used and should be removed
M31YachtClub.triggerAutoBuyback() (#818-827) is never used and should be removed
Remove unused functions.
Additional information: link
DividendDistributor.process(uint256) (#295-319) has costly operations inside a loop:
- currentIndex = 0 (#307)
DividendDistributor.distributeDividend(address) (#326-337) has costly operations inside a loop:
- totalDistributed = totalDistributed.add(amount) (#331)
DividendDistributor.process(uint256) (#295-319) has costly operations inside a loop:
- currentIndex ++ (#316)
Use a local variable to hold the loop computation result.
Additional information: link
Reentrancy in DividendDistributor.distributeDividend(address) (#326-337):
External calls:
- BUSD.transfer(shareholder,amount) (#332)
State variables written after the call(s):
- shares[shareholder].totalRealised = shares[shareholder].totalRealised.add(amount) (#334)
- shares[shareholder].totalExcluded = getCumulativeDividends(shares[shareholder].amount) (#335)
Reentrancy in DividendDistributor.process(uint256) (#295-319):
External calls:
- distributeDividend(shareholders[currentIndex]) (#311)
- BUSD.transfer(shareholder,amount) (#332)
State variables written after the call(s):
- currentIndex ++ (#316)
Reentrancy in DividendDistributor.setShare(address,uint256) (#259-273):
External calls:
- distributeDividend(shareholder) (#261)
- BUSD.transfer(shareholder,amount) (#332)
State variables written after the call(s):
- shares[shareholder].amount = amount (#271)
- shares[shareholder].totalExcluded = getCumulativeDividends(shares[shareholder].amount) (#272)
Apply the check-effects-interactions pattern.
Additional information: link
M31YachtClub.swapBack() (#747-790) ignores return value by router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#780-787)
Ensure that all the return values of the function calls are used.
Additional information: link
DividendDistributor.setDistributionCriteria(uint256,uint256) (#254-257) should emit an event for:
- minPeriod = _minPeriod (#255)
- minDistribution = _minDistribution (#256)
M31YachtClub.setMaxWalletPercent(uint256) (#543-546) should emit an event for:
- _maxWalletToken = _totalSupply.mul(maxWallPercent).div(10000) (#544)
M31YachtClub.enableGREED(uint256) (#725-728) should emit an event for:
- GREEDDuration = _seconds (#727)
M31YachtClub.cooldownEnabled(bool,uint8) (#736-739) should emit an event for:
- cooldownTimerInterval = _interval (#738)
M31YachtClub.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool) (#844-852) should emit an event for:
- autoBuybackCap = _cap (#846)
- autoBuybackAmount = _amount (#848)
M31YachtClub.setBuybackMultiplierSettings(uint256,uint256,uint256) (#855-860) should emit an event for:
- buybackMultiplierNumerator = numerator (#857)
- buybackMultiplierDenominator = denominator (#858)
- buybackMultiplierLength = length (#859)
M31YachtClub.setBuyTxLimitInPercent(uint256) (#873-875) should emit an event for:
- _maxBuyTxAmount = _totalSupply.mul(maxBuyTxPercent).div(10000) (#874)
M31YachtClub.setSellTxLimitInPercent(uint256) (#878-880) should emit an event for:
- _maxSellTxAmount = _totalSupply.mul(maxSellTxPercent).div(10000) (#879)
M31YachtClub.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256) (#909-917) should emit an event for:
- liquidityFeeBuy = _liquidityFeeBuy (#910)
- buybackFeeBuy = _buybackFeeBuy (#911)
- reflectionFeeBuy = _reflectionFeeBuy (#912)
- marketingFeeBuy = _marketingFeeBuy (#913)
- devFeeBuy = _devFeeBuy (#914)
- totalFeeBuy = _liquidityFeeBuy.add(_buybackFeeBuy).add(_reflectionFeeBuy).add(_marketingFeeBuy).add(_devFeeBuy) (#915)
- feeDenominator = _feeDenominator (#916)
M31YachtClub.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256) (#920-928) should emit an event for:
- liquidityFeeSell = _liquidityFeeSell (#921)
- buybackFeeSell = _buybackFeeSell (#922)
- reflectionFeeSell = _reflectionFeeSell (#923)
- marketingFeeSell = _marketingFeeSell (#924)
- devFeeSell = _devFeeSell (#925)
- totalFeeSell = _liquidityFeeSell.add(_buybackFeeSell).add(_reflectionFeeSell).add(_marketingFeeSell).add(_devFeeSell) (#926)
- feeDenominator = _feeDenominator (#927)
M31YachtClub.setSwapBackSettings(bool,uint256) (#937-940) should emit an event for:
- swapThreshold = _totalSupply * _amount / 10000 (#939)
M31YachtClub.setTargetLiquidity(uint256,uint256) (#943-946) should emit an event for:
- targetLiquidity = _target (#944)
- targetLiquidityDenominator = _denominator (#945)
Emit an event for critical parameter changes.
Additional information: link
Auth.transferOwnership(address).account (#131) lacks a zero-check on :
- owner = account (#132)
M31YachtClub.setFeeReceivers(address,address)._autoLiquidityReceiver (#931) lacks a zero-check on :
- autoLiquidityReceiver = _autoLiquidityReceiver (#932)
M31YachtClub.setFeeReceivers(address,address)._marketingFeeReceiver (#931) lacks a zero-check on :
- marketingFeeReceiver = _marketingFeeReceiver (#933)
Check that the address is not zero.
Additional information: link
DividendDistributor.distributeDividend(address) (#326-337) has external calls inside a loop: BUSD.transfer(shareholder,amount) (#332)
Favor pull over push strategy for external calls.
Additional information: link
Reentrancy in M31YachtClub.constructor() (#478-508):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#481)
State variables written after the call(s):
- _allowances[address(this)][address(router)] = type()(uint256).max (#482)
- _balances[_presaler] = _totalSupply (#506)
- autoLiquidityReceiver = msg.sender (#503)
- distributor = new DividendDistributor(address(router)) (#484)
- isDividendExempt[pair] = true (#498)
- isDividendExempt[address(this)] = true (#499)
- isDividendExempt[DEAD] = true (#500)
- isFeeExempt[_presaler] = true (#488)
- isTimelockExempt[msg.sender] = true (#492)
- isTimelockExempt[DEAD] = true (#493)
- isTimelockExempt[address(this)] = true (#494)
- isTimelockExempt[DEV] = true (#495)
- isTxLimitExempt[_presaler] = true (#489)
- marketingFeeReceiver = 0xAd5b383AFCe173e97078DE42BceF6802366B63BA (#504)
Reentrancy in DividendDistributor.deposit() (#275-293):
External calls:
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: msg.value}(0,path,address(this),block.timestamp) (#282-287)
State variables written after the call(s):
- dividendsPerShare = dividendsPerShare.add(dividendsPerShareAccuracyFactor.mul(amount).div(totalShares)) (#292)
- totalDividends = totalDividends.add(amount) (#291)
Reentrancy in DividendDistributor.distributeDividend(address) (#326-337):
External calls:
- BUSD.transfer(shareholder,amount) (#332)
State variables written after the call(s):
- shareholderClaims[shareholder] = block.timestamp (#333)
Reentrancy in DividendDistributor.setShare(address,uint256) (#259-273):
External calls:
- distributeDividend(shareholder) (#261)
- BUSD.transfer(shareholder,amount) (#332)
State variables written after the call(s):
- addShareholder(shareholder) (#265)
- shareholderIndexes[shareholder] = shareholders.length (#359)
- removeShareholder(shareholder) (#267)
- shareholderIndexes[shareholders[shareholders.length - 1]] = shareholderIndexes[shareholder] (#365)
- addShareholder(shareholder) (#265)
- shareholders.push(shareholder) (#360)
- removeShareholder(shareholder) (#267)
- shareholders[shareholderIndexes[shareholder]] = shareholders[shareholders.length - 1] (#364)
- shareholders.pop() (#366)
- totalShares = totalShares.sub(shares[shareholder].amount).add(amount) (#270)
Reentrancy in M31YachtClub.triggerAutoBuyback() (#818-827):
External calls:
- buyTokens(autoBuybackAmount,DEAD) (#819)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,to,block.timestamp) (#835-840)
State variables written after the call(s):
- autoBuybackAccumulator = autoBuybackAccumulator.add(autoBuybackAmount) (#825)
- autoBuybackBlockLast = block.number (#824)
- autoBuybackEnabled = false (#826)
- buybackMultiplierTriggeredAt = block.timestamp (#821)
Reentrancy in M31YachtClub.triggerManualBuyback(uint256,bool) (#802-810):
External calls:
- buyTokens(amountToBuy,DEAD) (#805)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,to,block.timestamp) (#835-840)
State variables written after the call(s):
- buybackMultiplierTriggeredAt = block.timestamp (#807)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in M31YachtClub._transferFrom(address,address,uint256) (#549-600):
External calls:
- swapBack() (#579)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#758-764)
- distributor.deposit{value: amountBNBReflection}() (#773)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#774)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#775)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#780-787)
- distributor.setShare(sender,_balances[sender]) (#589)
- distributor.setShare(recipient,_balances[recipient]) (#593)
- distributor.process(distributorGas) (#596)
External calls sending eth:
- swapBack() (#579)
- distributor.deposit{value: amountBNBReflection}() (#773)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#774)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#775)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#780-787)
Event emitted after the call(s):
- Transfer(sender,recipient,amountReceived) (#598)
Reentrancy in M31YachtClub._transferFrom(address,address,uint256) (#549-600):
External calls:
- swapBack() (#579)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#758-764)
- distributor.deposit{value: amountBNBReflection}() (#773)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#774)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#775)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#780-787)
External calls sending eth:
- swapBack() (#579)
- distributor.deposit{value: amountBNBReflection}() (#773)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#774)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#775)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#780-787)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#705)
- amountReceived = takeFee(sender,amount,isSell,GREEDMode) (#584)
Reentrancy in M31YachtClub.constructor() (#478-508):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#481)
Event emitted after the call(s):
- Transfer(address(0),_presaler,_totalSupply) (#507)
Reentrancy in M31YachtClub.swapBack() (#747-790):
External calls:
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#758-764)
- distributor.deposit{value: amountBNBReflection}() (#773)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#774)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#775)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#780-787)
External calls sending eth:
- distributor.deposit{value: amountBNBReflection}() (#773)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#774)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#775)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#780-787)
Event emitted after the call(s):
- AutoLiquify(amountBNBLiquidity,amountToLiquify) (#788)
Reentrancy in M31YachtClub.triggerAutoBuyback() (#818-827):
External calls:
- buyTokens(autoBuybackAmount,DEAD) (#819)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,to,block.timestamp) (#835-840)
Event emitted after the call(s):
- BuybackMultiplierActive(buybackMultiplierLength) (#822)
Reentrancy in M31YachtClub.triggerManualBuyback(uint256,bool) (#802-810):
External calls:
- buyTokens(amountToBuy,DEAD) (#805)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,to,block.timestamp) (#835-840)
Event emitted after the call(s):
- BuybackMultiplierActive(buybackMultiplierLength) (#808)
Apply the check-effects-interactions pattern.
Additional information: link
DividendDistributor.shouldDistribute(address) (#321-324) uses timestamp for comparisons
Dangerous comparisons:
- shareholderClaims[shareholder] + minPeriod < block.timestamp && getUnpaidEarnings(shareholder) > minDistribution (#322-323)
M31YachtClub.inGREEDTime() (#630-636) uses timestamp for comparisons
Dangerous comparisons:
- GREEDTriggeredAt.add(GREEDDuration) > block.timestamp (#631)
M31YachtClub.checkBuyCooldown(address,address) (#651-658) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(cooldownTimer[recipient] < block.timestamp,Please wait between two buys) (#655)
M31YachtClub.getTotalFee(bool) (#674-678) uses timestamp for comparisons
Dangerous comparisons:
- selling && buybackMultiplierTriggeredAt.add(buybackMultiplierLength) > block.timestamp (#676)
Avoid relying on block.timestamp.
Additional information: link
M31YachtClub._maxBuyTxAmount (#394) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 200 / 10000
M31YachtClub._maxSellTxAmount (#395) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 200 / 10000
M31YachtClub._maxWalletToken (#396) is set pre-construction with a non-constant function or state variable:
- (_totalSupply * 400) / 10000
M31YachtClub.swapThreshold (#440) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 70 / 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 M31YachtClub.swapBack() (#747-790):
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#774)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#775)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Function IDEXRouter.WETH() (#148) is not in mixedCase
Parameter DividendDistributor.setDistributionCriteria(uint256,uint256)._minPeriod (#254) is not in mixedCase
Parameter DividendDistributor.setDistributionCriteria(uint256,uint256)._minDistribution (#254) is not in mixedCase
Variable DividendDistributor._token (#206) is not in mixedCase
Variable DividendDistributor.BUSD (#215) is not in mixedCase
Variable DividendDistributor.WBNB (#216) is not in mixedCase
Parameter M31YachtClub.takeFee(address,uint256,bool,bool).GREEDMode (#688) is not in mixedCase
Parameter M31YachtClub.tradingStatus(bool)._status (#719) is not in mixedCase
Parameter M31YachtClub.enableGREED(uint256)._seconds (#725) is not in mixedCase
Parameter M31YachtClub.cooldownEnabled(bool,uint8)._status (#736) is not in mixedCase
Parameter M31YachtClub.cooldownEnabled(bool,uint8)._interval (#736) is not in mixedCase
Parameter M31YachtClub.blacklistAddress(address,bool)._address (#742) is not in mixedCase
Parameter M31YachtClub.blacklistAddress(address,bool)._value (#742) is not in mixedCase
Parameter M31YachtClub.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._enabled (#844) is not in mixedCase
Parameter M31YachtClub.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._cap (#844) is not in mixedCase
Parameter M31YachtClub.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._amount (#844) is not in mixedCase
Parameter M31YachtClub.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._period (#844) is not in mixedCase
Parameter M31YachtClub.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._autoBuybackMultiplier (#844) is not in mixedCase
Parameter M31YachtClub.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._liquidityFeeBuy (#909) is not in mixedCase
Parameter M31YachtClub.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._buybackFeeBuy (#909) is not in mixedCase
Parameter M31YachtClub.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._reflectionFeeBuy (#909) is not in mixedCase
Parameter M31YachtClub.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._marketingFeeBuy (#909) is not in mixedCase
Parameter M31YachtClub.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._devFeeBuy (#909) is not in mixedCase
Parameter M31YachtClub.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._feeDenominator (#909) is not in mixedCase
Parameter M31YachtClub.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._liquidityFeeSell (#920) is not in mixedCase
Parameter M31YachtClub.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._buybackFeeSell (#920) is not in mixedCase
Parameter M31YachtClub.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._reflectionFeeSell (#920) is not in mixedCase
Parameter M31YachtClub.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._marketingFeeSell (#920) is not in mixedCase
Parameter M31YachtClub.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._devFeeSell (#920) is not in mixedCase
Parameter M31YachtClub.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._feeDenominator (#920) is not in mixedCase
Parameter M31YachtClub.setFeeReceivers(address,address)._autoLiquidityReceiver (#931) is not in mixedCase
Parameter M31YachtClub.setFeeReceivers(address,address)._marketingFeeReceiver (#931) is not in mixedCase
Parameter M31YachtClub.setSwapBackSettings(bool,uint256)._enabled (#937) is not in mixedCase
Parameter M31YachtClub.setSwapBackSettings(bool,uint256)._amount (#937) is not in mixedCase
Parameter M31YachtClub.setTargetLiquidity(uint256,uint256)._target (#943) is not in mixedCase
Parameter M31YachtClub.setTargetLiquidity(uint256,uint256)._denominator (#943) is not in mixedCase
Parameter M31YachtClub.setDistributionCriteria(uint256,uint256)._minPeriod (#955) is not in mixedCase
Parameter M31YachtClub.setDistributionCriteria(uint256,uint256)._minDistribution (#955) is not in mixedCase
Variable M31YachtClub.BUSD (#375) is not in mixedCase
Variable M31YachtClub.WBNB (#376) is not in mixedCase
Variable M31YachtClub.DEAD (#377) is not in mixedCase
Variable M31YachtClub.ZERO (#378) is not in mixedCase
Variable M31YachtClub.DEV (#379) is not in mixedCase
Constant M31YachtClub._name (#386) is not in UPPER_CASE_WITH_UNDERSCORES
Constant M31YachtClub._symbol (#387) is not in UPPER_CASE_WITH_UNDERSCORES
Constant M31YachtClub._decimals (#388) is not in UPPER_CASE_WITH_UNDERSCORES
Variable M31YachtClub._totalSupply (#391) is not in mixedCase
Variable M31YachtClub._maxBuyTxAmount (#394) is not in mixedCase
Variable M31YachtClub._maxSellTxAmount (#395) is not in mixedCase
Variable M31YachtClub._maxWalletToken (#396) is not in mixedCase
Variable M31YachtClub._balances (#399) is not in mixedCase
Variable M31YachtClub._allowances (#400) is not in mixedCase
Variable M31YachtClub.GREEDTriggeredAt (#433) is not in mixedCase
Variable M31YachtClub.GREEDDuration (#434) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Variable IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#153) is too similar to IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#154)
Prevent variables from having similar names.
Additional information: link
M31YachtClub.slitherConstructorVariables() (#371-993) uses literals with too many digits:
- DEAD = 0x000000000000000000000000000000000000dEaD (#377)
M31YachtClub.slitherConstructorVariables() (#371-993) uses literals with too many digits:
- ZERO = 0x0000000000000000000000000000000000000000 (#378)
M31YachtClub.slitherConstructorVariables() (#371-993) uses literals with too many digits:
- _totalSupply = 1000000 * (10 ** _decimals) (#391)
M31YachtClub.slitherConstructorVariables() (#371-993) uses literals with too many digits:
- distributorGas = 500000 (#461)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
M31YachtClub.BUSD (#375) is never used in M31YachtClub (#371-993)
Remove unused state variables.
Additional information: link
DividendDistributor.WBNB (#216) should be constant
DividendDistributor.dividendsPerShareAccuracyFactor (#229) should be constant
M31YachtClub.BUSD (#375) should be constant
M31YachtClub.DEAD (#377) should be constant
M31YachtClub.DEV (#379) should be constant
M31YachtClub.WBNB (#376) should be constant
M31YachtClub.ZERO (#378) should be constant
M31YachtClub._totalSupply (#391) should be constant
M31YachtClub.deadBlocks (#437) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
authorize(address) should be declared external:
- Auth.authorize(address) (#103-105)
unauthorize(address) should be declared external:
- Auth.unauthorize(address) (#110-112)
transferOwnership(address) should be declared external:
- Auth.transferOwnership(address) (#131-135)
getTotalFee(bool) should be declared external:
- M31YachtClub.getTotalFee(bool) (#674-678)
tradingStatus(bool) should be declared external:
- M31YachtClub.tradingStatus(bool) (#719-722)
enableGREED(uint256) should be declared external:
- M31YachtClub.enableGREED(uint256) (#725-728)
cooldownEnabled(bool,uint8) should be declared external:
- M31YachtClub.cooldownEnabled(bool,uint8) (#736-739)
blacklistAddress(address,bool) should be declared external:
- M31YachtClub.blacklistAddress(address,bool) (#742-744)
getUnpaidEarnings(address) should be declared external:
- M31YachtClub.getUnpaidEarnings(address) (#965-967)
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