Cupid Billionaire aims to reward diamond holders. Stealth launch
Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
CupidBillionaireOfficial.swapBack() (#333-376) sends eth to arbitrary user
Dangerous calls:
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#368)
- (buybackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#370)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#372)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in CupidBillionaireOfficial._transferFrom(address,address,uint256) (#254-287):
External calls:
- swapBack() (#278)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,autoLiquidityReceiver,block.timestamp) (#380-387)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#351-357)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#368)
- (buybackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#370)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#372)
External calls sending eth:
- swapBack() (#278)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,autoLiquidityReceiver,block.timestamp) (#380-387)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#368)
- (buybackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#370)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#372)
State variables written after the call(s):
- _balances[sender] = _balances[sender] - amount (#280)
- _balances[recipient] = _balances[recipient] + amountReceived (#283)
- amountReceived = takeFee(recipient,amount) (#282)
- _balances[address(this)] = _balances[address(this)] + (feeAmount) (#320)
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.
Contract ticker (Cupid Billionaire) contains non-alphanumeric characters.
Not a direct threat, but may indicate unreliable intentions of developer. Non-alphanumeric chars (,.;!#*&") are extremely rare among low risk tokens.
Contract ticker (Cupid Billionaire) has length of 17 chars.
Not a direct threat, but may indicate unreliable intentions of developer.
CupidBillionaireOfficial.takeFee(address,uint256) (#317-324) performs a multiplication on the result of a division:
-feeAmount = amount / 100 * (totalFee) (#318)
CupidBillionaireOfficial.swapBack() (#333-376) performs a multiplication on the result of a division:
-amountToLiquify = tokensToSell / (totalFee) * (liquidityFee) / (2) (#342)
CupidBillionaireOfficial.setSwapBackSettings(bool,uint256,uint256) (#437-441) performs a multiplication on the result of a division:
-swapThreshold = _totalSupply / (10000) * (_percentage_min_base10000) (#439)
CupidBillionaireOfficial.setSwapBackSettings(bool,uint256,uint256) (#437-441) performs a multiplication on the result of a division:
-maxSwapSize = _totalSupply / (10000) * (_percentage_max_base10000) (#440)
CupidBillionaireOfficial.setMaxWalletPercent_base1000(uint256) (#451-453) performs a multiplication on the result of a division:
-_maxWalletToken = _totalSupply / (1000) * (maxWallPercent_base1000) (#452)
CupidBillionaireOfficial.setMaxBuyTxPercent_base1000(uint256) (#455-457) performs a multiplication on the result of a division:
-_maxBuyTxAmount = _totalSupply / (1000) * (maxBuyTXPercentage_base1000) (#456)
CupidBillionaireOfficial.setMaxSellTxPercent_base1000(uint256) (#459-461) performs a multiplication on the result of a division:
-_maxSellTxAmount = _totalSupply / (1000) * (maxSellTXPercentage_base1000) (#460)
CupidBillionaireOfficial.slitherConstructorVariables() (#154-476) performs a multiplication on the result of a division:
-_maxBuyTxAmount = _totalSupply / (100) * (2) (#190)
CupidBillionaireOfficial.slitherConstructorVariables() (#154-476) performs a multiplication on the result of a division:
-_maxSellTxAmount = _totalSupply / (100) * (1) (#191)
CupidBillionaireOfficial.slitherConstructorVariables() (#154-476) performs a multiplication on the result of a division:
-_maxWalletToken = _totalSupply / (100) * (3) (#192)
CupidBillionaireOfficial.slitherConstructorVariables() (#154-476) performs a multiplication on the result of a division:
-swapThreshold = _totalSupply / 1000 * 1 (#204)
CupidBillionaireOfficial.slitherConstructorVariables() (#154-476) performs a multiplication on the result of a division:
-maxSwapSize = _totalSupply / 100 * 1 (#205)
Consider ordering multiplication before division.
Additional information: link
CupidBillionaireOfficial.addLiquidity(uint256,uint256) (#378-390) ignores return value by router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,autoLiquidityReceiver,block.timestamp) (#380-387)
Ensure that all the return values of the function calls are used.
Additional information: link
CupidBillionaireOfficial.setBuyFees(uint256,uint256,uint256,uint256) (#414-420) should emit an event for:
- BuyliquidityFee = _liquidityFee (#415)
- BuybuybackFee = _buybackFee (#416)
- BuymarketingFee = _marketingFee (#417)
- BuydevFee = _devFee (#418)
- BuytotalFee = _liquidityFee + (_buybackFee) + (_marketingFee) + (_devFee) (#419)
CupidBillionaireOfficial.setSellFees(uint256,uint256,uint256,uint256) (#422-428) should emit an event for:
- SellliquidityFee = _liquidityFee (#423)
- SellbuybackFee = _buybackFee (#424)
- SellmarketingFee = _marketingFee (#425)
- SelldevFee = _devFee (#426)
- SelltotalFee = _liquidityFee + (_buybackFee) + (_marketingFee) + (_devFee) (#427)
CupidBillionaireOfficial.setSwapBackSettings(bool,uint256,uint256) (#437-441) should emit an event for:
- swapThreshold = _totalSupply / (10000) * (_percentage_min_base10000) (#439)
- maxSwapSize = _totalSupply / (10000) * (_percentage_max_base10000) (#440)
CupidBillionaireOfficial.setMaxWalletPercent_base1000(uint256) (#451-453) should emit an event for:
- _maxWalletToken = _totalSupply / (1000) * (maxWallPercent_base1000) (#452)
CupidBillionaireOfficial.setMaxBuyTxPercent_base1000(uint256) (#455-457) should emit an event for:
- _maxBuyTxAmount = _totalSupply / (1000) * (maxBuyTXPercentage_base1000) (#456)
CupidBillionaireOfficial.setMaxSellTxPercent_base1000(uint256) (#459-461) should emit an event for:
- _maxSellTxAmount = _totalSupply / (1000) * (maxSellTXPercentage_base1000) (#460)
Emit an event for critical parameter changes.
Additional information: link
CupidBillionaireOfficial.setFeeReceivers(address,address,address,address)._autoLiquidityReceiver (#430) lacks a zero-check on :
- autoLiquidityReceiver = _autoLiquidityReceiver (#431)
CupidBillionaireOfficial.setFeeReceivers(address,address,address,address)._marketingFeeReceiver (#430) lacks a zero-check on :
- marketingFeeReceiver = _marketingFeeReceiver (#432)
CupidBillionaireOfficial.setFeeReceivers(address,address,address,address)._buybackFeeReceiver (#430) lacks a zero-check on :
- buybackFeeReceiver = _buybackFeeReceiver (#433)
CupidBillionaireOfficial.setFeeReceivers(address,address,address,address)._devFeeReceiver (#430) lacks a zero-check on :
- devFeeReceiver = _devFeeReceiver (#434)
Check that the address is not zero.
Additional information: link
Reentrancy in CupidBillionaireOfficial.constructor() (#211-221):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#213)
State variables written after the call(s):
- _allowances[address(this)][address(router)] = type()(uint256).max (#214)
- _balances[msg.sender] = _totalSupply (#219)
- isFeeExempt[msg.sender] = true (#216)
- isTxLimitExempt[msg.sender] = true (#217)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in CupidBillionaireOfficial._transferFrom(address,address,uint256) (#254-287):
External calls:
- swapBack() (#278)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,autoLiquidityReceiver,block.timestamp) (#380-387)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#351-357)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#368)
- (buybackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#370)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#372)
External calls sending eth:
- swapBack() (#278)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,autoLiquidityReceiver,block.timestamp) (#380-387)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#368)
- (buybackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#370)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#372)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#321)
- amountReceived = takeFee(recipient,amount) (#282)
- Transfer(sender,recipient,amountReceived) (#285)
Reentrancy in CupidBillionaireOfficial.addLiquidity(uint256,uint256) (#378-390):
External calls:
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,autoLiquidityReceiver,block.timestamp) (#380-387)
Event emitted after the call(s):
- AutoLiquify(bnbAmount,tokenAmount) (#388)
Reentrancy in CupidBillionaireOfficial.constructor() (#211-221):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#213)
Event emitted after the call(s):
- Transfer(address(0),msg.sender,_totalSupply) (#220)
Reentrancy in CupidBillionaireOfficial.swapBack() (#333-376):
External calls:
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#351-357)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#368)
- (buybackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#370)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#372)
- addLiquidity(amountToLiquify,amountBNBLiquidity) (#375)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,autoLiquidityReceiver,block.timestamp) (#380-387)
External calls sending eth:
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#368)
- (buybackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#370)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#372)
- addLiquidity(amountToLiquify,amountBNBLiquidity) (#375)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,autoLiquidityReceiver,block.timestamp) (#380-387)
Event emitted after the call(s):
- AutoLiquify(bnbAmount,tokenAmount) (#388)
- addLiquidity(amountToLiquify,amountBNBLiquidity) (#375)
Apply the check-effects-interactions pattern.
Additional information: link
Ownable.Ownershipunlock() (#86-91) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp > _lockTime,Contract is locked) (#88)
Avoid relying on block.timestamp.
Additional information: link
CupidBillionaireOfficial.BuytotalFee (#181) is set pre-construction with a non-constant function or state variable:
- BuyliquidityFee + BuydevFee + BuymarketingFee + BuybuybackFee
CupidBillionaireOfficial.SelltotalFee (#187) is set pre-construction with a non-constant function or state variable:
- SellliquidityFee + SelldevFee + SellmarketingFee + SellbuybackFee
CupidBillionaireOfficial._maxBuyTxAmount (#190) is set pre-construction with a non-constant function or state variable:
- _totalSupply / (100) * (2)
CupidBillionaireOfficial._maxSellTxAmount (#191) is set pre-construction with a non-constant function or state variable:
- _totalSupply / (100) * (1)
CupidBillionaireOfficial._maxWalletToken (#192) is set pre-construction with a non-constant function or state variable:
- _totalSupply / (100) * (3)
CupidBillionaireOfficial.swapThreshold (#204) is set pre-construction with a non-constant function or state variable:
- _totalSupply / 1000 * 1
CupidBillionaireOfficial.maxSwapSize (#205) is set pre-construction with a non-constant function or state variable:
- _totalSupply / 100 * 1
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 CupidBillionaireOfficial.swapBack() (#333-376):
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#368)
- (buybackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#370)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#372)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Function Ownable.Ownershiplock(uint256) (#79-84) is not in mixedCase
Function Ownable.Ownershipunlock() (#86-91) is not in mixedCase
Function IDEXRouter.WETH() (#104) is not in mixedCase
Parameter CupidBillionaireOfficial.setBuyFees(uint256,uint256,uint256,uint256)._liquidityFee (#414) is not in mixedCase
Parameter CupidBillionaireOfficial.setBuyFees(uint256,uint256,uint256,uint256)._buybackFee (#414) is not in mixedCase
Parameter CupidBillionaireOfficial.setBuyFees(uint256,uint256,uint256,uint256)._marketingFee (#414) is not in mixedCase
Parameter CupidBillionaireOfficial.setBuyFees(uint256,uint256,uint256,uint256)._devFee (#414) is not in mixedCase
Parameter CupidBillionaireOfficial.setSellFees(uint256,uint256,uint256,uint256)._liquidityFee (#422) is not in mixedCase
Parameter CupidBillionaireOfficial.setSellFees(uint256,uint256,uint256,uint256)._buybackFee (#422) is not in mixedCase
Parameter CupidBillionaireOfficial.setSellFees(uint256,uint256,uint256,uint256)._marketingFee (#422) is not in mixedCase
Parameter CupidBillionaireOfficial.setSellFees(uint256,uint256,uint256,uint256)._devFee (#422) is not in mixedCase
Parameter CupidBillionaireOfficial.setFeeReceivers(address,address,address,address)._autoLiquidityReceiver (#430) is not in mixedCase
Parameter CupidBillionaireOfficial.setFeeReceivers(address,address,address,address)._marketingFeeReceiver (#430) is not in mixedCase
Parameter CupidBillionaireOfficial.setFeeReceivers(address,address,address,address)._buybackFeeReceiver (#430) is not in mixedCase
Parameter CupidBillionaireOfficial.setFeeReceivers(address,address,address,address)._devFeeReceiver (#430) is not in mixedCase
Parameter CupidBillionaireOfficial.setSwapBackSettings(bool,uint256,uint256)._enabled (#437) is not in mixedCase
Parameter CupidBillionaireOfficial.setSwapBackSettings(bool,uint256,uint256)._percentage_min_base10000 (#437) is not in mixedCase
Parameter CupidBillionaireOfficial.setSwapBackSettings(bool,uint256,uint256)._percentage_max_base10000 (#437) is not in mixedCase
Function CupidBillionaireOfficial.setMaxWalletPercent_base1000(uint256) (#451-453) is not in mixedCase
Parameter CupidBillionaireOfficial.setMaxWalletPercent_base1000(uint256).maxWallPercent_base1000 (#451) is not in mixedCase
Function CupidBillionaireOfficial.setMaxBuyTxPercent_base1000(uint256) (#455-457) is not in mixedCase
Parameter CupidBillionaireOfficial.setMaxBuyTxPercent_base1000(uint256).maxBuyTXPercentage_base1000 (#455) is not in mixedCase
Function CupidBillionaireOfficial.setMaxSellTxPercent_base1000(uint256) (#459-461) is not in mixedCase
Parameter CupidBillionaireOfficial.setMaxSellTxPercent_base1000(uint256).maxSellTXPercentage_base1000 (#459) is not in mixedCase
Variable CupidBillionaireOfficial.WBNB (#156) is not in mixedCase
Variable CupidBillionaireOfficial.DEAD (#157) is not in mixedCase
Variable CupidBillionaireOfficial.ZERO (#158) is not in mixedCase
Constant CupidBillionaireOfficial._name (#160) is not in UPPER_CASE_WITH_UNDERSCORES
Constant CupidBillionaireOfficial._symbol (#161) is not in UPPER_CASE_WITH_UNDERSCORES
Constant CupidBillionaireOfficial._decimals (#162) is not in UPPER_CASE_WITH_UNDERSCORES
Variable CupidBillionaireOfficial._totalSupply (#163) is not in mixedCase
Variable CupidBillionaireOfficial._balances (#165) is not in mixedCase
Variable CupidBillionaireOfficial._allowances (#166) is not in mixedCase
Variable CupidBillionaireOfficial.BuyliquidityFee (#177) is not in mixedCase
Variable CupidBillionaireOfficial.BuydevFee (#178) is not in mixedCase
Variable CupidBillionaireOfficial.BuymarketingFee (#179) is not in mixedCase
Variable CupidBillionaireOfficial.BuybuybackFee (#180) is not in mixedCase
Variable CupidBillionaireOfficial.BuytotalFee (#181) is not in mixedCase
Variable CupidBillionaireOfficial.SellliquidityFee (#183) is not in mixedCase
Variable CupidBillionaireOfficial.SelldevFee (#184) is not in mixedCase
Variable CupidBillionaireOfficial.SellmarketingFee (#185) is not in mixedCase
Variable CupidBillionaireOfficial.SellbuybackFee (#186) is not in mixedCase
Variable CupidBillionaireOfficial.SelltotalFee (#187) is not in mixedCase
Variable CupidBillionaireOfficial._maxBuyTxAmount (#190) is not in mixedCase
Variable CupidBillionaireOfficial._maxSellTxAmount (#191) is not in mixedCase
Variable CupidBillionaireOfficial._maxWalletToken (#192) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Variable IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#109) is too similar to IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#110)
Variable CupidBillionaireOfficial.setSwapBackSettings(bool,uint256,uint256)._percentage_max_base10000 (#437) is too similar to CupidBillionaireOfficial.setSwapBackSettings(bool,uint256,uint256)._percentage_min_base10000 (#437)
Prevent variables from having similar names.
Additional information: link
CupidBillionaireOfficial.slitherConstructorVariables() (#154-476) uses literals with too many digits:
- DEAD = 0x000000000000000000000000000000000000dEaD (#157)
CupidBillionaireOfficial.slitherConstructorVariables() (#154-476) uses literals with too many digits:
- ZERO = 0x0000000000000000000000000000000000000000 (#158)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
CupidBillionaireOfficial.ZERO (#158) is never used in CupidBillionaireOfficial (#154-476)
Remove unused state variables.
Additional information: link
CupidBillionaireOfficial.DEAD (#157) should be constant
CupidBillionaireOfficial.WBNB (#156) should be constant
CupidBillionaireOfficial.ZERO (#158) should be constant
CupidBillionaireOfficial._totalSupply (#163) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#69-73)
getUnlockTime() should be declared external:
- Ownable.getUnlockTime() (#75-77)
Ownershiplock(uint256) should be declared external:
- Ownable.Ownershiplock(uint256) (#79-84)
Ownershipunlock() should be declared external:
- Ownable.Ownershipunlock() (#86-91)
rescueToken(address,uint256) should be declared external:
- CupidBillionaireOfficial.rescueToken(address,uint256) (#464-466)
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