Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
BabyValentineElon.swapBack() (#516-560) sends eth to arbitrary user
Dangerous calls:
- (tmpSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#544)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in BabyValentineElon._transferFrom(address,address,uint256) (#446-479):
External calls:
- swapBack() (#458)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#527-533)
- distributor.deposit{value: amountBNBReflection}() (#543)
- (tmpSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#544)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#549-556)
External calls sending eth:
- swapBack() (#458)
- distributor.deposit{value: amountBNBReflection}() (#543)
- (tmpSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#544)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#549-556)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#461)
- _balances[recipient] = _balances[recipient].add(amountReceived) (#464)
- amountReceived = takeFee(sender,amount) (#463)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#498)
Apply the check-effects-interactions pattern.
Additional information: link
DividendDistributor.distributeDividend(address) (#289-300) ignores return value by REWARDS.transfer(shareholder,amount) (#295)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.
Additional information: link
Too many vulnerabilities (Unchecked transfer, Reentrancy vulnerability, etc.). High risk of a scam. DYOR & manual audit are advised.
BabyValentineElon.swapBack().tmpSuccess (#544) is written in both
(tmpSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#544)
tmpSuccess = false (#546)
Fix or remove the writes.
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.
Reentrancy in DividendDistributor.distributeDividend(address) (#289-300):
External calls:
- REWARDS.transfer(shareholder,amount) (#295)
State variables written after the call(s):
- shares[shareholder].totalRealised = shares[shareholder].totalRealised.add(amount) (#297)
- shares[shareholder].totalExcluded = getCumulativeDividends(shares[shareholder].amount) (#298)
Reentrancy in DividendDistributor.process(uint256) (#259-283):
External calls:
- distributeDividend(shareholders[currentIndex]) (#275)
- REWARDS.transfer(shareholder,amount) (#295)
State variables written after the call(s):
- currentIndex ++ (#280)
Reentrancy in DividendDistributor.setShare(address,uint256) (#223-237):
External calls:
- distributeDividend(shareholder) (#225)
- REWARDS.transfer(shareholder,amount) (#295)
State variables written after the call(s):
- shares[shareholder].amount = amount (#235)
- shares[shareholder].totalExcluded = getCumulativeDividends(shares[shareholder].amount) (#236)
Apply the check-effects-interactions pattern.
Additional information: link
BabyValentineElon.swapBack() (#516-560) ignores return value by router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#549-556)
Ensure that all the return values of the function calls are used.
Additional information: link
DividendDistributor.setDistributionCriteria(uint256,uint256) (#218-221) should emit an event for:
- minPeriod = _minPeriod (#219)
- minDistribution = _minDistribution (#220)
BabyValentineElon.setFees(uint256,uint256,uint256,uint256) (#580-587) should emit an event for:
- liquidityFee = _liquidityFee (#581)
- reflectionFee = _reflectionFee (#582)
- marketingFee = _marketingFee (#583)
- totalFee = _liquidityFee.add(_reflectionFee).add(_marketingFee) (#584)
BabyValentineElon.setSwapBackSettings(bool,uint256) (#596-599) should emit an event for:
- swapThreshold = _amount (#598)
Emit an event for critical parameter changes.
Additional information: link
Auth.transferOwnership(address).adr (#99) lacks a zero-check on :
- owner = adr (#100)
BabyValentineElon.setMarketingReciver(address)._marketingFeeReceiver (#589) lacks a zero-check on :
- marketingFeeReceiver = _marketingFeeReceiver (#590)
BabyValentineElon.setLiquidityReciver(address)._autoLiquidityReceiver (#592) lacks a zero-check on :
- autoLiquidityReceiver = _autoLiquidityReceiver (#593)
Check that the address is not zero.
Additional information: link
DividendDistributor.distributeDividend(address) (#289-300) has external calls inside a loop: REWARDS.transfer(shareholder,amount) (#295)
Favor pull over push strategy for external calls.
Additional information: link
Reentrancy in BabyValentineElon.constructor() (#385-407):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#387)
State variables written after the call(s):
- _allowances[address(this)][address(router)] = _totalSupply (#388)
- approve(routerv2,_totalSupply) (#403)
- _allowances[msg.sender][spender] = amount (#420)
- approve(address(pair),_totalSupply) (#404)
- _allowances[msg.sender][spender] = amount (#420)
- _balances[msg.sender] = _totalSupply (#405)
- autoLiquidityReceiver = DEAD (#400)
- distributor = new DividendDistributor(address(router)) (#390)
- isDividendExempt[pair] = true (#396)
- isDividendExempt[address(this)] = true (#397)
- isDividendExempt[DEAD] = true (#398)
- isFeeExempt[msg.sender] = true (#392)
- isTxLimitExempt[msg.sender] = true (#393)
- marketingFeeReceiver = _marketingAdr (#401)
Reentrancy in DividendDistributor.deposit() (#239-257):
External calls:
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: msg.value}(0,path,address(this),block.timestamp) (#246-251)
State variables written after the call(s):
- dividendsPerShare = dividendsPerShare.add(dividendsPerShareAccuracyFactor.mul(amount).div(totalShares)) (#256)
- totalDividends = totalDividends.add(amount) (#255)
Reentrancy in DividendDistributor.distributeDividend(address) (#289-300):
External calls:
- REWARDS.transfer(shareholder,amount) (#295)
State variables written after the call(s):
- shareholderClaims[shareholder] = block.timestamp (#296)
Reentrancy in DividendDistributor.setShare(address,uint256) (#223-237):
External calls:
- distributeDividend(shareholder) (#225)
- REWARDS.transfer(shareholder,amount) (#295)
State variables written after the call(s):
- addShareholder(shareholder) (#229)
- shareholderIndexes[shareholder] = shareholders.length (#321)
- removeShareholder(shareholder) (#231)
- shareholderIndexes[shareholders[shareholders.length - 1]] = shareholderIndexes[shareholder] (#327)
- addShareholder(shareholder) (#229)
- shareholders.push(shareholder) (#322)
- removeShareholder(shareholder) (#231)
- shareholders[shareholderIndexes[shareholder]] = shareholders[shareholders.length - 1] (#326)
- shareholders.pop() (#328)
- totalShares = totalShares.sub(shares[shareholder].amount).add(amount) (#234)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in BabyValentineElon._transferFrom(address,address,uint256) (#446-479):
External calls:
- swapBack() (#458)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#527-533)
- distributor.deposit{value: amountBNBReflection}() (#543)
- (tmpSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#544)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#549-556)
- distributor.setShare(sender,_balances[sender]) (#468)
- distributor.setShare(recipient,_balances[recipient]) (#472)
- distributor.process(distributorGas) (#475)
External calls sending eth:
- swapBack() (#458)
- distributor.deposit{value: amountBNBReflection}() (#543)
- (tmpSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#544)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#549-556)
Event emitted after the call(s):
- Transfer(sender,recipient,amountReceived) (#477)
Reentrancy in BabyValentineElon._transferFrom(address,address,uint256) (#446-479):
External calls:
- swapBack() (#458)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#527-533)
- distributor.deposit{value: amountBNBReflection}() (#543)
- (tmpSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#544)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#549-556)
External calls sending eth:
- swapBack() (#458)
- distributor.deposit{value: amountBNBReflection}() (#543)
- (tmpSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#544)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#549-556)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#499)
- amountReceived = takeFee(sender,amount) (#463)
Reentrancy in BabyValentineElon.constructor() (#385-407):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#387)
Event emitted after the call(s):
- Approval(msg.sender,spender,amount) (#421)
- approve(routerv2,_totalSupply) (#403)
- Approval(msg.sender,spender,amount) (#421)
- approve(address(pair),_totalSupply) (#404)
- Transfer(address(0),msg.sender,_totalSupply) (#406)
Reentrancy in BabyValentineElon.swapBack() (#516-560):
External calls:
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#527-533)
- distributor.deposit{value: amountBNBReflection}() (#543)
- (tmpSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#544)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#549-556)
External calls sending eth:
- distributor.deposit{value: amountBNBReflection}() (#543)
- (tmpSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#544)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#549-556)
Event emitted after the call(s):
- AutoLiquify(amountBNBLiquidity,amountToLiquify) (#558)
Apply the check-effects-interactions pattern.
Additional information: link
DividendDistributor.shouldDistribute(address) (#284-287) uses timestamp for comparisons
Dangerous comparisons:
- shareholderClaims[shareholder] + minPeriod < block.timestamp && getUnpaidEarnings(shareholder) > minDistribution (#285-286)
Avoid relying on block.timestamp.
Additional information: link
DividendDistributor.process(uint256) (#259-283) has costly operations inside a loop:
- currentIndex = 0 (#271)
DividendDistributor.distributeDividend(address) (#289-300) has costly operations inside a loop:
- totalDistributed = totalDistributed.add(amount) (#294)
DividendDistributor.process(uint256) (#259-283) has costly operations inside a loop:
- currentIndex ++ (#280)
Use a local variable to hold the loop computation result.
Additional information: link
BabyValentineElon._maxTxAmount (#349) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 100 / 100
BabyValentineElon._maxWalletToken (#352) is set pre-construction with a non-constant function or state variable:
- (_totalSupply * 3) / 100
BabyValentineElon.swapThreshold (#381) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 10 / 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 BabyValentineElon.swapBack() (#516-560):
- (tmpSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#544)
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() (#114) is not in mixedCase
Parameter DividendDistributor.setDistributionCriteria(uint256,uint256)._minPeriod (#218) is not in mixedCase
Parameter DividendDistributor.setDistributionCriteria(uint256,uint256)._minDistribution (#218) is not in mixedCase
Variable DividendDistributor._token (#170) is not in mixedCase
Variable DividendDistributor.REWARDS (#178) is not in mixedCase
Variable DividendDistributor.WBNB (#179) is not in mixedCase
Parameter BabyValentineElon.setFees(uint256,uint256,uint256,uint256)._liquidityFee (#580) is not in mixedCase
Parameter BabyValentineElon.setFees(uint256,uint256,uint256,uint256)._reflectionFee (#580) is not in mixedCase
Parameter BabyValentineElon.setFees(uint256,uint256,uint256,uint256)._marketingFee (#580) is not in mixedCase
Parameter BabyValentineElon.setMarketingReciver(address)._marketingFeeReceiver (#589) is not in mixedCase
Parameter BabyValentineElon.setLiquidityReciver(address)._autoLiquidityReceiver (#592) is not in mixedCase
Parameter BabyValentineElon.setSwapBackSettings(bool,uint256)._enabled (#596) is not in mixedCase
Parameter BabyValentineElon.setSwapBackSettings(bool,uint256)._amount (#596) is not in mixedCase
Parameter BabyValentineElon.setDistributionCriteria(uint256,uint256)._minPeriod (#601) is not in mixedCase
Parameter BabyValentineElon.setDistributionCriteria(uint256,uint256)._minDistribution (#601) is not in mixedCase
Variable BabyValentineElon.REWARDS (#335) is not in mixedCase
Variable BabyValentineElon.WBNB (#336) is not in mixedCase
Variable BabyValentineElon.DEAD (#339) is not in mixedCase
Variable BabyValentineElon.ZERO (#340) is not in mixedCase
Variable BabyValentineElon._marketingAdr (#342) is not in mixedCase
Constant BabyValentineElon._name (#344) is not in UPPER_CASE_WITH_UNDERSCORES
Constant BabyValentineElon._symbol (#345) is not in UPPER_CASE_WITH_UNDERSCORES
Constant BabyValentineElon._decimals (#346) is not in UPPER_CASE_WITH_UNDERSCORES
Variable BabyValentineElon._totalSupply (#348) is not in mixedCase
Variable BabyValentineElon._maxTxAmount (#349) is not in mixedCase
Variable BabyValentineElon._maxWalletToken (#352) is not in mixedCase
Variable BabyValentineElon._balances (#354) is not in mixedCase
Variable BabyValentineElon._allowances (#355) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Variable IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#119) is too similar to IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#120)
Prevent variables from having similar names.
Additional information: link
BabyValentineElon.slitherConstructorVariables() (#332-625) uses literals with too many digits:
- DEAD = 0x000000000000000000000000000000000000dEaD (#339)
BabyValentineElon.slitherConstructorVariables() (#332-625) uses literals with too many digits:
- ZERO = 0x0000000000000000000000000000000000000000 (#340)
BabyValentineElon.slitherConstructorVariables() (#332-625) uses literals with too many digits:
- distributorGas = 500000 (#378)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
BabyValentineElon.REWARDS (#335) is never used in BabyValentineElon (#332-625)
Remove unused state variables.
Additional information: link
BabyValentineElon.DEAD (#339) should be constant
BabyValentineElon.REWARDS (#335) should be constant
BabyValentineElon.WBNB (#336) should be constant
BabyValentineElon.ZERO (#340) should be constant
BabyValentineElon._marketingAdr (#342) should be constant
BabyValentineElon._totalSupply (#348) should be constant
BabyValentineElon.feeDenominator (#366) should be constant
BabyValentineElon.routerv2 (#337) should be constant
BabyValentineElon.targetLiquidity (#371) should be constant
BabyValentineElon.targetLiquidityDenominator (#372) should be constant
DividendDistributor.WBNB (#179) should be constant
DividendDistributor.dividendsPerShareAccuracyFactor (#193) 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) (#76-78)
transferOwnership(address) should be declared external:
- Auth.transferOwnership(address) (#99-103)
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