Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
FIRSTVALENTINE.swapBack() (#742-785) sends eth to arbitrary user
Dangerous calls:
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#769)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in FIRSTVALENTINE._transferFrom(address,address,uint256) (#544-595):
External calls:
- swapBack() (#574)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#753-759)
- distributor.deposit{value: amountBNBReflection}() (#768)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#769)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#770)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#775-782)
External calls sending eth:
- swapBack() (#574)
- distributor.deposit{value: amountBNBReflection}() (#768)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#769)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#770)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#775-782)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#577)
- _balances[recipient] = _balances[recipient].add(amountReceived) (#580)
- amountReceived = takeFee(sender,amount,isSell,GREEDMode) (#579)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#699)
Apply the check-effects-interactions pattern.
Additional information: link
DividendDistributor.distributeDividend(address) (#321-332) ignores return value by BUSD.transfer(shareholder,amount) (#327)
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.
Reentrancy in DividendDistributor.distributeDividend(address) (#321-332):
External calls:
- BUSD.transfer(shareholder,amount) (#327)
State variables written after the call(s):
- shares[shareholder].totalRealised = shares[shareholder].totalRealised.add(amount) (#329)
- shares[shareholder].totalExcluded = getCumulativeDividends(shares[shareholder].amount) (#330)
Reentrancy in DividendDistributor.process(uint256) (#290-314):
External calls:
- distributeDividend(shareholders[currentIndex]) (#306)
- BUSD.transfer(shareholder,amount) (#327)
State variables written after the call(s):
- currentIndex ++ (#311)
Reentrancy in DividendDistributor.setShare(address,uint256) (#254-268):
External calls:
- distributeDividend(shareholder) (#256)
- BUSD.transfer(shareholder,amount) (#327)
State variables written after the call(s):
- shares[shareholder].amount = amount (#266)
- shares[shareholder].totalExcluded = getCumulativeDividends(shares[shareholder].amount) (#267)
Apply the check-effects-interactions pattern.
Additional information: link
FIRSTVALENTINE.swapBack() (#742-785) ignores return value by router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#775-782)
Ensure that all the return values of the function calls are used.
Additional information: link
DividendDistributor.setDistributionCriteria(uint256,uint256) (#249-252) should emit an event for:
- minPeriod = _minPeriod (#250)
- minDistribution = _minDistribution (#251)
FIRSTVALENTINE.setMaxWalletPercent(uint256) (#538-541) should emit an event for:
- _maxWalletToken = _totalSupply.mul(maxWallPercent).div(10000) (#539)
FIRSTVALENTINE.enableGREED(uint256) (#720-723) should emit an event for:
- GREEDDuration = _seconds (#722)
FIRSTVALENTINE.cooldownEnabled(bool,uint8) (#731-734) should emit an event for:
- cooldownTimerInterval = _interval (#733)
FIRSTVALENTINE.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool) (#839-847) should emit an event for:
- autoBuybackCap = _cap (#841)
- autoBuybackAmount = _amount (#843)
FIRSTVALENTINE.setBuybackMultiplierSettings(uint256,uint256,uint256) (#850-855) should emit an event for:
- buybackMultiplierNumerator = numerator (#852)
- buybackMultiplierDenominator = denominator (#853)
- buybackMultiplierLength = length (#854)
FIRSTVALENTINE.setBuyTxLimitInPercent(uint256) (#868-870) should emit an event for:
- _maxBuyTxAmount = _totalSupply.mul(maxBuyTxPercent).div(10000) (#869)
FIRSTVALENTINE.setSellTxLimitInPercent(uint256) (#873-875) should emit an event for:
- _maxSellTxAmount = _totalSupply.mul(maxSellTxPercent).div(10000) (#874)
FIRSTVALENTINE.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256) (#904-912) should emit an event for:
- liquidityFeeBuy = _liquidityFeeBuy (#905)
- buybackFeeBuy = _buybackFeeBuy (#906)
- reflectionFeeBuy = _reflectionFeeBuy (#907)
- marketingFeeBuy = _marketingFeeBuy (#908)
- devFeeBuy = _devFeeBuy (#909)
- totalFeeBuy = _liquidityFeeBuy.add(_buybackFeeBuy).add(_reflectionFeeBuy).add(_marketingFeeBuy).add(_devFeeBuy) (#910)
- feeDenominator = _feeDenominator (#911)
FIRSTVALENTINE.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256) (#915-923) should emit an event for:
- liquidityFeeSell = _liquidityFeeSell (#916)
- buybackFeeSell = _buybackFeeSell (#917)
- reflectionFeeSell = _reflectionFeeSell (#918)
- marketingFeeSell = _marketingFeeSell (#919)
- devFeeSell = _devFeeSell (#920)
- totalFeeSell = _liquidityFeeSell.add(_buybackFeeSell).add(_reflectionFeeSell).add(_marketingFeeSell).add(_devFeeSell) (#921)
- feeDenominator = _feeDenominator (#922)
FIRSTVALENTINE.setSwapBackSettings(bool,uint256) (#932-935) should emit an event for:
- swapThreshold = _totalSupply * _amount / 10000 (#934)
FIRSTVALENTINE.setTargetLiquidity(uint256,uint256) (#938-941) should emit an event for:
- targetLiquidity = _target (#939)
- targetLiquidityDenominator = _denominator (#940)
Emit an event for critical parameter changes.
Additional information: link
Auth.transferOwnership(address).account (#126) lacks a zero-check on :
- owner = account (#127)
FIRSTVALENTINE.setFeeReceivers(address,address)._autoLiquidityReceiver (#926) lacks a zero-check on :
- autoLiquidityReceiver = _autoLiquidityReceiver (#927)
FIRSTVALENTINE.setFeeReceivers(address,address)._marketingFeeReceiver (#926) lacks a zero-check on :
- marketingFeeReceiver = _marketingFeeReceiver (#928)
Check that the address is not zero.
Additional information: link
DividendDistributor.distributeDividend(address) (#321-332) has external calls inside a loop: BUSD.transfer(shareholder,amount) (#327)
Favor pull over push strategy for external calls.
Additional information: link
Reentrancy in FIRSTVALENTINE.constructor() (#473-503):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#476)
State variables written after the call(s):
- _allowances[address(this)][address(router)] = type()(uint256).max (#477)
- _balances[_presaler] = _totalSupply (#501)
- autoLiquidityReceiver = msg.sender (#498)
- distributor = new DividendDistributor(address(router)) (#479)
- isDividendExempt[pair] = true (#493)
- isDividendExempt[address(this)] = true (#494)
- isDividendExempt[DEAD] = true (#495)
- isFeeExempt[_presaler] = true (#483)
- isTimelockExempt[msg.sender] = true (#487)
- isTimelockExempt[DEAD] = true (#488)
- isTimelockExempt[address(this)] = true (#489)
- isTimelockExempt[DEV] = true (#490)
- isTxLimitExempt[_presaler] = true (#484)
- marketingFeeReceiver = 0xEB402BAfd40A393d7EdBae1E1FfeadA019F41396 (#499)
Reentrancy in DividendDistributor.deposit() (#270-288):
External calls:
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: msg.value}(0,path,address(this),block.timestamp) (#277-282)
State variables written after the call(s):
- dividendsPerShare = dividendsPerShare.add(dividendsPerShareAccuracyFactor.mul(amount).div(totalShares)) (#287)
- totalDividends = totalDividends.add(amount) (#286)
Reentrancy in DividendDistributor.distributeDividend(address) (#321-332):
External calls:
- BUSD.transfer(shareholder,amount) (#327)
State variables written after the call(s):
- shareholderClaims[shareholder] = block.timestamp (#328)
Reentrancy in DividendDistributor.setShare(address,uint256) (#254-268):
External calls:
- distributeDividend(shareholder) (#256)
- BUSD.transfer(shareholder,amount) (#327)
State variables written after the call(s):
- addShareholder(shareholder) (#260)
- shareholderIndexes[shareholder] = shareholders.length (#354)
- removeShareholder(shareholder) (#262)
- shareholderIndexes[shareholders[shareholders.length - 1]] = shareholderIndexes[shareholder] (#360)
- addShareholder(shareholder) (#260)
- shareholders.push(shareholder) (#355)
- removeShareholder(shareholder) (#262)
- shareholders[shareholderIndexes[shareholder]] = shareholders[shareholders.length - 1] (#359)
- shareholders.pop() (#361)
- totalShares = totalShares.sub(shares[shareholder].amount).add(amount) (#265)
Reentrancy in FIRSTVALENTINE.triggerAutoBuyback() (#813-822):
External calls:
- buyTokens(autoBuybackAmount,DEAD) (#814)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,to,block.timestamp) (#830-835)
State variables written after the call(s):
- autoBuybackAccumulator = autoBuybackAccumulator.add(autoBuybackAmount) (#820)
- autoBuybackBlockLast = block.number (#819)
- autoBuybackEnabled = false (#821)
- buybackMultiplierTriggeredAt = block.timestamp (#816)
Reentrancy in FIRSTVALENTINE.triggerManualBuyback(uint256,bool) (#797-805):
External calls:
- buyTokens(amountToBuy,DEAD) (#800)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,to,block.timestamp) (#830-835)
State variables written after the call(s):
- buybackMultiplierTriggeredAt = block.timestamp (#802)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in FIRSTVALENTINE._transferFrom(address,address,uint256) (#544-595):
External calls:
- swapBack() (#574)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#753-759)
- distributor.deposit{value: amountBNBReflection}() (#768)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#769)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#770)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#775-782)
- distributor.setShare(sender,_balances[sender]) (#584)
- distributor.setShare(recipient,_balances[recipient]) (#588)
- distributor.process(distributorGas) (#591)
External calls sending eth:
- swapBack() (#574)
- distributor.deposit{value: amountBNBReflection}() (#768)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#769)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#770)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#775-782)
Event emitted after the call(s):
- Transfer(sender,recipient,amountReceived) (#593)
Reentrancy in FIRSTVALENTINE._transferFrom(address,address,uint256) (#544-595):
External calls:
- swapBack() (#574)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#753-759)
- distributor.deposit{value: amountBNBReflection}() (#768)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#769)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#770)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#775-782)
External calls sending eth:
- swapBack() (#574)
- distributor.deposit{value: amountBNBReflection}() (#768)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#769)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#770)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#775-782)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#700)
- amountReceived = takeFee(sender,amount,isSell,GREEDMode) (#579)
Reentrancy in FIRSTVALENTINE.constructor() (#473-503):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#476)
Event emitted after the call(s):
- Transfer(address(0),_presaler,_totalSupply) (#502)
Reentrancy in FIRSTVALENTINE.swapBack() (#742-785):
External calls:
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#753-759)
- distributor.deposit{value: amountBNBReflection}() (#768)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#769)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#770)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#775-782)
External calls sending eth:
- distributor.deposit{value: amountBNBReflection}() (#768)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#769)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#770)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#775-782)
Event emitted after the call(s):
- AutoLiquify(amountBNBLiquidity,amountToLiquify) (#783)
Reentrancy in FIRSTVALENTINE.triggerAutoBuyback() (#813-822):
External calls:
- buyTokens(autoBuybackAmount,DEAD) (#814)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,to,block.timestamp) (#830-835)
Event emitted after the call(s):
- BuybackMultiplierActive(buybackMultiplierLength) (#817)
Reentrancy in FIRSTVALENTINE.triggerManualBuyback(uint256,bool) (#797-805):
External calls:
- buyTokens(amountToBuy,DEAD) (#800)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,to,block.timestamp) (#830-835)
Event emitted after the call(s):
- BuybackMultiplierActive(buybackMultiplierLength) (#803)
Apply the check-effects-interactions pattern.
Additional information: link
DividendDistributor.shouldDistribute(address) (#316-319) uses timestamp for comparisons
Dangerous comparisons:
- shareholderClaims[shareholder] + minPeriod < block.timestamp && getUnpaidEarnings(shareholder) > minDistribution (#317-318)
FIRSTVALENTINE.inGREEDTime() (#625-631) uses timestamp for comparisons
Dangerous comparisons:
- GREEDTriggeredAt.add(GREEDDuration) > block.timestamp (#626)
FIRSTVALENTINE.checkBuyCooldown(address,address) (#646-653) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(cooldownTimer[recipient] < block.timestamp,Please wait between two buys) (#650)
FIRSTVALENTINE.getTotalFee(bool) (#669-673) uses timestamp for comparisons
Dangerous comparisons:
- selling && buybackMultiplierTriggeredAt.add(buybackMultiplierLength) > block.timestamp (#671)
Avoid relying on block.timestamp.
Additional information: link
DividendDistributor.process(uint256) (#290-314) has costly operations inside a loop:
- currentIndex = 0 (#302)
DividendDistributor.distributeDividend(address) (#321-332) has costly operations inside a loop:
- totalDistributed = totalDistributed.add(amount) (#326)
DividendDistributor.process(uint256) (#290-314) has costly operations inside a loop:
- currentIndex ++ (#311)
Use a local variable to hold the loop computation result.
Additional information: link
FIRSTVALENTINE.launched() (#858-860) is never used and should be removed
FIRSTVALENTINE.shouldAutoBuyback() (#788-794) is never used and should be removed
FIRSTVALENTINE.triggerAutoBuyback() (#813-822) is never used and should be removed
Remove unused functions.
Additional information: link
FIRSTVALENTINE._maxBuyTxAmount (#389) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 150 / 10000
FIRSTVALENTINE._maxSellTxAmount (#390) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 150 / 10000
FIRSTVALENTINE._maxWalletToken (#391) is set pre-construction with a non-constant function or state variable:
- (_totalSupply * 300) / 10000
FIRSTVALENTINE.swapThreshold (#435) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 75 / 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 FIRSTVALENTINE.swapBack() (#742-785):
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#769)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#770)
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() (#143) is not in mixedCase
Parameter DividendDistributor.setDistributionCriteria(uint256,uint256)._minPeriod (#249) is not in mixedCase
Parameter DividendDistributor.setDistributionCriteria(uint256,uint256)._minDistribution (#249) is not in mixedCase
Variable DividendDistributor._token (#201) is not in mixedCase
Variable DividendDistributor.BUSD (#210) is not in mixedCase
Variable DividendDistributor.WBNB (#211) is not in mixedCase
Parameter FIRSTVALENTINE.takeFee(address,uint256,bool,bool).GREEDMode (#683) is not in mixedCase
Parameter FIRSTVALENTINE.tradingStatus(bool)._status (#714) is not in mixedCase
Parameter FIRSTVALENTINE.enableGREED(uint256)._seconds (#720) is not in mixedCase
Parameter FIRSTVALENTINE.cooldownEnabled(bool,uint8)._status (#731) is not in mixedCase
Parameter FIRSTVALENTINE.cooldownEnabled(bool,uint8)._interval (#731) is not in mixedCase
Parameter FIRSTVALENTINE.blacklistAddress(address,bool)._address (#737) is not in mixedCase
Parameter FIRSTVALENTINE.blacklistAddress(address,bool)._value (#737) is not in mixedCase
Parameter FIRSTVALENTINE.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._enabled (#839) is not in mixedCase
Parameter FIRSTVALENTINE.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._cap (#839) is not in mixedCase
Parameter FIRSTVALENTINE.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._amount (#839) is not in mixedCase
Parameter FIRSTVALENTINE.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._period (#839) is not in mixedCase
Parameter FIRSTVALENTINE.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._autoBuybackMultiplier (#839) is not in mixedCase
Parameter FIRSTVALENTINE.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._liquidityFeeBuy (#904) is not in mixedCase
Parameter FIRSTVALENTINE.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._buybackFeeBuy (#904) is not in mixedCase
Parameter FIRSTVALENTINE.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._reflectionFeeBuy (#904) is not in mixedCase
Parameter FIRSTVALENTINE.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._marketingFeeBuy (#904) is not in mixedCase
Parameter FIRSTVALENTINE.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._devFeeBuy (#904) is not in mixedCase
Parameter FIRSTVALENTINE.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._feeDenominator (#904) is not in mixedCase
Parameter FIRSTVALENTINE.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._liquidityFeeSell (#915) is not in mixedCase
Parameter FIRSTVALENTINE.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._buybackFeeSell (#915) is not in mixedCase
Parameter FIRSTVALENTINE.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._reflectionFeeSell (#915) is not in mixedCase
Parameter FIRSTVALENTINE.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._marketingFeeSell (#915) is not in mixedCase
Parameter FIRSTVALENTINE.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._devFeeSell (#915) is not in mixedCase
Parameter FIRSTVALENTINE.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._feeDenominator (#915) is not in mixedCase
Parameter FIRSTVALENTINE.setFeeReceivers(address,address)._autoLiquidityReceiver (#926) is not in mixedCase
Parameter FIRSTVALENTINE.setFeeReceivers(address,address)._marketingFeeReceiver (#926) is not in mixedCase
Parameter FIRSTVALENTINE.setSwapBackSettings(bool,uint256)._enabled (#932) is not in mixedCase
Parameter FIRSTVALENTINE.setSwapBackSettings(bool,uint256)._amount (#932) is not in mixedCase
Parameter FIRSTVALENTINE.setTargetLiquidity(uint256,uint256)._target (#938) is not in mixedCase
Parameter FIRSTVALENTINE.setTargetLiquidity(uint256,uint256)._denominator (#938) is not in mixedCase
Parameter FIRSTVALENTINE.setDistributionCriteria(uint256,uint256)._minPeriod (#950) is not in mixedCase
Parameter FIRSTVALENTINE.setDistributionCriteria(uint256,uint256)._minDistribution (#950) is not in mixedCase
Variable FIRSTVALENTINE.BUSD (#370) is not in mixedCase
Variable FIRSTVALENTINE.WBNB (#371) is not in mixedCase
Variable FIRSTVALENTINE.DEAD (#372) is not in mixedCase
Variable FIRSTVALENTINE.ZERO (#373) is not in mixedCase
Variable FIRSTVALENTINE.DEV (#374) is not in mixedCase
Constant FIRSTVALENTINE._name (#381) is not in UPPER_CASE_WITH_UNDERSCORES
Constant FIRSTVALENTINE._symbol (#382) is not in UPPER_CASE_WITH_UNDERSCORES
Constant FIRSTVALENTINE._decimals (#383) is not in UPPER_CASE_WITH_UNDERSCORES
Variable FIRSTVALENTINE._totalSupply (#386) is not in mixedCase
Variable FIRSTVALENTINE._maxBuyTxAmount (#389) is not in mixedCase
Variable FIRSTVALENTINE._maxSellTxAmount (#390) is not in mixedCase
Variable FIRSTVALENTINE._maxWalletToken (#391) is not in mixedCase
Variable FIRSTVALENTINE._balances (#394) is not in mixedCase
Variable FIRSTVALENTINE._allowances (#395) is not in mixedCase
Variable FIRSTVALENTINE.GREEDTriggeredAt (#428) is not in mixedCase
Variable FIRSTVALENTINE.GREEDDuration (#429) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Variable IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#148) is too similar to IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#149)
Prevent variables from having similar names.
Additional information: link
FIRSTVALENTINE.slitherConstructorVariables() (#366-988) uses literals with too many digits:
- DEAD = 0x000000000000000000000000000000000000dEaD (#372)
FIRSTVALENTINE.slitherConstructorVariables() (#366-988) uses literals with too many digits:
- ZERO = 0x0000000000000000000000000000000000000000 (#373)
FIRSTVALENTINE.slitherConstructorVariables() (#366-988) uses literals with too many digits:
- _totalSupply = 100000000 * (10 ** _decimals) (#386)
FIRSTVALENTINE.slitherConstructorVariables() (#366-988) uses literals with too many digits:
- distributorGas = 500000 (#456)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
FIRSTVALENTINE.BUSD (#370) is never used in FIRSTVALENTINE (#366-988)
Remove unused state variables.
Additional information: link
DividendDistributor.WBNB (#211) should be constant
DividendDistributor.dividendsPerShareAccuracyFactor (#224) should be constant
FIRSTVALENTINE.BUSD (#370) should be constant
FIRSTVALENTINE.DEAD (#372) should be constant
FIRSTVALENTINE.DEV (#374) should be constant
FIRSTVALENTINE.WBNB (#371) should be constant
FIRSTVALENTINE.ZERO (#373) should be constant
FIRSTVALENTINE._totalSupply (#386) should be constant
FIRSTVALENTINE.deadBlocks (#432) 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) (#98-100)
unauthorize(address) should be declared external:
- Auth.unauthorize(address) (#105-107)
transferOwnership(address) should be declared external:
- Auth.transferOwnership(address) (#126-130)
getTotalFee(bool) should be declared external:
- FIRSTVALENTINE.getTotalFee(bool) (#669-673)
tradingStatus(bool) should be declared external:
- FIRSTVALENTINE.tradingStatus(bool) (#714-717)
enableGREED(uint256) should be declared external:
- FIRSTVALENTINE.enableGREED(uint256) (#720-723)
cooldownEnabled(bool,uint8) should be declared external:
- FIRSTVALENTINE.cooldownEnabled(bool,uint8) (#731-734)
blacklistAddress(address,bool) should be declared external:
- FIRSTVALENTINE.blacklistAddress(address,bool) (#737-739)
getUnpaidEarnings(address) should be declared external:
- FIRSTVALENTINE.getUnpaidEarnings(address) (#960-962)
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
Telegram account has relatively few subscribers
Unable to find Blog account (Reddit or Medium)
Unable to find Youtube account
Unable to find Discord account