Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
DEGEN.swapBack() (#296-337) sends eth to arbitrary user
Dangerous calls:
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#321)
- (developmentSuccess) = address(developmentFeeReceiver).call{gas: 30000,value: amountBNBdevelopment}() (#323)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in DEGEN._transferFrom(address,address,uint256) (#237-257):
External calls:
- swapBack() (#246)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#307-313)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#321)
- (developmentSuccess) = address(developmentFeeReceiver).call{gas: 30000,value: amountBNBdevelopment}() (#323)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#327-334)
External calls sending eth:
- swapBack() (#246)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#321)
- (developmentSuccess) = address(developmentFeeReceiver).call{gas: 30000,value: amountBNBdevelopment}() (#323)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#327-334)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#250)
- _balances[recipient] = _balances[recipient].add(amountReceived) (#253)
- amountReceived = takeFee(sender,recipient,amount) (#252)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#283)
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.
Not a direct threat, but may indicate unreliable intentions of developer. Widespread names (e.g. Elon, King, Moon, Doge) are common among meme-tokens and scams. The allow to gain free hype and attract unexperienced investors.
DEGEN.slitherConstructorVariables() (#152-421) uses literals with too many digits:
- DEAD = 0x000000000000000000000000000000000000dEaD (#156)
DEGEN.slitherConstructorVariables() (#152-421) uses literals with too many digits:
- ZERO = 0x0000000000000000000000000000000000000000 (#157)
DEGEN.slitherConstructorVariables() (#152-421) uses literals with too many digits:
- _totalSupply = 1000000000 * (10 ** _decimals) (#163)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
DEGEN.DEAD (#156) should be constant
DEGEN.WBNB (#155) should be constant
DEGEN.ZERO (#157) should be constant
DEGEN._totalSupply (#163) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
DEGEN.slitherConstructorVariables() (#152-421) performs a multiplication on the result of a division:
-swapThreshold = _totalSupply / 1000 * 3 (#188)
Consider ordering multiplication before division.
Additional information: link
DEGEN.swapBack() (#296-337) ignores return value by router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#327-334)
Ensure that all the return values of the function calls are used.
Additional information: link
DEGEN.setTxLimit(uint256) (#360-363) should emit an event for:
- _maxTxAmount = amount (#362)
DEGEN.setFees(uint256,uint256,uint256,uint256) (#378-384) should emit an event for:
- liquidityFee = _liquidityFee (#379)
- developmentFee = _developmentFee (#380)
- marketingFee = _marketingFee (#381)
- totalFee = _liquidityFee.add(_developmentFee).add(_marketingFee) (#382)
- feeDenominator = _feeDenominator (#383)
Emit an event for critical parameter changes.
Additional information: link
Auth.transferOwnership(address).adr (#91) lacks a zero-check on :
- owner = adr (#92)
DEGEN.setFeeReceiver(address,address)._marketingFeeReceiver (#386) lacks a zero-check on :
- marketingFeeReceiver = _marketingFeeReceiver (#387)
DEGEN.setFeeReceiver(address,address)._developmentFeeReceiver (#386) lacks a zero-check on :
- developmentFeeReceiver = _developmentFeeReceiver (#388)
Check that the address is not zero.
Additional information: link
Reentrancy in DEGEN._transferFrom(address,address,uint256) (#237-257):
External calls:
- swapBack() (#246)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#307-313)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#321)
- (developmentSuccess) = address(developmentFeeReceiver).call{gas: 30000,value: amountBNBdevelopment}() (#323)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#327-334)
External calls sending eth:
- swapBack() (#246)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#321)
- (developmentSuccess) = address(developmentFeeReceiver).call{gas: 30000,value: amountBNBdevelopment}() (#323)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#327-334)
State variables written after the call(s):
- launch() (#248)
- launchedAt = block.number (#357)
Reentrancy in DEGEN.constructor() (#192-203):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#194)
State variables written after the call(s):
- _allowances[address(this)][address(router)] = type()(uint256).max (#195)
- _balances[_owner] = _totalSupply (#201)
- isFeeExempt[_owner] = true (#198)
- isTxLimitExempt[_owner] = true (#199)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in DEGEN._transferFrom(address,address,uint256) (#237-257):
External calls:
- swapBack() (#246)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#307-313)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#321)
- (developmentSuccess) = address(developmentFeeReceiver).call{gas: 30000,value: amountBNBdevelopment}() (#323)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#327-334)
External calls sending eth:
- swapBack() (#246)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#321)
- (developmentSuccess) = address(developmentFeeReceiver).call{gas: 30000,value: amountBNBdevelopment}() (#323)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#327-334)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#284)
- amountReceived = takeFee(sender,recipient,amount) (#252)
- Transfer(sender,recipient,amountReceived) (#255)
Reentrancy in DEGEN.constructor() (#192-203):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#194)
Event emitted after the call(s):
- Transfer(address(0),_owner,_totalSupply) (#202)
Reentrancy in DEGEN.swapBack() (#296-337):
External calls:
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#307-313)
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#321)
- (developmentSuccess) = address(developmentFeeReceiver).call{gas: 30000,value: amountBNBdevelopment}() (#323)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#327-334)
External calls sending eth:
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#321)
- (developmentSuccess) = address(developmentFeeReceiver).call{gas: 30000,value: amountBNBdevelopment}() (#323)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,marketingFeeReceiver,block.timestamp) (#327-334)
Event emitted after the call(s):
- AutoLiquify(amountBNBLiquidity,amountToLiquify) (#335)
Apply the check-effects-interactions pattern.
Additional information: link
DEGEN.buyTokens(uint256,address) (#339-350) is never used and should be removed
Remove unused functions.
Additional information: link
DEGEN._maxTxAmount (#164) is set pre-construction with a non-constant function or state variable:
- (_totalSupply * 6) / 200
DEGEN._maxWalletSize (#165) is set pre-construction with a non-constant function or state variable:
- (_totalSupply * 10) / 100
DEGEN.swapThreshold (#188) 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 DEGEN.swapBack() (#296-337):
- (MarketingSuccess) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#321)
- (developmentSuccess) = address(developmentFeeReceiver).call{gas: 30000,value: amountBNBdevelopment}() (#323)
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() (#106) is not in mixedCase
Parameter DEGEN.setFees(uint256,uint256,uint256,uint256)._liquidityFee (#378) is not in mixedCase
Parameter DEGEN.setFees(uint256,uint256,uint256,uint256)._developmentFee (#378) is not in mixedCase
Parameter DEGEN.setFees(uint256,uint256,uint256,uint256)._marketingFee (#378) is not in mixedCase
Parameter DEGEN.setFees(uint256,uint256,uint256,uint256)._feeDenominator (#378) is not in mixedCase
Parameter DEGEN.setFeeReceiver(address,address)._marketingFeeReceiver (#386) is not in mixedCase
Parameter DEGEN.setFeeReceiver(address,address)._developmentFeeReceiver (#386) is not in mixedCase
Parameter DEGEN.setSwapBackSettings(bool,uint256)._enabled (#391) is not in mixedCase
Parameter DEGEN.setSwapBackSettings(bool,uint256)._amount (#391) is not in mixedCase
Parameter DEGEN.transferForeignToken(address)._token (#401) is not in mixedCase
Variable DEGEN.WBNB (#155) is not in mixedCase
Variable DEGEN.DEAD (#156) is not in mixedCase
Variable DEGEN.ZERO (#157) is not in mixedCase
Constant DEGEN._name (#159) is not in UPPER_CASE_WITH_UNDERSCORES
Constant DEGEN._symbol (#160) is not in UPPER_CASE_WITH_UNDERSCORES
Constant DEGEN._decimals (#161) is not in UPPER_CASE_WITH_UNDERSCORES
Variable DEGEN._totalSupply (#163) is not in mixedCase
Variable DEGEN._maxTxAmount (#164) is not in mixedCase
Variable DEGEN._maxWalletSize (#165) is not in mixedCase
Variable DEGEN._balances (#167) is not in mixedCase
Variable DEGEN._allowances (#168) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Variable IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#111) is too similar to IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#112)
Prevent variables from having similar names.
Additional information: link
authorize(address) should be declared external:
- Auth.authorize(address) (#75-77)
unauthorize(address) should be declared external:
- Auth.unauthorize(address) (#79-81)
transferOwnership(address) should be declared external:
- Auth.transferOwnership(address) (#91-95)
transferForeignToken(address) should be declared external:
- DEGEN.transferForeignToken(address) (#401-405)
isOverLiquified(uint256,uint256) should be declared external:
- DEGEN.isOverLiquified(uint256,uint256) (#415-417)
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