DOPEFLOKI [Dope FLoki] Token

Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)

DOPEFLOKI.swapBack() (#331-372) sends eth to arbitrary user
Dangerous calls:
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#356)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBdev}() (#358)
DOPEFLOKI.manualSend() (#438-441) sends eth to arbitrary user
Dangerous calls:
- address(devFeeReceiver).transfer(contractETHBalance) (#440)
DOPEFLOKI.transferForeignToken(address) (#443-447) sends eth to arbitrary user
Dangerous calls:
- address(devFeeReceiver).transfer(_contractBalance) (#446)
Ensure that an arbitrary user cannot withdraw unauthorized funds.

Additional information: link

Reentrancy in DOPEFLOKI._transferFrom(address,address,uint256) (#266-292):
External calls:
- swapBack() (#281)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#342-348)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#356)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBdev}() (#358)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,devFeeReceiver,block.timestamp) (#362-369)
External calls sending eth:
- swapBack() (#281)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#356)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBdev}() (#358)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,devFeeReceiver,block.timestamp) (#362-369)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#285)
- _balances[recipient] = _balances[recipient].add(amountReceived) (#288)
- amountReceived = takeFee(sender,recipient,amount) (#287)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#318)
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.

DOPEFLOKI.buyTokens(uint256,address) (#374-385) is never used and should be removed
Remove unused functions.

Additional information: link

Pragma version^0.8.11 (#3) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
solc-0.8.11 is not recommended for deployment
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 DOPEFLOKI.swapBack() (#331-372):
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#356)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBdev}() (#358)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

DOPEFLOKI.DEAD (#168) should be constant
DOPEFLOKI.WBNB (#167) should be constant
DOPEFLOKI.ZERO (#169) should be constant
DOPEFLOKI._totalSupply (#176) should be constant
DOPEFLOKI.routerAddress (#170) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

transferOwnership(address) should be declared external:
- Auth.transferOwnership(address) (#93-97)
renounceOwnership() should be declared external:
- Auth.renounceOwnership() (#105-107)
cooldownEnabled(bool,uint8) should be declared external:
- DOPEFLOKI.cooldownEnabled(bool,uint8) (#408-411)
isBots(address,bool) should be declared external:
- DOPEFLOKI.isBots(address,bool) (#435-437)
transferForeignToken(address) should be declared external:
- DOPEFLOKI.transferForeignToken(address) (#443-447)
isOverLiquified(uint256,uint256) should be declared external:
- DOPEFLOKI.isOverLiquified(uint256,uint256) (#457-459)
Use the external attribute for functions never called from the contract.

Additional information: link

Reentrancy in DOPEFLOKI._transferFrom(address,address,uint256) (#266-292):
External calls:
- swapBack() (#281)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#342-348)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#356)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBdev}() (#358)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,devFeeReceiver,block.timestamp) (#362-369)
External calls sending eth:
- swapBack() (#281)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#356)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBdev}() (#358)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,devFeeReceiver,block.timestamp) (#362-369)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#319)
- amountReceived = takeFee(sender,recipient,amount) (#287)
- Transfer(sender,recipient,amountReceived) (#290)
Reentrancy in DOPEFLOKI.constructor() (#213-232):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#215)
Event emitted after the call(s):
- Transfer(address(0),_owner,_totalSupply) (#231)
Reentrancy in DOPEFLOKI.swapBack() (#331-372):
External calls:
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#342-348)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#356)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBdev}() (#358)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,devFeeReceiver,block.timestamp) (#362-369)
External calls sending eth:
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#356)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBdev}() (#358)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,devFeeReceiver,block.timestamp) (#362-369)
Event emitted after the call(s):
- AutoLiquify(amountBNBLiquidity,amountToLiquify) (#370)
Apply the check-effects-interactions pattern.

Additional information: link

DOPEFLOKI.slitherConstructorVariables() (#164-463) performs a multiplication on the result of a division:
-swapThreshold = _totalSupply / 1000 * 4 (#204)
Consider ordering multiplication before division.

Additional information: link

DOPEFLOKI.swapBack() (#331-372) ignores return value by router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,devFeeReceiver,block.timestamp) (#362-369)
Ensure that all the return values of the function calls are used.

Additional information: link

Auth.transferOwnership(address).adr (#93) lacks a zero-check on :
- owner = adr (#94)
DOPEFLOKI.setFeeReceiver(address,address)._marketingFeeReceiver (#421) lacks a zero-check on :
- marketingFeeReceiver = _marketingFeeReceiver (#422)
DOPEFLOKI.setFeeReceiver(address,address)._devFeeReceiver (#421) lacks a zero-check on :
- devFeeReceiver = _devFeeReceiver (#423)
Check that the address is not zero.

Additional information: link

DOPEFLOKI.slitherConstructorVariables() (#164-463) uses literals with too many digits:
- DEAD = 0x000000000000000000000000000000000000dEaD (#168)
DOPEFLOKI.slitherConstructorVariables() (#164-463) uses literals with too many digits:
- ZERO = 0x0000000000000000000000000000000000000000 (#169)
DOPEFLOKI.slitherConstructorVariables() (#164-463) uses literals with too many digits:
- _totalSupply = 10000000000 * (10 ** _decimals) (#176)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

DOPEFLOKI.setFees(uint256,uint256,uint256,uint256) (#400-406) should emit an event for:
- liquidityFee = _liquidityFee (#401)
- marketingFee = _marketingFee (#402)
- devFee = _devFee (#403)
- totalFee = _liquidityFee.add(_marketingFee).add(_devFee) (#404)
- feeDenominator = _feeDenominator (#405)
DOPEFLOKI.setSellMultiplier(uint256) (#418-420) should emit an event for:
- _sellMultiplier = multiplier (#419)
DOPEFLOKI.setTxLimit(uint256) (#426-429) should emit an event for:
- _maxTxAmount = amountBuy (#427)
DOPEFLOKI.setSwapBackSettings(bool,uint256) (#430-433) should emit an event for:
- swapThreshold = _amount (#432)
Emit an event for critical parameter changes.

Additional information: link

Reentrancy in DOPEFLOKI._transferFrom(address,address,uint256) (#266-292):
External calls:
- swapBack() (#281)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#342-348)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#356)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBdev}() (#358)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,devFeeReceiver,block.timestamp) (#362-369)
External calls sending eth:
- swapBack() (#281)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#356)
- (devSuccess) = address(devFeeReceiver).call{gas: 30000,value: amountBNBdev}() (#358)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,devFeeReceiver,block.timestamp) (#362-369)
State variables written after the call(s):
- launch() (#283)
- launchedAt = block.number (#392)
Reentrancy in DOPEFLOKI.constructor() (#213-232):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#215)
State variables written after the call(s):
- _allowances[address(this)][address(router)] = type()(uint256).max (#216)
- _balances[_owner] = _totalSupply (#230)
- isFeeExempt[msg.sender] = true (#219)
- isTimelockExempt[msg.sender] = true (#225)
- isTimelockExempt[DEAD] = true (#226)
- isTimelockExempt[address(this)] = true (#227)
- isTxLimitExempt[address(this)] = true (#220)
- isTxLimitExempt[routerAddress] = true (#221)
- isTxLimitExempt[msg.sender] = true (#222)
Apply the check-effects-interactions pattern.

Additional information: link

DOPEFLOKI._transferFrom(address,address,uint256) (#266-292) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(cooldownTimer[recipient] < block.timestamp,Please wait for 1min between two operations) (#278)
Avoid relying on block.timestamp.

Additional information: link

DOPEFLOKI._maxTxAmount (#177) is set pre-construction with a non-constant function or state variable:
- (_totalSupply * 2) / 100
DOPEFLOKI._maxWalletSize (#178) is set pre-construction with a non-constant function or state variable:
- (_totalSupply * 2) / 100
DOPEFLOKI.swapThreshold (#204) is set pre-construction with a non-constant function or state variable:
- _totalSupply / 1000 * 4
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

Variable Auth._intAddr (#68) is not in mixedCase
Function IDEXRouter.WETH() (#118) is not in mixedCase
Parameter DOPEFLOKI.setFees(uint256,uint256,uint256,uint256)._liquidityFee (#400) is not in mixedCase
Parameter DOPEFLOKI.setFees(uint256,uint256,uint256,uint256)._marketingFee (#400) is not in mixedCase
Parameter DOPEFLOKI.setFees(uint256,uint256,uint256,uint256)._devFee (#400) is not in mixedCase
Parameter DOPEFLOKI.setFees(uint256,uint256,uint256,uint256)._feeDenominator (#400) is not in mixedCase
Parameter DOPEFLOKI.cooldownEnabled(bool,uint8)._status (#408) is not in mixedCase
Parameter DOPEFLOKI.cooldownEnabled(bool,uint8)._interval (#408) is not in mixedCase
Parameter DOPEFLOKI.setFeeReceiver(address,address)._marketingFeeReceiver (#421) is not in mixedCase
Parameter DOPEFLOKI.setFeeReceiver(address,address)._devFeeReceiver (#421) is not in mixedCase
Parameter DOPEFLOKI.setSwapBackSettings(bool,uint256)._enabled (#430) is not in mixedCase
Parameter DOPEFLOKI.setSwapBackSettings(bool,uint256)._amount (#430) is not in mixedCase
Parameter DOPEFLOKI.isBots(address,bool)._address (#435) is not in mixedCase
Parameter DOPEFLOKI.isBots(address,bool)._value (#435) is not in mixedCase
Parameter DOPEFLOKI.transferForeignToken(address)._token (#443) is not in mixedCase
Variable DOPEFLOKI.WBNB (#167) is not in mixedCase
Variable DOPEFLOKI.DEAD (#168) is not in mixedCase
Variable DOPEFLOKI.ZERO (#169) is not in mixedCase
Constant DOPEFLOKI._name (#172) is not in UPPER_CASE_WITH_UNDERSCORES
Constant DOPEFLOKI._symbol (#173) is not in UPPER_CASE_WITH_UNDERSCORES
Constant DOPEFLOKI._decimals (#174) is not in UPPER_CASE_WITH_UNDERSCORES
Variable DOPEFLOKI._totalSupply (#176) is not in mixedCase
Variable DOPEFLOKI._maxTxAmount (#177) is not in mixedCase
Variable DOPEFLOKI._maxWalletSize (#178) is not in mixedCase
Variable DOPEFLOKI._balances (#180) is not in mixedCase
Variable DOPEFLOKI._allowances (#181) is not in mixedCase
Variable DOPEFLOKI._sellMultiplier (#193) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Variable IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#123) is too similar to IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#124)
Prevent variables from having similar names.

Additional information: link

Number of Binance Smart Chain (BSC) token holders is less than 100. Token is either dead or inactive. Ignore for presale.

Average 30d PancakeSwap liquidity is less than $100. Token is either dead or inactive.

Average 30d PancakeSwap volume is less than $100. Token is either dead or inactive.

Contract has 9% buy tax and 9% sell tax.
Taxes are low but contract ownership is not renounced. Token has a high risk of becoming a honeypot.

Average 30d number of PancakeSwap swaps is low.

Token is deployed only at one blockchain

Token has only one trading pair

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