Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
BabyKevin.addLiquidity(uint256,uint256) (#713-725) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in BabyKevin._transfer(address,address,uint256) (#636-666):
External calls:
- swapAndLiquify(contractTokenBalance) (#656)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#703-709)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#656)
- recipient.transfer(amount) (#602)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
State variables written after the call(s):
- _rBalance[sender] = _rBalance[sender].sub(amount,Insufficient Balance) (#658)
- _rBalance[recipient] = _rBalance[recipient].add(finalAmount) (#662)
- finalAmount = takeFee(sender,recipient,amount) (#659)
- _rBalance[address(this)] = _rBalance[address(this)].add(feeAmount) (#745)
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 ($BKevin) 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.
BabyKevin.addLiquidity(uint256,uint256) (#713-725) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
Ensure that all the return values of the function calls are used.
Additional information: link
BabyKevin.allowance(address,address).owner (#495) shadows:
- Ownable.owner() (#152-154) (function)
BabyKevin._approve(address,address,uint256).owner (#516) shadows:
- Ownable.owner() (#152-154) (function)
Rename the local variables that shadow another component.
Additional information: link
BabyKevin.setBuyTaxes(uint256,uint256,uint256) (#536-542) should emit an event for:
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#541)
BabyKevin.setSelTaxes(uint256,uint256,uint256) (#544-550) should emit an event for:
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#549)
BabyKevin.setDistributionSettings(uint256,uint256,uint256) (#552-558) should emit an event for:
- _liquidityShare = newLiquidityShare (#553)
- _teamShare = newTeamShare (#555)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#557)
BabyKevin.setMaxTxAmount(uint256) (#560-562) should emit an event for:
- _maxTxAmount = maxTxAmount (#561)
BabyKevin.setWalletLimit(uint256) (#572-574) should emit an event for:
- _walletMax = newLimit (#573)
BabyKevin.setNumTokensBeforeSwap(uint256) (#576-578) should emit an event for:
- minimumTokensBeforeSwap = newLimit (#577)
Emit an event for critical parameter changes.
Additional information: link
BabyKevin.setMarketingWalletAddress(address).newAddress (#580) lacks a zero-check on :
- marketingWalletAddress = address(newAddress) (#581)
BabyKevin.setTeamWalletAddress(address).newAddress (#584) lacks a zero-check on :
- teamWalletAddress = address(newAddress) (#585)
Check that the address is not zero.
Additional information: link
Reentrancy in BabyKevin.changeRouterVersion(address) (#605-621):
External calls:
- newPairAddress = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#613)
State variables written after the call(s):
- isMarketPair[address(uniswapPair)] = true (#620)
- isWalletLimitExempt[address(uniswapPair)] = true (#619)
- uniswapPair = newPairAddress (#616)
- uniswapV2Router = _uniswapV2Router (#617)
Reentrancy in BabyKevin.constructor() (#459-488):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#464-465)
State variables written after the call(s):
- _allowances[address(this)][address(uniswapV2Router)] = _totalSupply (#468)
- _rBalance[_msgSender()] = _totalSupply (#486)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#475)
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#473)
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#474)
- isExcludedFromFee[owner()] = true (#470)
- isExcludedFromFee[address(this)] = true (#471)
- isMarketPair[address(uniswapPair)] = true (#484)
- isTxLimitExempt[owner()] = true (#481)
- isTxLimitExempt[address(this)] = true (#482)
- isWalletLimitExempt[owner()] = true (#477)
- isWalletLimitExempt[address(uniswapPair)] = true (#478)
- isWalletLimitExempt[address(this)] = true (#479)
- uniswapV2Router = _uniswapV2Router (#467)
Reentrancy in BabyKevin.swapAndLiquify(uint256) (#675-694):
External calls:
- swapTokensForEth(tokensForSwap) (#679)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#703-709)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#693)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#689)
- recipient.transfer(amount) (#602)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#691)
- recipient.transfer(amount) (#602)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#693)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#693)
- _allowances[owner][spender] = amount (#520)
Reentrancy in BabyKevin.transferFrom(address,address,uint256) (#630-634):
External calls:
- _transfer(sender,recipient,amount) (#631)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#703-709)
External calls sending eth:
- _transfer(sender,recipient,amount) (#631)
- recipient.transfer(amount) (#602)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#632)
- _allowances[owner][spender] = amount (#520)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in BabyKevin._transfer(address,address,uint256) (#636-666):
External calls:
- swapAndLiquify(contractTokenBalance) (#656)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#703-709)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#656)
- recipient.transfer(amount) (#602)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#746)
- finalAmount = takeFee(sender,recipient,amount) (#659)
- Transfer(sender,recipient,finalAmount) (#663)
Reentrancy in BabyKevin.constructor() (#459-488):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#464-465)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_totalSupply) (#487)
Reentrancy in BabyKevin.swapAndLiquify(uint256) (#675-694):
External calls:
- swapTokensForEth(tokensForSwap) (#679)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#703-709)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#693)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#689)
- recipient.transfer(amount) (#602)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#691)
- recipient.transfer(amount) (#602)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#693)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#521)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#693)
Reentrancy in BabyKevin.swapTokensForEth(uint256) (#696-711):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#703-709)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#710)
Reentrancy in BabyKevin.transferFrom(address,address,uint256) (#630-634):
External calls:
- _transfer(sender,recipient,amount) (#631)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#703-709)
External calls sending eth:
- _transfer(sender,recipient,amount) (#631)
- recipient.transfer(amount) (#602)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#521)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#632)
Apply the check-effects-interactions pattern.
Additional information: link
Address.isContract(address) (#84-93) uses assembly
- INLINE ASM (#91)
Address._functionCallWithValue(address,bytes,uint256,string) (#120-137) uses assembly
- INLINE ASM (#129-132)
Do not use evm assembly.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#120-137) is never used and should be removed
Address.functionCall(address,bytes) (#103-105) is never used and should be removed
Address.functionCall(address,bytes,string) (#107-109) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#111-113) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#115-118) is never used and should be removed
Address.isContract(address) (#84-93) is never used and should be removed
Address.sendValue(address,uint256) (#95-101) is never used and should be removed
Context._msgData() (#12-15) is never used and should be removed
SafeMath.mod(uint256,uint256) (#72-74) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#76-79) is never used and should be removed
Remove unused functions.
Additional information: link
BabyKevin._totalSupply (#420) is set pre-construction with a non-constant function or state variable:
- 100000000000 * 10 ** _decimals
BabyKevin._maxTxAmount (#421) is set pre-construction with a non-constant function or state variable:
- 100000000000 * 10 ** _decimals
BabyKevin._walletMax (#422) is set pre-construction with a non-constant function or state variable:
- 100000000000 * 10 ** _decimals
BabyKevin.minimumTokensBeforeSwap (#423) is set pre-construction with a non-constant function or state variable:
- 1 * 10 ** _decimals
BabyKevin._Deci (#424) is set pre-construction with a non-constant function or state variable:
- (10 ** _decimals)
BabyKevin._rSupply (#426) is set pre-construction with a non-constant function or state variable:
- _supplyAmount * _Deci
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 Address.sendValue(address,uint256) (#95-101):
- (success) = recipient.call{value: amount}() (#99)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#120-137):
- (success,returndata) = target.call{value: weiValue}(data) (#123)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#209) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#210) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#226) is not in mixedCase
Function IUniswapV2Router01.WETH() (#245) is not in mixedCase
Parameter BabyKevin.setSwapAndLiquifyEnabled(bool)._enabled (#588) is not in mixedCase
Parameter BabyKevin.setblocklist(address)._account (#727) is not in mixedCase
Variable BabyKevin._rBalance (#395) is not in mixedCase
Variable BabyKevin._buyLiquidityFee (#404) is not in mixedCase
Variable BabyKevin._buyMarketingFee (#405) is not in mixedCase
Variable BabyKevin._buyTeamFee (#406) is not in mixedCase
Variable BabyKevin._sellLiquidityFee (#408) is not in mixedCase
Variable BabyKevin._sellMarketingFee (#409) is not in mixedCase
Variable BabyKevin._sellTeamFee (#410) is not in mixedCase
Variable BabyKevin._liquidityShare (#412) is not in mixedCase
Variable BabyKevin._marketingShare (#413) is not in mixedCase
Variable BabyKevin._teamShare (#414) is not in mixedCase
Variable BabyKevin._totalTaxIfBuying (#416) is not in mixedCase
Variable BabyKevin._totalTaxIfSelling (#417) is not in mixedCase
Variable BabyKevin._totalDistributionShares (#418) is not in mixedCase
Variable BabyKevin._maxTxAmount (#421) is not in mixedCase
Variable BabyKevin._walletMax (#422) is not in mixedCase
Variable BabyKevin._Deci (#424) is not in mixedCase
Variable BabyKevin._supplyAmount (#425) is not in mixedCase
Variable BabyKevin._rSupply (#426) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#13)" inContext (#8-16)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in BabyKevin._transfer(address,address,uint256) (#636-666):
External calls:
- swapAndLiquify(contractTokenBalance) (#656)
- recipient.transfer(amount) (#602)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#656)
- recipient.transfer(amount) (#602)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
State variables written after the call(s):
- _rBalance[sender] = _rBalance[sender].sub(amount,Insufficient Balance) (#658)
- _rBalance[recipient] = _rBalance[recipient].add(finalAmount) (#662)
- finalAmount = takeFee(sender,recipient,amount) (#659)
- _rBalance[address(this)] = _rBalance[address(this)].add(feeAmount) (#745)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#746)
- finalAmount = takeFee(sender,recipient,amount) (#659)
- Transfer(sender,recipient,finalAmount) (#663)
Reentrancy in BabyKevin.swapAndLiquify(uint256) (#675-694):
External calls:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#689)
- recipient.transfer(amount) (#602)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#691)
- recipient.transfer(amount) (#602)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#689)
- recipient.transfer(amount) (#602)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#691)
- recipient.transfer(amount) (#602)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#693)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#693)
- _allowances[owner][spender] = amount (#520)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#521)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#693)
Reentrancy in BabyKevin.transferFrom(address,address,uint256) (#630-634):
External calls:
- _transfer(sender,recipient,amount) (#631)
- recipient.transfer(amount) (#602)
External calls sending eth:
- _transfer(sender,recipient,amount) (#631)
- recipient.transfer(amount) (#602)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#717-724)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#632)
- _allowances[owner][spender] = amount (#520)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#521)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#632)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#250) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#251)
Prevent variables from having similar names.
Additional information: link
Ownable.waiveOwnership() (#161-164) uses literals with too many digits:
- OwnershipTransferred(_owner,address(0x000000000000000000000000000000000000dEaD)) (#162)
Ownable.waiveOwnership() (#161-164) uses literals with too many digits:
- _owner = address(0x000000000000000000000000000000000000dEaD) (#163)
BabyKevin.slitherConstructorVariables() (#378-751) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#393)
BabyKevin.slitherConstructorVariables() (#378-751) uses literals with too many digits:
- _totalSupply = 100000000000 * 10 ** _decimals (#420)
BabyKevin.slitherConstructorVariables() (#378-751) uses literals with too many digits:
- _maxTxAmount = 100000000000 * 10 ** _decimals (#421)
BabyKevin.slitherConstructorVariables() (#378-751) uses literals with too many digits:
- _walletMax = 100000000000 * 10 ** _decimals (#422)
BabyKevin.slitherConstructorVariables() (#378-751) uses literals with too many digits:
- _supplyAmount = 100000000000000 (#425)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
Ownable._lockTime (#142) is never used in BabyKevin (#378-751)
Remove unused state variables.
Additional information: link
BabyKevin._decimals (#385) should be constant
BabyKevin._name (#383) should be constant
BabyKevin._supplyAmount (#425) should be constant
BabyKevin._symbol (#384) should be constant
Ownable._lockTime (#142) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
waiveOwnership() should be declared external:
- Ownable.waiveOwnership() (#161-164)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#166-170)
getTime() should be declared external:
- Ownable.getTime() (#172-174)
name() should be declared external:
- BabyKevin.name() (#490)
symbol() should be declared external:
- BabyKevin.symbol() (#491)
decimals() should be declared external:
- BabyKevin.decimals() (#492)
totalSupply() should be declared external:
- BabyKevin.totalSupply() (#493)
allowance(address,address) should be declared external:
- BabyKevin.allowance(address,address) (#495)
initializeAllowance() should be declared external:
- BabyKevin.initializeAllowance() (#496)
increaseAllowance(address,uint256) should be declared external:
- BabyKevin.increaseAllowance(address,uint256) (#497-500)
decreaseAllowance(address,uint256) should be declared external:
- BabyKevin.decreaseAllowance(address,uint256) (#502-505)
minimumTokensBeforeSwapAmount() should be declared external:
- BabyKevin.minimumTokensBeforeSwapAmount() (#507-509)
approve(address,uint256) should be declared external:
- BabyKevin.approve(address,uint256) (#511-514)
setMarketPairStatus(address,bool) should be declared external:
- BabyKevin.setMarketPairStatus(address,bool) (#524-526)
setIsExcludedFromFee(address,bool) should be declared external:
- BabyKevin.setIsExcludedFromFee(address,bool) (#532-534)
setSwapAndLiquifyEnabled(bool) should be declared external:
- BabyKevin.setSwapAndLiquifyEnabled(bool) (#588-591)
setSwapAndLiquifyByLimitOnly(bool) should be declared external:
- BabyKevin.setSwapAndLiquifyByLimitOnly(bool) (#593-595)
getCirculatingSupply() should be declared external:
- BabyKevin.getCirculatingSupply() (#597-599)
changeRouterVersion(address) should be declared external:
- BabyKevin.changeRouterVersion(address) (#605-621)
transfer(address,uint256) should be declared external:
- BabyKevin.transfer(address,uint256) (#624-628)
transferFrom(address,address,uint256) should be declared external:
- BabyKevin.transferFrom(address,address,uint256) (#630-634)
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