BabyAbortionInu.swapBack(address) (#667-731) sends eth to arbitrary user
Dangerous calls:
- (tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in BabyAbortionInu._transferFrom(address,address,uint256) (#600-639):
External calls:
- swapBack(sender) (#607)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#685-691)
- dividendDistributor.deposit{value: amountBNBReflection}() (#714)
- (tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#721-728)
External calls sending eth:
- swapBack(sender) (#607)
- dividendDistributor.deposit{value: amountBNBReflection}() (#714)
- (tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#721-728)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#616)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#624)
- finalAmount = takeFee(sender,recipient,amount) (#623)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#661)
Apply the check-effects-interactions pattern.
Additional information: link
DividendDistributor.distributeDividend(address) (#248-260) ignores return value by RewardToken.transfer(shareholder,amount) (#254)
DividendDistributor.rescueDividends(address) (#266-268) ignores return value by RewardToken.transfer(to,RewardToken.balanceOf(address(this))) (#267)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.
Additional information: link
Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
Too many vulnerabilities (Unchecked transfer, Reentrancy vulnerability, etc.). High risk of a scam. DYOR & manual audit are advised.
BabyAbortionInu.swapBack(address).tmpSuccess (#716) is written in both
(tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
tmpSuccess = false (#718)
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.
BabyAbortionInu.swapBack(address) (#667-731) performs a multiplication on the result of a division:
-amountBNBMarketing = amountBNB.mul(marketingFee / 2).div(totalBNBFee) (#700)
Consider ordering multiplication before division.
Additional information: link
Reentrancy in DividendDistributor.distributeDividend(address) (#248-260):
External calls:
- RewardToken.transfer(shareholder,amount) (#254)
State variables written after the call(s):
- shares[shareholder].totalRealised = shares[shareholder].totalRealised.add(amount) (#256)
- shares[shareholder].totalExcluded = getCumulativeDividends(shares[shareholder].amount) (#257)
Reentrancy in DividendDistributor.process(uint256) (#219-241):
External calls:
- distributeDividend(shareholders[currentIndex]) (#233)
- RewardToken.transfer(shareholder,amount) (#254)
State variables written after the call(s):
- currentIndex ++ (#238)
Reentrancy in DividendDistributor.setShare(address,uint256) (#182-197):
External calls:
- distributeDividend(shareholder) (#185)
- RewardToken.transfer(shareholder,amount) (#254)
State variables written after the call(s):
- shares[shareholder].amount = amount (#195)
- shares[shareholder].totalExcluded = getCumulativeDividends(shares[shareholder].amount) (#196)
Apply the check-effects-interactions pattern.
Additional information: link
BabyAbortionInu.swapBack(address) (#667-731) ignores return value by router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#721-728)
Ensure that all the return values of the function calls are used.
Additional information: link
DividendDistributor.setDistributionCriteria(uint256,uint256) (#177-180) should emit an event for:
- minPeriod = newMinPeriod (#178)
- minDistribution = newMinDistribution (#179)
BabyAbortionInu.changeTxLimit(uint256) (#521-523) should emit an event for:
- _maxTxAmount = newLimit (#522)
BabyAbortionInu.changeFees(uint256,uint256,uint256) (#552-561) should emit an event for:
- liquidityFee = newLiqFee (#553)
- rewardsFee = newRewardFee (#554)
- marketingFee = newMarketingFee (#555)
- totalFee = liquidityFee.add(marketingFee).add(rewardsFee) (#557)
- totalFeeIfSelling = totalFee (#558)
- totalFeeForPopcornPerk = liquidityFee.add(marketingFee / 1).add(rewardsFee) (#559)
- totalFeeForPopcornPerkIfSelling = totalFeeForPopcornPerk (#560)
Emit an event for critical parameter changes.
Additional information: link
Auth.transferOwnership(address).adr (#356) lacks a zero-check on :
- owner = adr (#357)
BabyAbortionInu.changeFeeReceivers(address,address).newLiquidityReceiver (#563) lacks a zero-check on :
- autoLiquidityReceiver = newLiquidityReceiver (#564)
BabyAbortionInu.changeFeeReceivers(address,address).newMarketingWallet (#563) lacks a zero-check on :
- marketingWallet = newMarketingWallet (#565)
Check that the address is not zero.
Additional information: link
DividendDistributor.distributeDividend(address) (#248-260) has external calls inside a loop: RewardToken.transfer(shareholder,amount) (#254)
Favor pull over push strategy for external calls.
Additional information: link
Reentrancy in BabyAbortionInu._transferFrom(address,address,uint256) (#600-639):
External calls:
- swapBack(sender) (#607)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#685-691)
- dividendDistributor.deposit{value: amountBNBReflection}() (#714)
- (tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#721-728)
External calls sending eth:
- swapBack(sender) (#607)
- dividendDistributor.deposit{value: amountBNBReflection}() (#714)
- (tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#721-728)
State variables written after the call(s):
- launch() (#611)
- launchedAt = block.number (#518)
Reentrancy in BabyAbortionInu.constructor() (#436-477):
External calls:
- pair = IDEXFactory(router.factory()).createPair(router.WETH(),address(this)) (#439)
State variables written after the call(s):
- _allowances[address(this)][address(router)] = uint256(- 1) (#440)
- _balances[msg.sender] = _totalSupply (#475)
- autoLiquidityReceiver = msg.sender (#465)
- dividendDistributor = new DividendDistributor(address(router)) (#442)
- isDividendExempt[pair] = true (#453)
- isDividendExempt[msg.sender] = true (#454)
- isDividendExempt[address(this)] = true (#455)
- isDividendExempt[DEAD] = true (#456)
- isDividendExempt[ZERO] = true (#457)
- isDividendExempt[Sixtynine] = true (#458)
- isFeeExempt[msg.sender] = true (#444)
- isFeeExempt[address(this)] = true (#445)
- isTxLimitExempt[msg.sender] = true (#447)
- isTxLimitExempt[pair] = true (#448)
- isTxLimitExempt[DEAD] = true (#449)
- isTxLimitExempt[ZERO] = true (#450)
- isTxLimitExempt[Sixtynine] = true (#451)
- iswalletMaxExempt[DEAD] = true (#460)
- iswalletMaxExempt[ZERO] = true (#461)
- iswalletMaxExempt[msg.sender] = true (#462)
- iswalletMaxExempt[Sixtynine] = true (#463)
- marketingWallet = 0x8c4296DC8C288f47c0047245B6AA4dbb04AE9B6A (#466)
- totalFee = liquidityFee.add(marketingFee).add(rewardsFee) (#469)
- totalFeeForPopcornPerk = liquidityFee.add(marketingFee / 2).add(rewardsFee) (#472)
- totalFeeForPopcornPerkIfSelling = totalFeeForPopcornPerk (#473)
- totalFeeIfSelling = totalFee (#470)
Reentrancy in DividendDistributor.deposit() (#199-217):
External calls:
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: msg.value}(0,path,address(this),block.timestamp) (#207-212)
State variables written after the call(s):
- dividendsPerShare = dividendsPerShare.add(dividendsPerShareAccuracyFactor.mul(amount).div(totalShares)) (#216)
- totalDividends = totalDividends.add(amount) (#215)
Reentrancy in DividendDistributor.distributeDividend(address) (#248-260):
External calls:
- RewardToken.transfer(shareholder,amount) (#254)
State variables written after the call(s):
- shareholderClaims[shareholder] = block.timestamp (#255)
Reentrancy in DividendDistributor.setShare(address,uint256) (#182-197):
External calls:
- distributeDividend(shareholder) (#185)
- RewardToken.transfer(shareholder,amount) (#254)
State variables written after the call(s):
- addShareholder(shareholder) (#189)
- shareholderIndexes[shareholder] = shareholders.length (#290)
- removeShareholder(shareholder) (#191)
- shareholderIndexes[shareholders[shareholders.length - 1]] = shareholderIndexes[shareholder] (#296)
- addShareholder(shareholder) (#189)
- shareholders.push(shareholder) (#291)
- removeShareholder(shareholder) (#191)
- shareholders[shareholderIndexes[shareholder]] = shareholders[shareholders.length - 1] (#295)
- shareholders.pop() (#297)
- totalShares = totalShares.sub(shares[shareholder].amount).add(amount) (#194)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in BabyAbortionInu._transferFrom(address,address,uint256) (#600-639):
External calls:
- swapBack(sender) (#607)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#685-691)
- dividendDistributor.deposit{value: amountBNBReflection}() (#714)
- (tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#721-728)
- dividendDistributor.setShare(sender,_balances[sender]) (#628)
- dividendDistributor.setShare(recipient,_balances[recipient]) (#632)
- dividendDistributor.process(distributorGas) (#635)
External calls sending eth:
- swapBack(sender) (#607)
- dividendDistributor.deposit{value: amountBNBReflection}() (#714)
- (tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#721-728)
Event emitted after the call(s):
- Transfer(sender,recipient,finalAmount) (#637)
Reentrancy in BabyAbortionInu._transferFrom(address,address,uint256) (#600-639):
External calls:
- swapBack(sender) (#607)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#685-691)
- dividendDistributor.deposit{value: amountBNBReflection}() (#714)
- (tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#721-728)
External calls sending eth:
- swapBack(sender) (#607)
- dividendDistributor.deposit{value: amountBNBReflection}() (#714)
- (tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#721-728)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#662)
- finalAmount = takeFee(sender,recipient,amount) (#623)
Reentrancy in BabyAbortionInu.constructor() (#436-477):
External calls:
- pair = IDEXFactory(router.factory()).createPair(router.WETH(),address(this)) (#439)
Event emitted after the call(s):
- Transfer(address(0),msg.sender,_totalSupply) (#476)
Reentrancy in BabyAbortionInu.swapBack(address) (#667-731):
External calls:
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#685-691)
- dividendDistributor.deposit{value: amountBNBReflection}() (#714)
- (tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#721-728)
External calls sending eth:
- dividendDistributor.deposit{value: amountBNBReflection}() (#714)
- (tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#721-728)
Event emitted after the call(s):
- AutoLiquify(amountBNBLiquidity,amountToLiquify) (#729)
Apply the check-effects-interactions pattern.
Additional information: link
DividendDistributor.shouldDistribute(address) (#243-246) uses timestamp for comparisons
Dangerous comparisons:
- shareholderClaims[shareholder] + minPeriod < block.timestamp && getUnpaidEarnings(shareholder) > minDistribution (#244-245)
Avoid relying on block.timestamp.
Additional information: link
DividendDistributor.process(uint256) (#219-241) has costly operations inside a loop:
- currentIndex = 0 (#230)
DividendDistributor.distributeDividend(address) (#248-260) has costly operations inside a loop:
- totalDistributed = totalDistributed.add(amount) (#253)
DividendDistributor.process(uint256) (#219-241) has costly operations inside a loop:
- currentIndex ++ (#238)
Use a local variable to hold the loop computation result.
Additional information: link
BabyAbortionInu.checkTxLimit(address,uint256) (#525-527) is never used and should be removed
Remove unused functions.
Additional information: link
BabyAbortionInu._maxTxAmount (#389) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 20 / 1000
BabyAbortionInu._walletMax (#390) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 20 / 1000
BabyAbortionInu.swapThreshold (#428) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 5 / 2000
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
Pragma version^0.7.4 (#5) allows old versions
Deploy with any of the following Solidity versions: 0.5.16 - 0.5.17, 0.6.11 - 0.6.12, 0.7.5 - 0.7.6 Use a simple pragma version that allows any of these versions. Consider using the latest version of Solidity for testing.
Additional information: link
Low level call in BabyAbortionInu.swapBack(address) (#667-731):
- (tmpSuccess) = address(marketingWallet).call{gas: 30000,value: amountBNBMarketing}() (#716)
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() (#68) is not in mixedCase
Parameter DividendDistributor.setRewardToken(address)._rewardToken (#270) is not in mixedCase
Variable DividendDistributor._token (#124) is not in mixedCase
Variable DividendDistributor.RewardToken (#135) is not in mixedCase
Variable DividendDistributor.DEAD (#141) is not in mixedCase
Variable DividendDistributor.ZERO (#142) is not in mixedCase
Variable DividendDistributor.Sixtynine (#143) is not in mixedCase
Parameter BabyAbortionInu.popcornBalance(address)._addressToQuery (#494) is not in mixedCase
Parameter BabyAbortionInu.setRewardToken(address)._rewardToken (#584) is not in mixedCase
Constant BabyAbortionInu._name (#373) is not in UPPER_CASE_WITH_UNDERSCORES
Constant BabyAbortionInu._symbol (#374) is not in UPPER_CASE_WITH_UNDERSCORES
Constant BabyAbortionInu._decimals (#375) is not in UPPER_CASE_WITH_UNDERSCORES
Variable BabyAbortionInu.DEAD (#377) is not in mixedCase
Variable BabyAbortionInu.ZERO (#378) is not in mixedCase
Variable BabyAbortionInu.Sixtynine (#379) is not in mixedCase
Variable BabyAbortionInu.RewardToken (#383) is not in mixedCase
Variable BabyAbortionInu._totalSupply (#388) is not in mixedCase
Variable BabyAbortionInu._maxTxAmount (#389) is not in mixedCase
Variable BabyAbortionInu._walletMax (#390) is not in mixedCase
Variable BabyAbortionInu._balances (#393) is not in mixedCase
Variable BabyAbortionInu._allowances (#394) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Variable IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#73) is too similar to IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#74)
Prevent variables from having similar names.
Additional information: link
DividendDistributor.slitherConstructorVariables() (#121-300) uses literals with too many digits:
- DEAD = 0x000000000000000000000000000000000000dEaD (#141)
DividendDistributor.slitherConstructorVariables() (#121-300) uses literals with too many digits:
- ZERO = 0x0000000000000000000000000000000000000000 (#142)
DividendDistributor.slitherConstructorVariables() (#121-300) uses literals with too many digits:
- Sixtynine = 0x0000000000000000000000000000000000000069 (#143)
BabyAbortionInu.swapBack(address) (#667-731) uses literals with too many digits:
- popcornBalance(sender) > 100000000 (#698)
BabyAbortionInu.slitherConstructorVariables() (#369-736) uses literals with too many digits:
- DEAD = 0x000000000000000000000000000000000000dEaD (#377)
BabyAbortionInu.slitherConstructorVariables() (#369-736) uses literals with too many digits:
- ZERO = 0x0000000000000000000000000000000000000000 (#378)
BabyAbortionInu.slitherConstructorVariables() (#369-736) uses literals with too many digits:
- Sixtynine = 0x0000000000000000000000000000000000000069 (#379)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
DividendDistributor.popcorn (#138) is never used in DividendDistributor (#121-300)
DividendDistributor.DEAD (#141) is never used in DividendDistributor (#121-300)
DividendDistributor.ZERO (#142) is never used in DividendDistributor (#121-300)
DividendDistributor.Sixtynine (#143) is never used in DividendDistributor (#121-300)
BabyAbortionInu.RewardToken (#383) is never used in BabyAbortionInu (#369-736)
Remove unused state variables.
Additional information: link
Contract ticker ($BABYBORT) 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.
BabyAbortionInu.DEAD (#377) should be constant
BabyAbortionInu.RewardToken (#383) should be constant
BabyAbortionInu.Sixtynine (#379) should be constant
BabyAbortionInu.ZERO (#378) should be constant
BabyAbortionInu._totalSupply (#388) should be constant
BabyAbortionInu.popcorn (#385) should be constant
BabyAbortionInu.restrictWhales (#391) should be constant
BabyAbortionInu.routerAddress (#381) should be constant
BabyAbortionInu.tradingOpen (#419) should be constant
DividendDistributor.DEAD (#141) should be constant
DividendDistributor.Sixtynine (#143) should be constant
DividendDistributor.ZERO (#142) should be constant
DividendDistributor.dividendsPerShareAccuracyFactor (#154) should be constant
DividendDistributor.routerAddress (#133) 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) (#328-330)
unauthorize(address) should be declared external:
- Auth.unauthorize(address) (#335-337)
transferOwnership(address) should be declared external:
- Auth.transferOwnership(address) (#356-360)
balanceOf(address) should be declared external:
- BEP20Interface.balanceOf(address) (#366)
getCirculatingSupply() should be declared external:
- BabyAbortionInu.getCirculatingSupply() (#487-489)
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