Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
Reentrancy in McGreen._transfer(address,address,uint256) (#1029-1080):
External calls:
- swapAndLiquify(contractTokenBalance) (#1063)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1126-1132)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1063)
- recipient.transfer(amount) (#905)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
State variables written after the call(s):
- _liquidityFee = _sellLiquidityFee (#1068)
- _liquidityFee = _BuyLiquidityFee (#1074)
- _tokenTransfer(from,to,amount) (#1079)
- _liquidityFee = _previousLiquidityFee (#989)
- _liquidityFee = 0 (#983)
- _marketingFee = _sellMarketingFee (#1069)
- _marketingFee = _BuyMarketingFee (#1075)
- _tokenTransfer(from,to,amount) (#1079)
- _marketingFee = _previousmarketingFee (#990)
- _marketingFee = 0 (#984)
- _tokenTransfer(from,to,amount) (#1079)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#958)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1205)
- _rOwned[address(this)] = _rOwned[address(this)].add(rMarketing) (#1199)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#863)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1184)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1216)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1207)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1217)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1185)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#865)
- _tokenTransfer(from,to,amount) (#1079)
- _rTotal = _rTotal.sub(rFee) (#913)
- _tokenTransfer(from,to,amount) (#1079)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#960)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1215)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#862)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1206)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#864)
Apply the check-effects-interactions pattern.
Additional information: link
Contract name (McGreen (t.me/McGreenBSC)) 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.
Contract ticker ($McGreen) 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.
McGreen.addLiquidity(uint256,uint256) (#1135-1148) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
Ensure that all the return values of the function calls are used.
Additional information: link
McGreen.allowance(address,address).owner (#779) shadows:
- Ownable.owner() (#416-418) (function)
McGreen._approve(address,address,uint256).owner (#997) shadows:
- Ownable.owner() (#416-418) (function)
Rename the local variables that shadow another component.
Additional information: link
McGreen.takeMarketing(uint256,uint256,uint256) (#1191-1201) performs a multiplication on the result of a division:
-tMarketing = tAmount.div(100).mul(_marketingFee) (#1195)
Consider ordering multiplication before division.
Additional information: link
McGreen.setBuyFeesPercentage(uint256,uint256,uint256) (#878-882) should emit an event for:
- _BuyTaxFee = taxFee (#879)
- _BuyLiquidityFee = liquidityFee (#880)
- _BuyMarketingFee = mFee (#881)
McGreen.setSellFeesPercentage(uint256,uint256,uint256) (#884-888) should emit an event for:
- _sellTaxFee = taxFee (#885)
- _sellLiquidityFee = liquidityFee (#886)
- _sellMarketingFee = mFee (#887)
McGreen.setMaxTxAmount(uint256) (#889-892) should emit an event for:
- _maxTxAmount = maxTxAmount * 10 ** _decimals (#891)
McGreen.setMaxWalletHoldingAmount(uint256) (#894-897) should emit an event for:
- _maxWalletHoldingAmount = maxWalletHolding * 10 ** _decimals (#896)
Emit an event for critical parameter changes.
Additional information: link
Reentrancy in McGreen._transfer(address,address,uint256) (#1029-1080):
External calls:
- swapAndLiquify(contractTokenBalance) (#1063)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1126-1132)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1063)
- recipient.transfer(amount) (#905)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
State variables written after the call(s):
- _tokenTransfer(from,to,amount) (#1079)
- _previousLiquidityFee = _liquidityFee (#979)
- _tokenTransfer(from,to,amount) (#1079)
- _previousTaxFee = _taxFee (#978)
- _tokenTransfer(from,to,amount) (#1079)
- _previousmarketingFee = _marketingFee (#980)
- _tokenTransfer(from,to,amount) (#1079)
- _tFeeTotal = _tFeeTotal.add(tFee) (#914)
- _taxFee = _sellTaxFee (#1067)
- _taxFee = _BuyTaxFee (#1073)
- _tokenTransfer(from,to,amount) (#1079)
- _taxFee = _previousTaxFee (#988)
- _taxFee = 0 (#982)
Reentrancy in McGreen.constructor() (#733-751):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#738-739)
State variables written after the call(s):
- _isAutoMaticMarketMaker[uniswapV2Pair] = true (#747)
- _isExcludedFromFee[owner()] = true (#745)
- _isExcludedFromFee[address(this)] = true (#746)
- _isExcludedFromWalletLimit[uniswapV2Pair] = true (#748)
- uniswapV2Router = _uniswapV2Router (#742)
Reentrancy in McGreen.swapAndLiquify(uint256) (#1082-1115):
External calls:
- swapTokensForEth(half) (#1099)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1126-1132)
- addLiquidity(otherHalf,newBalance) (#1105)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1105)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (#1105)
- _allowances[owner][spender] = amount (#1001)
Reentrancy in McGreen.swapAndLiquify(uint256) (#1082-1115):
External calls:
- swapTokensForEth(half) (#1099)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1126-1132)
- addLiquidity(otherHalf,newBalance) (#1105)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
- swapTokensForEth(tokensForMarketing) (#1110)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1126-1132)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1105)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
State variables written after the call(s):
- swapTokensForEth(tokensForMarketing) (#1110)
- _allowances[owner][spender] = amount (#1001)
Reentrancy in McGreen.transferFrom(address,address,uint256) (#788-792):
External calls:
- _transfer(sender,recipient,amount) (#789)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1126-1132)
External calls sending eth:
- _transfer(sender,recipient,amount) (#789)
- recipient.transfer(amount) (#905)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,BEP20: transfer amount exceeds allowance)) (#790)
- _allowances[owner][spender] = amount (#1001)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in McGreen._transfer(address,address,uint256) (#1029-1080):
External calls:
- swapAndLiquify(contractTokenBalance) (#1063)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1126-1132)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1063)
- recipient.transfer(amount) (#905)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (#1210)
- _tokenTransfer(from,to,amount) (#1079)
- Transfer(sender,recipient,tTransferAmount) (#1188)
- _tokenTransfer(from,to,amount) (#1079)
- Transfer(sender,recipient,tTransferAmount) (#1220)
- _tokenTransfer(from,to,amount) (#1079)
- Transfer(sender,recipient,tTransferAmount) (#868)
- _tokenTransfer(from,to,amount) (#1079)
Reentrancy in McGreen.constructor() (#733-751):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#738-739)
Event emitted after the call(s):
- Transfer(address(0),owner(),_tTotal) (#750)
Reentrancy in McGreen.swapAndLiquify(uint256) (#1082-1115):
External calls:
- swapTokensForEth(half) (#1099)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1126-1132)
- addLiquidity(otherHalf,newBalance) (#1105)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1105)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1002)
- addLiquidity(otherHalf,newBalance) (#1105)
- SwapAndLiquify(half,newBalance,otherHalf) (#1107)
Reentrancy in McGreen.swapAndLiquify(uint256) (#1082-1115):
External calls:
- swapTokensForEth(half) (#1099)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1126-1132)
- addLiquidity(otherHalf,newBalance) (#1105)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
- swapTokensForEth(tokensForMarketing) (#1110)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1126-1132)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1105)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1002)
- swapTokensForEth(tokensForMarketing) (#1110)
Reentrancy in McGreen.transferFrom(address,address,uint256) (#788-792):
External calls:
- _transfer(sender,recipient,amount) (#789)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1126-1132)
External calls sending eth:
- _transfer(sender,recipient,amount) (#789)
- recipient.transfer(amount) (#905)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1002)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,BEP20: transfer amount exceeds allowance)) (#790)
Apply the check-effects-interactions pattern.
Additional information: link
McGreen._transfer(address,address,uint256) (#1029-1080) uses timestamp for comparisons
Dangerous comparisons:
- require(bool)(TimeLimit[to] <= block.timestamp) (#1043)
Avoid relying on block.timestamp.
Additional information: link
Address.isContract(address) (#269-278) uses assembly
- INLINE ASM (#276)
Address._functionCallWithValue(address,bytes,uint256,string) (#362-383) uses assembly
- INLINE ASM (#375-378)
Do not use evm assembly.
Additional information: link
McGreen.includeInReward(address) (#847-858) has costly operations inside a loop:
- _excluded.pop() (#854)
Use a local variable to hold the loop computation result.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#362-383) is never used and should be removed
Address.functionCall(address,bytes) (#322-324) is never used and should be removed
Address.functionCall(address,bytes,string) (#332-334) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#347-349) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#357-360) is never used and should be removed
Address.isContract(address) (#269-278) is never used and should be removed
Address.sendValue(address,uint256) (#296-302) is never used and should be removed
Context._msgData() (#241-244) is never used and should be removed
SafeMath.mod(uint256,uint256) (#214-216) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#230-233) is never used and should be removed
Remove unused functions.
Additional information: link
McGreen._rTotal (#679) is set pre-construction with a non-constant function or state variable:
- (MAX - (MAX % _tTotal))
McGreen._previousTaxFee (#692) is set pre-construction with a non-constant function or state variable:
- _taxFee
McGreen._previousLiquidityFee (#695) is set pre-construction with a non-constant function or state variable:
- _liquidityFee
McGreen._previousmarketingFee (#698) is set pre-construction with a non-constant function or state variable:
- _marketingFee
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
Pragma version^0.8.0 (#6) allows old versions
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 Address.sendValue(address,uint256) (#296-302):
- (success) = recipient.call{value: amount}() (#300)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#362-383):
- (success,returndata) = target.call{value: weiValue}(data) (#366)
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() (#488) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#489) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#505) is not in mixedCase
Function IUniswapV2Router01.WETH() (#526) is not in mixedCase
Parameter McGreen.setSwapAndLiquifyEnabled(bool)._enabled (#899) is not in mixedCase
Parameter McGreen.calculateTaxFee(uint256)._amount (#963) is not in mixedCase
Parameter McGreen.calculateLiquidityFee(uint256)._amount (#969) is not in mixedCase
Function McGreen._isblackListAddress(address) (#1005-1007) is not in mixedCase
Function McGreen._isAutoMaticMarketMakerAddress(address) (#1009-1011) is not in mixedCase
Function McGreen._isAddressExcludedFromWalletLimit(address) (#1013-1015) is not in mixedCase
Parameter McGreen.setBlackListAddress(address,bool).Account (#1017) is not in mixedCase
Parameter McGreen.setAutoMaticMarketMaker(address,bool).Account (#1021) is not in mixedCase
Function McGreen.ExcludeFromHoldingLimit(address,bool) (#1025-1027) is not in mixedCase
Parameter McGreen.ExcludeFromHoldingLimit(address,bool).Account (#1025) is not in mixedCase
Variable McGreen.TimeLimit (#683) is not in mixedCase
Variable McGreen._BuyTaxFee (#700) is not in mixedCase
Variable McGreen._BuyLiquidityFee (#701) is not in mixedCase
Variable McGreen._BuyMarketingFee (#702) is not in mixedCase
Variable McGreen._sellTaxFee (#704) is not in mixedCase
Variable McGreen._sellLiquidityFee (#705) is not in mixedCase
Variable McGreen._sellMarketingFee (#706) is not in mixedCase
Variable McGreen._maxTxAmount (#715) is not in mixedCase
Variable McGreen._maxWalletHoldingAmount (#716) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#242)" inContext (#236-245)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in McGreen._transfer(address,address,uint256) (#1029-1080):
External calls:
- swapAndLiquify(contractTokenBalance) (#1063)
- recipient.transfer(amount) (#905)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1063)
- recipient.transfer(amount) (#905)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
State variables written after the call(s):
- _liquidityFee = _sellLiquidityFee (#1068)
- _liquidityFee = _BuyLiquidityFee (#1074)
- _tokenTransfer(from,to,amount) (#1079)
- _liquidityFee = _previousLiquidityFee (#989)
- _liquidityFee = 0 (#983)
- _marketingFee = _sellMarketingFee (#1069)
- _marketingFee = _BuyMarketingFee (#1075)
- _tokenTransfer(from,to,amount) (#1079)
- _marketingFee = _previousmarketingFee (#990)
- _marketingFee = 0 (#984)
- _tokenTransfer(from,to,amount) (#1079)
- _previousLiquidityFee = _liquidityFee (#979)
- _tokenTransfer(from,to,amount) (#1079)
- _previousTaxFee = _taxFee (#978)
- _tokenTransfer(from,to,amount) (#1079)
- _previousmarketingFee = _marketingFee (#980)
- _tokenTransfer(from,to,amount) (#1079)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#958)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1205)
- _rOwned[address(this)] = _rOwned[address(this)].add(rMarketing) (#1199)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#863)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1184)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1216)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1207)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1217)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1185)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#865)
- _tokenTransfer(from,to,amount) (#1079)
- _rTotal = _rTotal.sub(rFee) (#913)
- _tokenTransfer(from,to,amount) (#1079)
- _tFeeTotal = _tFeeTotal.add(tFee) (#914)
- _tokenTransfer(from,to,amount) (#1079)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#960)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1215)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#862)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1206)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#864)
- _taxFee = _sellTaxFee (#1067)
- _taxFee = _BuyTaxFee (#1073)
- _tokenTransfer(from,to,amount) (#1079)
- _taxFee = _previousTaxFee (#988)
- _taxFee = 0 (#982)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (#1220)
- _tokenTransfer(from,to,amount) (#1079)
- Transfer(sender,recipient,tTransferAmount) (#1210)
- _tokenTransfer(from,to,amount) (#1079)
- Transfer(sender,recipient,tTransferAmount) (#1188)
- _tokenTransfer(from,to,amount) (#1079)
- Transfer(sender,recipient,tTransferAmount) (#868)
- _tokenTransfer(from,to,amount) (#1079)
Reentrancy in McGreen.transferFrom(address,address,uint256) (#788-792):
External calls:
- _transfer(sender,recipient,amount) (#789)
- recipient.transfer(amount) (#905)
External calls sending eth:
- _transfer(sender,recipient,amount) (#789)
- recipient.transfer(amount) (#905)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1140-1147)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,BEP20: transfer amount exceeds allowance)) (#790)
- _allowances[owner][spender] = amount (#1001)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1002)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,BEP20: transfer amount exceeds allowance)) (#790)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#531) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#532)
Variable McGreen._transferStandard(address,address,uint256).rTransferAmount (#1182) is too similar to McGreen._transferStandard(address,address,uint256).tTransferAmount (#1182)
Variable McGreen._getValues(uint256).rTransferAmount (#919) is too similar to McGreen._transferFromExcluded(address,address,uint256).tTransferAmount (#1214)
Variable McGreen._transferBothExcluded(address,address,uint256).rTransferAmount (#861) is too similar to McGreen._transferToExcluded(address,address,uint256).tTransferAmount (#1204)
Variable McGreen._transferBothExcluded(address,address,uint256).rTransferAmount (#861) is too similar to McGreen._getValues(uint256).tTransferAmount (#918)
Variable McGreen.takeMarketing(uint256,uint256,uint256).rTransferAmount (#1191) is too similar to McGreen._getTValues(uint256).tTransferAmount (#926)
Variable McGreen._transferToExcluded(address,address,uint256).rTransferAmount (#1204) is too similar to McGreen._transferFromExcluded(address,address,uint256).tTransferAmount (#1214)
Variable McGreen.reflectionFromToken(uint256,bool).rTransferAmount (#827) is too similar to McGreen._transferFromExcluded(address,address,uint256).tTransferAmount (#1214)
Variable McGreen._getValues(uint256).rTransferAmount (#919) is too similar to McGreen._transferStandard(address,address,uint256).tTransferAmount (#1182)
Variable McGreen._transferToExcluded(address,address,uint256).rTransferAmount (#1204) is too similar to McGreen._transferStandard(address,address,uint256).tTransferAmount (#1182)
Variable McGreen.reflectionFromToken(uint256,bool).rTransferAmount (#827) is too similar to McGreen._transferStandard(address,address,uint256).tTransferAmount (#1182)
Variable McGreen._transferFromExcluded(address,address,uint256).rTransferAmount (#1214) is too similar to McGreen._transferFromExcluded(address,address,uint256).tTransferAmount (#1214)
Variable McGreen._transferBothExcluded(address,address,uint256).rTransferAmount (#861) is too similar to McGreen._getTValues(uint256).tTransferAmount (#926)
Variable McGreen._getValues(uint256).rTransferAmount (#919) is too similar to McGreen._transferBothExcluded(address,address,uint256).tTransferAmount (#861)
Variable McGreen.takeMarketing(uint256,uint256,uint256).rTransferAmount (#1191) is too similar to McGreen._transferFromExcluded(address,address,uint256).tTransferAmount (#1214)
Variable McGreen.takeMarketing(uint256,uint256,uint256).rTransferAmount (#1191) is too similar to McGreen._transferStandard(address,address,uint256).tTransferAmount (#1182)
Variable McGreen._getValues(uint256).rTransferAmount (#919) is too similar to McGreen._transferToExcluded(address,address,uint256).tTransferAmount (#1204)
Variable McGreen._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#934) is too similar to McGreen._transferFromExcluded(address,address,uint256).tTransferAmount (#1214)
Variable McGreen._transferToExcluded(address,address,uint256).rTransferAmount (#1204) is too similar to McGreen._transferToExcluded(address,address,uint256).tTransferAmount (#1204)
Variable McGreen._getValues(uint256).rTransferAmount (#919) is too similar to McGreen._getValues(uint256).tTransferAmount (#918)
Variable McGreen._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#934) is too similar to McGreen._transferStandard(address,address,uint256).tTransferAmount (#1182)
Variable McGreen._transferToExcluded(address,address,uint256).rTransferAmount (#1204) is too similar to McGreen._getValues(uint256).tTransferAmount (#918)
Variable McGreen._transferBothExcluded(address,address,uint256).rTransferAmount (#861) is too similar to McGreen._transferFromExcluded(address,address,uint256).tTransferAmount (#1214)
Variable McGreen.takeMarketing(uint256,uint256,uint256).rTransferAmount (#1191) is too similar to McGreen._transferBothExcluded(address,address,uint256).tTransferAmount (#861)
Variable McGreen._transferBothExcluded(address,address,uint256).rTransferAmount (#861) is too similar to McGreen._transferStandard(address,address,uint256).tTransferAmount (#1182)
Variable McGreen._getValues(uint256).rTransferAmount (#919) is too similar to McGreen._getTValues(uint256).tTransferAmount (#926)
Variable McGreen.takeMarketing(uint256,uint256,uint256).rTransferAmount (#1191) is too similar to McGreen._transferToExcluded(address,address,uint256).tTransferAmount (#1204)
Variable McGreen.takeMarketing(uint256,uint256,uint256).rTransferAmount (#1191) is too similar to McGreen._getValues(uint256).tTransferAmount (#918)
Variable McGreen._transferBothExcluded(address,address,uint256).rTransferAmount (#861) is too similar to McGreen._transferBothExcluded(address,address,uint256).tTransferAmount (#861)
Variable McGreen._transferStandard(address,address,uint256).rTransferAmount (#1182) is too similar to McGreen._transferFromExcluded(address,address,uint256).tTransferAmount (#1214)
Variable McGreen._transferStandard(address,address,uint256).rTransferAmount (#1182) is too similar to McGreen._transferBothExcluded(address,address,uint256).tTransferAmount (#861)
Variable McGreen._transferBothExcluded(address,address,uint256).rTransferAmount (#861) is too similar to McGreen.takeMarketing(uint256,uint256,uint256).tTransferAmount (#1191)
Variable McGreen._transferStandard(address,address,uint256).rTransferAmount (#1182) is too similar to McGreen._getValues(uint256).tTransferAmount (#918)
Variable McGreen._transferToExcluded(address,address,uint256).rTransferAmount (#1204) is too similar to McGreen.takeMarketing(uint256,uint256,uint256).tTransferAmount (#1191)
Variable McGreen._getValues(uint256).rTransferAmount (#919) is too similar to McGreen.takeMarketing(uint256,uint256,uint256).tTransferAmount (#1191)
Variable McGreen.reflectionFromToken(uint256,bool).rTransferAmount (#827) is too similar to McGreen.takeMarketing(uint256,uint256,uint256).tTransferAmount (#1191)
Variable McGreen._transferFromExcluded(address,address,uint256).rTransferAmount (#1214) is too similar to McGreen._transferStandard(address,address,uint256).tTransferAmount (#1182)
Variable McGreen._transferFromExcluded(address,address,uint256).rTransferAmount (#1214) is too similar to McGreen._transferToExcluded(address,address,uint256).tTransferAmount (#1204)
Variable McGreen._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#934) is too similar to McGreen._transferToExcluded(address,address,uint256).tTransferAmount (#1204)
Variable McGreen._transferToExcluded(address,address,uint256).rTransferAmount (#1204) is too similar to McGreen._transferBothExcluded(address,address,uint256).tTransferAmount (#861)
Variable McGreen.reflectionFromToken(uint256,bool).rTransferAmount (#827) is too similar to McGreen._transferBothExcluded(address,address,uint256).tTransferAmount (#861)
Variable McGreen._transferStandard(address,address,uint256).rTransferAmount (#1182) is too similar to McGreen._transferToExcluded(address,address,uint256).tTransferAmount (#1204)
Variable McGreen.reflectionFromToken(uint256,bool).rTransferAmount (#827) is too similar to McGreen._getValues(uint256).tTransferAmount (#918)
Variable McGreen._transferFromExcluded(address,address,uint256).rTransferAmount (#1214) is too similar to McGreen._getTValues(uint256).tTransferAmount (#926)
Variable McGreen._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#934) is too similar to McGreen._getTValues(uint256).tTransferAmount (#926)
Variable McGreen._transferStandard(address,address,uint256).rTransferAmount (#1182) is too similar to McGreen._getTValues(uint256).tTransferAmount (#926)
Variable McGreen.takeMarketing(uint256,uint256,uint256).rTransferAmount (#1191) is too similar to McGreen.takeMarketing(uint256,uint256,uint256).tTransferAmount (#1191)
Variable McGreen.reflectionFromToken(uint256,bool).rTransferAmount (#827) is too similar to McGreen._transferToExcluded(address,address,uint256).tTransferAmount (#1204)
Variable McGreen._transferFromExcluded(address,address,uint256).rTransferAmount (#1214) is too similar to McGreen.takeMarketing(uint256,uint256,uint256).tTransferAmount (#1191)
Variable McGreen._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#934) is too similar to McGreen.takeMarketing(uint256,uint256,uint256).tTransferAmount (#1191)
Variable McGreen._transferStandard(address,address,uint256).rTransferAmount (#1182) is too similar to McGreen.takeMarketing(uint256,uint256,uint256).tTransferAmount (#1191)
Variable McGreen._transferToExcluded(address,address,uint256).rTransferAmount (#1204) is too similar to McGreen._getTValues(uint256).tTransferAmount (#926)
Variable McGreen.reflectionFromToken(uint256,bool).rTransferAmount (#827) is too similar to McGreen._getTValues(uint256).tTransferAmount (#926)
Variable McGreen._transferFromExcluded(address,address,uint256).rTransferAmount (#1214) is too similar to McGreen._transferBothExcluded(address,address,uint256).tTransferAmount (#861)
Variable McGreen._transferFromExcluded(address,address,uint256).rTransferAmount (#1214) is too similar to McGreen._getValues(uint256).tTransferAmount (#918)
Variable McGreen._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#934) is too similar to McGreen._transferBothExcluded(address,address,uint256).tTransferAmount (#861)
Variable McGreen._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#934) is too similar to McGreen._getValues(uint256).tTransferAmount (#918)
Prevent variables from having similar names.
Additional information: link
McGreen.slitherConstructorVariables() (#664-1223) uses literals with too many digits:
- numTokensSellToAddToLiquidity = 2000000 * 10 ** 18 (#714)
McGreen.slitherConstructorVariables() (#664-1223) uses literals with too many digits:
- _maxTxAmount = 20000000 * 10 ** 18 (#715)
McGreen.slitherConstructorVariables() (#664-1223) uses literals with too many digits:
- _maxWalletHoldingAmount = 20000000 * 10 ** 18 (#716)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
Ownable._previousOwner (#400) is never used in McGreen (#664-1223)
Ownable._lockTime (#401) is never used in McGreen (#664-1223)
Remove unused state variables.
Additional information: link
McGreen._decimals (#689) should be constant
McGreen._name (#687) should be constant
McGreen._symbol (#688) should be constant
McGreen._tTotal (#678) should be constant
McGreen.coldDownTime (#717) should be constant
McGreen.marketingWallet (#685) should be constant
McGreen.numTokensSellToAddToLiquidity (#714) should be constant
Ownable._lockTime (#401) should be constant
Ownable._previousOwner (#400) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#435-438)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#444-448)
name() should be declared external:
- McGreen.name() (#753-755)
symbol() should be declared external:
- McGreen.symbol() (#757-759)
decimals() should be declared external:
- McGreen.decimals() (#761-763)
totalSupply() should be declared external:
- McGreen.totalSupply() (#765-767)
transfer(address,uint256) should be declared external:
- McGreen.transfer(address,uint256) (#774-777)
allowance(address,address) should be declared external:
- McGreen.allowance(address,address) (#779-781)
approve(address,uint256) should be declared external:
- McGreen.approve(address,uint256) (#783-786)
transferFrom(address,address,uint256) should be declared external:
- McGreen.transferFrom(address,address,uint256) (#788-792)
increaseAllowance(address,uint256) should be declared external:
- McGreen.increaseAllowance(address,uint256) (#794-797)
decreaseAllowance(address,uint256) should be declared external:
- McGreen.decreaseAllowance(address,uint256) (#799-802)
isExcludedFromReward(address) should be declared external:
- McGreen.isExcludedFromReward(address) (#804-806)
totalFees() should be declared external:
- McGreen.totalFees() (#808-810)
deliver(uint256) should be declared external:
- McGreen.deliver(uint256) (#812-819)
reflectionFromToken(uint256,bool) should be declared external:
- McGreen.reflectionFromToken(uint256,bool) (#821-830)
excludeFromReward(address) should be declared external:
- McGreen.excludeFromReward(address) (#838-845)
excludeFromFee(address) should be declared external:
- McGreen.excludeFromFee(address) (#870-872)
includeInFee(address) should be declared external:
- McGreen.includeInFee(address) (#874-876)
setSwapAndLiquifyEnabled(bool) should be declared external:
- McGreen.setSwapAndLiquifyEnabled(bool) (#899-902)
isExcludedFromFee(address) should be declared external:
- McGreen.isExcludedFromFee(address) (#993-995)
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