Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
VladimirPutINU.swapBack() (#376-410) sends eth to arbitrary user
Dangerous calls:
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#402)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#404)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in VladimirPutINU._transferFrom(address,address,uint256) (#310-330):
External calls:
- swapBack() (#319)
- router.addLiquidityETH{value: BNBAmount}(address(this),tokenAmount,0,0,address(this),block.timestamp) (#414-421)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#387-393)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#402)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#404)
- (DevSuccess) = address(DevFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#406)
External calls sending eth:
- swapBack() (#319)
- router.addLiquidityETH{value: BNBAmount}(address(this),tokenAmount,0,0,address(this),block.timestamp) (#414-421)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#402)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#404)
- (DevSuccess) = address(DevFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#406)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#323)
- _balances[recipient] = _balances[recipient].add(amountReceived) (#326)
- amountReceived = takeFee(sender,recipient,amount) (#325)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#363)
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.
VladimirPutINU.slitherConstructorVariables() (#223-509) uses literals with too many digits:
- DEAD = 0x000000000000000000000000000000000000dEaD (#227)
VladimirPutINU.slitherConstructorVariables() (#223-509) uses literals with too many digits:
- ZERO = 0x0000000000000000000000000000000000000000 (#228)
VladimirPutINU.slitherConstructorVariables() (#223-509) uses literals with too many digits:
- _totalSupply = 100000000000 * (10 ** _decimals) (#234)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
VladimirPutINU.DEAD (#227) should be constant
VladimirPutINU.DevFeeReceiver (#253) should be constant
VladimirPutINU.WBNB (#226) should be constant
VladimirPutINU.ZERO (#228) should be constant
VladimirPutINU._totalSupply (#234) 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) (#134-136)
unauthorize(address) should be declared external:
- Auth.unauthorize(address) (#141-143)
transferOwnership(address) should be declared external:
- Auth.transferOwnership(address) (#162-166)
transferForeignToken(address) should be declared external:
- VladimirPutINU.transferForeignToken(address) (#489-493)
isOverLiquified(uint256,uint256) should be declared external:
- VladimirPutINU.isOverLiquified(uint256,uint256) (#503-505)
Use the external attribute for functions never called from the contract.
Additional information: link
VladimirPutINU.slitherConstructorVariables() (#223-509) performs a multiplication on the result of a division:
-swapThreshold = _totalSupply / 1000 * 3 (#261)
Consider ordering multiplication before division.
Additional information: link
VladimirPutINU.addLiquidity(uint256,uint256) (#412-424) ignores return value by router.addLiquidityETH{value: BNBAmount}(address(this),tokenAmount,0,0,address(this),block.timestamp) (#414-421)
Ensure that all the return values of the function calls are used.
Additional information: link
VladimirPutINU.setTxLimit(uint256) (#447-450) should emit an event for:
- _maxTxAmount = amount (#449)
VladimirPutINU.setFees(uint256,uint256,uint256,uint256,uint256) (#465-472) should emit an event for:
- liquidityFee = _liquidityFee (#466)
- buybackFee = _buybackFee (#468)
- marketingFee = _marketingFee (#469)
- totalFee = _liquidityFee.add(_buybackFee).add(_marketingFee) (#470)
- feeDenominator = _feeDenominator (#471)
Emit an event for critical parameter changes.
Additional information: link
Auth.transferOwnership(address).adr (#162) lacks a zero-check on :
- owner = adr (#163)
VladimirPutINU.setFeeReceiver(address,address)._marketingFeeReceiver (#474) lacks a zero-check on :
- marketingFeeReceiver = _marketingFeeReceiver (#475)
VladimirPutINU.setFeeReceiver(address,address)._buybackFeeReceiver (#474) lacks a zero-check on :
- buybackFeeReceiver = _buybackFeeReceiver (#476)
Check that the address is not zero.
Additional information: link
Reentrancy in VladimirPutINU._transferFrom(address,address,uint256) (#310-330):
External calls:
- swapBack() (#319)
- router.addLiquidityETH{value: BNBAmount}(address(this),tokenAmount,0,0,address(this),block.timestamp) (#414-421)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#387-393)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#402)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#404)
- (DevSuccess) = address(DevFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#406)
External calls sending eth:
- swapBack() (#319)
- router.addLiquidityETH{value: BNBAmount}(address(this),tokenAmount,0,0,address(this),block.timestamp) (#414-421)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#402)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#404)
- (DevSuccess) = address(DevFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#406)
State variables written after the call(s):
- launch() (#321)
- launchedAt = block.number (#444)
Reentrancy in VladimirPutINU.constructor() (#265-276):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#267)
State variables written after the call(s):
- _allowances[address(this)][address(router)] = type()(uint256).max (#268)
- _balances[_owner] = _totalSupply (#274)
- isFeeExempt[_owner] = true (#271)
- isTxLimitExempt[_owner] = true (#272)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in VladimirPutINU._transferFrom(address,address,uint256) (#310-330):
External calls:
- swapBack() (#319)
- router.addLiquidityETH{value: BNBAmount}(address(this),tokenAmount,0,0,address(this),block.timestamp) (#414-421)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#387-393)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#402)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#404)
- (DevSuccess) = address(DevFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#406)
External calls sending eth:
- swapBack() (#319)
- router.addLiquidityETH{value: BNBAmount}(address(this),tokenAmount,0,0,address(this),block.timestamp) (#414-421)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#402)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#404)
- (DevSuccess) = address(DevFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#406)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#364)
- amountReceived = takeFee(sender,recipient,amount) (#325)
- Transfer(sender,recipient,amountReceived) (#328)
Reentrancy in VladimirPutINU.addLiquidity(uint256,uint256) (#412-424):
External calls:
- router.addLiquidityETH{value: BNBAmount}(address(this),tokenAmount,0,0,address(this),block.timestamp) (#414-421)
Event emitted after the call(s):
- AutoLiquify(BNBAmount,tokenAmount) (#422)
Reentrancy in VladimirPutINU.constructor() (#265-276):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#267)
Event emitted after the call(s):
- Transfer(address(0),_owner,_totalSupply) (#275)
Reentrancy in VladimirPutINU.swapBack() (#376-410):
External calls:
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#387-393)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#402)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#404)
- (DevSuccess) = address(DevFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#406)
- addLiquidity(amountToLiquify,amountBNBLiquidity) (#409)
- router.addLiquidityETH{value: BNBAmount}(address(this),tokenAmount,0,0,address(this),block.timestamp) (#414-421)
External calls sending eth:
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#402)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#404)
- (DevSuccess) = address(DevFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#406)
- addLiquidity(amountToLiquify,amountBNBLiquidity) (#409)
- router.addLiquidityETH{value: BNBAmount}(address(this),tokenAmount,0,0,address(this),block.timestamp) (#414-421)
Event emitted after the call(s):
- AutoLiquify(BNBAmount,tokenAmount) (#422)
- addLiquidity(amountToLiquify,amountBNBLiquidity) (#409)
Apply the check-effects-interactions pattern.
Additional information: link
VladimirPutINU.AntiDumpMultiplier() (#353-358) uses timestamp for comparisons
Dangerous comparisons:
- time_since_start > 1 * hour (#356)
Avoid relying on block.timestamp.
Additional information: link
VladimirPutINU.buyTokens(uint256,address) (#426-437) is never used and should be removed
Remove unused functions.
Additional information: link
VladimirPutINU._maxTxAmount (#235) is set pre-construction with a non-constant function or state variable:
- (_totalSupply * 3) / 100
VladimirPutINU._maxWalletSize (#236) is set pre-construction with a non-constant function or state variable:
- (_totalSupply * 3) / 100
VladimirPutINU.swapThreshold (#261) is set pre-construction with a non-constant function or state variable:
- _totalSupply / 1000 * 3
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 VladimirPutINU.swapBack() (#376-410):
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#402)
- (BuyBackSuccess) = address(buybackFeeReceiver).call{gas: 30000,value: amountBNBbuyback}() (#404)
- (DevSuccess) = address(DevFeeReceiver).call{gas: 30000,value: amountBNBDev}() (#406)
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() (#177) is not in mixedCase
Function VladimirPutINU.AntiDumpMultiplier() (#353-358) is not in mixedCase
Parameter VladimirPutINU.addLiquidity(uint256,uint256).BNBAmount (#412) is not in mixedCase
Parameter VladimirPutINU.setFees(uint256,uint256,uint256,uint256,uint256)._liquidityFee (#465) is not in mixedCase
Parameter VladimirPutINU.setFees(uint256,uint256,uint256,uint256,uint256)._buybackFee (#465) is not in mixedCase
Parameter VladimirPutINU.setFees(uint256,uint256,uint256,uint256,uint256)._devFee (#465) is not in mixedCase
Parameter VladimirPutINU.setFees(uint256,uint256,uint256,uint256,uint256)._marketingFee (#465) is not in mixedCase
Parameter VladimirPutINU.setFees(uint256,uint256,uint256,uint256,uint256)._feeDenominator (#465) is not in mixedCase
Parameter VladimirPutINU.setFeeReceiver(address,address)._marketingFeeReceiver (#474) is not in mixedCase
Parameter VladimirPutINU.setFeeReceiver(address,address)._buybackFeeReceiver (#474) is not in mixedCase
Parameter VladimirPutINU.setSwapBackSettings(bool,uint256)._enabled (#479) is not in mixedCase
Parameter VladimirPutINU.setSwapBackSettings(bool,uint256)._amount (#479) is not in mixedCase
Parameter VladimirPutINU.transferForeignToken(address)._token (#489) is not in mixedCase
Variable VladimirPutINU.WBNB (#226) is not in mixedCase
Variable VladimirPutINU.DEAD (#227) is not in mixedCase
Variable VladimirPutINU.ZERO (#228) is not in mixedCase
Constant VladimirPutINU._name (#230) is not in UPPER_CASE_WITH_UNDERSCORES
Constant VladimirPutINU._symbol (#231) is not in UPPER_CASE_WITH_UNDERSCORES
Constant VladimirPutINU._decimals (#232) is not in UPPER_CASE_WITH_UNDERSCORES
Variable VladimirPutINU._totalSupply (#234) is not in mixedCase
Variable VladimirPutINU._maxTxAmount (#235) is not in mixedCase
Variable VladimirPutINU._maxWalletSize (#236) is not in mixedCase
Variable VladimirPutINU._balances (#238) is not in mixedCase
Variable VladimirPutINU._allowances (#239) is not in mixedCase
Variable VladimirPutINU.DevFeeReceiver (#253) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Variable IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#182) is too similar to IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#183)
Prevent variables from having similar names.
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