Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
Reentrancy in Forever._transfer(address,address,uint256) (#808-844):
External calls:
- swapAndSendToFee(sellTokens) (#830)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#878-884)
External calls sending eth:
- swapAndSendToFee(sellTokens) (#830)
- address(_marketingWallet).transfer(ethToSend) (#851)
State variables written after the call(s):
- _tOwned[from] -= amount (#834)
- transferAmount = _getValues(amount,from) (#839)
- _tOwned[address(this)] += marketingFee + liquidityFee (#790)
- _tOwned[to] += transferAmount (#842)
- swapping = false (#831)
Apply the check-effects-interactions pattern.
Additional information: link
Forever.allowance(address,address).owner (#749) shadows:
- Ownable.owner() (#428-430) (function)
Forever._approve(address,address,uint256).owner (#800) shadows:
- Ownable.owner() (#428-430) (function)
Rename the local variables that shadow another component.
Additional information: link
Reentrancy in Forever.constructor() (#706-722):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#711)
State variables written after the call(s):
- _isExcludedFromFee[owner()] = true (#717)
- _isExcludedFromFee[address(this)] = true (#718)
- _isExcludedFromFee[_marketingWallet] = true (#719)
- uniswapV2Pair = _uniswapV2Pair (#714)
- uniswapV2Router = _uniswapV2Router (#713)
Reentrancy in Forever.swapAndLiquify() (#854-866):
External calls:
- newBalance = swapTokensForEth(half) (#859)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#878-884)
- addLiquidity(otherHalf,newBalance) (#863)
- (ethFromLiquidity) = uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#893-900)
- (success) = recipient.call{value: amount}() (#313)
- address(_marketingWallet).sendValue(ethAmount - ethFromLiquidity) (#903)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#863)
- (ethFromLiquidity) = uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#893-900)
- (success) = recipient.call{value: amount}() (#313)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (#863)
- _allowances[owner][spender] = amount (#804)
Reentrancy in Forever.transferFrom(address,address,uint256) (#758-762):
External calls:
- _transfer(sender,recipient,amount) (#759)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#878-884)
External calls sending eth:
- _transfer(sender,recipient,amount) (#759)
- address(_marketingWallet).transfer(ethToSend) (#851)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()] - amount) (#760)
- _allowances[owner][spender] = amount (#804)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in Forever._transfer(address,address,uint256) (#808-844):
External calls:
- swapAndSendToFee(sellTokens) (#830)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#878-884)
External calls sending eth:
- swapAndSendToFee(sellTokens) (#830)
- address(_marketingWallet).transfer(ethToSend) (#851)
Event emitted after the call(s):
- Transfer(from,address(this),marketingFee + liquidityFee) (#791)
- transferAmount = _getValues(amount,from) (#839)
- Transfer(from,to,transferAmount) (#843)
Reentrancy in Forever.constructor() (#706-722):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#711)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_tTotal) (#721)
Reentrancy in Forever.swapAndLiquify() (#854-866):
External calls:
- newBalance = swapTokensForEth(half) (#859)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#878-884)
- addLiquidity(otherHalf,newBalance) (#863)
- (ethFromLiquidity) = uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#893-900)
- (success) = recipient.call{value: amount}() (#313)
- address(_marketingWallet).sendValue(ethAmount - ethFromLiquidity) (#903)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#863)
- (ethFromLiquidity) = uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#893-900)
- (success) = recipient.call{value: amount}() (#313)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#805)
- addLiquidity(otherHalf,newBalance) (#863)
- SwapAndLiquify(half,newBalance,otherHalf) (#864)
Reentrancy in Forever.transferFrom(address,address,uint256) (#758-762):
External calls:
- _transfer(sender,recipient,amount) (#759)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#878-884)
External calls sending eth:
- _transfer(sender,recipient,amount) (#759)
- address(_marketingWallet).transfer(ethToSend) (#851)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#805)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()] - amount) (#760)
Apply the check-effects-interactions pattern.
Additional information: link
Forever._transfer(address,address,uint256) (#808-844) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(_lastBuy[to] + _buyCooldown < block.timestamp,Must wait til after coooldown to buy) (#822)
Forever.addLiquidity(uint256,uint256) (#888-904) uses timestamp for comparisons
Dangerous comparisons:
- ethAmount - ethFromLiquidity > 0 (#902)
Avoid relying on block.timestamp.
Additional information: link
Address.isContract(address) (#282-291) uses assembly
- INLINE ASM (#289)
Address._functionCallWithValue(address,bytes,uint256,string) (#375-396) uses assembly
- INLINE ASM (#388-391)
Do not use evm assembly.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#375-396) is never used and should be removed
Address.functionCall(address,bytes) (#335-337) is never used and should be removed
Address.functionCall(address,bytes,string) (#345-347) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#360-362) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#370-373) is never used and should be removed
Address.isContract(address) (#282-291) is never used and should be removed
Address.sendValue(address,uint256) (#309-315) is never used and should be removed
Context._msgData() (#254-257) is never used and should be removed
Forever.addLiquidity(uint256,uint256) (#888-904) is never used and should be removed
Forever.swapAndLiquify() (#854-866) is never used and should be removed
SafeMath.add(uint256,uint256) (#117-122) is never used and should be removed
SafeMath.div(uint256,uint256) (#191-193) is never used and should be removed
SafeMath.div(uint256,uint256,string) (#207-213) is never used and should be removed
SafeMath.mod(uint256,uint256) (#227-229) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#243-246) is never used and should be removed
SafeMath.mul(uint256,uint256) (#165-177) is never used and should be removed
SafeMath.sub(uint256,uint256) (#134-136) is never used and should be removed
SafeMath.sub(uint256,uint256,string) (#148-153) is never used and should be removed
Remove unused functions.
Additional information: link
Low level call in Address.sendValue(address,uint256) (#309-315):
- (success) = recipient.call{value: amount}() (#313)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#375-396):
- (success,returndata) = target.call{value: weiValue}(data) (#379)
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() (#499) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#500) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#517) is not in mixedCase
Function IUniswapV2Router01.WETH() (#539) is not in mixedCase
Variable Forever._maxTxAmount (#689) is not in mixedCase
Constant Forever._name (#691) is not in UPPER_CASE_WITH_UNDERSCORES
Constant Forever._symbol (#692) is not in UPPER_CASE_WITH_UNDERSCORES
Constant Forever._decimals (#693) is not in UPPER_CASE_WITH_UNDERSCORES
Variable Forever._buyCooldown (#699) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#255)" inContext (#249-258)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in Forever._transfer(address,address,uint256) (#808-844):
External calls:
- swapAndSendToFee(sellTokens) (#830)
- address(_marketingWallet).transfer(ethToSend) (#851)
State variables written after the call(s):
- _tOwned[from] -= amount (#834)
- transferAmount = _getValues(amount,from) (#839)
- _tOwned[address(this)] += marketingFee + liquidityFee (#790)
- _tOwned[to] += transferAmount (#842)
- swapping = false (#831)
Event emitted after the call(s):
- Transfer(from,address(this),marketingFee + liquidityFee) (#791)
- transferAmount = _getValues(amount,from) (#839)
- Transfer(from,to,transferAmount) (#843)
Reentrancy in Forever.transferFrom(address,address,uint256) (#758-762):
External calls:
- _transfer(sender,recipient,amount) (#759)
- address(_marketingWallet).transfer(ethToSend) (#851)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()] - amount) (#760)
- _allowances[owner][spender] = amount (#804)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#805)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()] - amount) (#760)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#544) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#545)
Prevent variables from having similar names.
Additional information: link
Address.sendValue(address,uint256) (#309-315) uses literals with too many digits:
- recipient = address(0x000000000000000000000000000000000000dEaD) (#311)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
Forever._buyCooldown (#699) should be constant
Forever._liquidityFee (#696) should be constant
Forever._marketingFee (#695) should be constant
Forever._marketingWallet (#697) should be constant
Forever._maxTxAmount (#689) should be constant
Forever._tTotal (#688) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#447-450)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#456-460)
name() should be declared external:
- Forever.name() (#724-726)
symbol() should be declared external:
- Forever.symbol() (#728-730)
decimals() should be declared external:
- Forever.decimals() (#732-734)
totalSupply() should be declared external:
- Forever.totalSupply() (#736-738)
transfer(address,uint256) should be declared external:
- Forever.transfer(address,uint256) (#744-747)
allowance(address,address) should be declared external:
- Forever.allowance(address,address) (#749-751)
approve(address,uint256) should be declared external:
- Forever.approve(address,uint256) (#753-756)
transferFrom(address,address,uint256) should be declared external:
- Forever.transferFrom(address,address,uint256) (#758-762)
increaseAllowance(address,uint256) should be declared external:
- Forever.increaseAllowance(address,uint256) (#764-767)
decreaseAllowance(address,uint256) should be declared external:
- Forever.decreaseAllowance(address,uint256) (#769-772)
excludeFromFee(address) should be declared external:
- Forever.excludeFromFee(address) (#774-776)
includeInFee(address) should be declared external:
- Forever.includeInFee(address) (#778-780)
isExcludedFromFee(address) should be declared external:
- Forever.isExcludedFromFee(address) (#796-798)
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 Twitter account
Telegram account has relatively few subscribers
Unable to find Blog account (Reddit or Medium)
Unable to find Youtube account
Unable to find Discord account