Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
MCYen.deadWallet (#707) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#423-426)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#432-436)
geUnlockTime() should be declared external:
- Ownable.geUnlockTime() (#438-440)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#443-448)
unlock() should be declared external:
- Ownable.unlock() (#451-456)
name() should be declared external:
- MCYen.name() (#763-765)
symbol() should be declared external:
- MCYen.symbol() (#767-769)
decimals() should be declared external:
- MCYen.decimals() (#771-773)
totalSupply() should be declared external:
- MCYen.totalSupply() (#775-777)
transfer(address,uint256) should be declared external:
- MCYen.transfer(address,uint256) (#784-787)
allowance(address,address) should be declared external:
- MCYen.allowance(address,address) (#789-791)
approve(address,uint256) should be declared external:
- MCYen.approve(address,uint256) (#793-796)
transferFrom(address,address,uint256) should be declared external:
- MCYen.transferFrom(address,address,uint256) (#798-802)
increaseAllowance(address,uint256) should be declared external:
- MCYen.increaseAllowance(address,uint256) (#804-807)
decreaseAllowance(address,uint256) should be declared external:
- MCYen.decreaseAllowance(address,uint256) (#809-812)
isExcludedFromReward(address) should be declared external:
- MCYen.isExcludedFromReward(address) (#814-816)
totalFees() should be declared external:
- MCYen.totalFees() (#818-820)
deliver(uint256) should be declared external:
- MCYen.deliver(uint256) (#822-829)
excludeFromReward(address) should be declared external:
- MCYen.excludeFromReward(address) (#837-845)
excludeFromFee(address) should be declared external:
- MCYen.excludeFromFee(address) (#860-862)
includeInFee(address) should be declared external:
- MCYen.includeInFee(address) (#864-866)
setNumTokensSellToAddToLiquidity(uint256) should be declared external:
- MCYen.setNumTokensSellToAddToLiquidity(uint256) (#880-882)
setMaxTxPercent(uint256) should be declared external:
- MCYen.setMaxTxPercent(uint256) (#884-886)
setMaxWalletAmount(uint256) should be declared external:
- MCYen.setMaxWalletAmount(uint256) (#888-890)
setSwapAndLiquifyEnabled(bool) should be declared external:
- MCYen.setSwapAndLiquifyEnabled(bool) (#892-895)
setMarketingWalletAddress(address) should be declared external:
- MCYen.setMarketingWalletAddress(address) (#897-899)
claimTokens() should be declared external:
- MCYen.claimTokens() (#963-965)
isExcludedFromFee(address) should be declared external:
- MCYen.isExcludedFromFee(address) (#997-999)
Use the external attribute for functions never called from the contract.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#540) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#541)
Variable MCYen._transferFromExcluded(address,address,uint256).rReflectFee (#1158) is too similar to MCYen._transferFromExcluded(address,address,uint256).tReflectFee (#1158)
Variable MCYen._transferFromExcluded(address,address,uint256).rReflectFee (#1158) is too similar to MCYen._transferToExcluded(address,address,uint256).tReflectFee (#1137)
Variable MCYen._transferToExcluded(address,address,uint256).rReflectFee (#1137) is too similar to MCYen._transferBothExcluded(address,address,uint256).tReflectFee (#1179)
Variable MCYen._transferStandard(address,address,uint256).rReflectFee (#1117) is too similar to MCYen._transferBothExcluded(address,address,uint256).tReflectFee (#1179)
Variable MCYen._transferBothExcluded(address,address,uint256).rReflectFee (#1179) is too similar to MCYen._transferFromExcluded(address,address,uint256).tReflectFee (#1158)
Variable MCYen._transferToExcluded(address,address,uint256).rReflectFee (#1137) is too similar to MCYen._transferFromExcluded(address,address,uint256).tReflectFee (#1158)
Variable MCYen._transferStandard(address,address,uint256).rReflectFee (#1117) is too similar to MCYen._transferFromExcluded(address,address,uint256).tReflectFee (#1158)
Variable MCYen._transferStandard(address,address,uint256).rReflectFee (#1117) is too similar to MCYen._transferToExcluded(address,address,uint256).tReflectFee (#1137)
Variable MCYen._transferFromExcluded(address,address,uint256).rReflectFee (#1158) is too similar to MCYen._transferBothExcluded(address,address,uint256).tReflectFee (#1179)
Variable MCYen._transferFromExcluded(address,address,uint256).rReflectFee (#1158) is too similar to MCYen._transferStandard(address,address,uint256).tReflectFee (#1117)
Variable MCYen._transferStandard(address,address,uint256).rTransferAmount (#1122) is too similar to MCYen._transferToExcluded(address,address,uint256).tTransferAmount (#1141)
Variable MCYen._transferFromExcluded(address,address,uint256).rTransferAmount (#1163) is too similar to MCYen._transferBothExcluded(address,address,uint256).tTransferAmount (#1183)
Variable MCYen._transferFromExcluded(address,address,uint256).rTransferAmount (#1163) is too similar to MCYen._transferStandard(address,address,uint256).tTransferAmount (#1121)
Variable MCYen._transferToExcluded(address,address,uint256).rTransferAmount (#1142) is too similar to MCYen._transferBothExcluded(address,address,uint256).tTransferAmount (#1183)
Variable MCYen._transferFromExcluded(address,address,uint256).rTransferAmount (#1163) is too similar to MCYen._transferToExcluded(address,address,uint256).tTransferAmount (#1141)
Variable MCYen._transferStandard(address,address,uint256).rTransferAmount (#1122) is too similar to MCYen._transferBothExcluded(address,address,uint256).tTransferAmount (#1183)
Variable MCYen._transferBothExcluded(address,address,uint256).rReflectFee (#1179) is too similar to MCYen._transferStandard(address,address,uint256).tReflectFee (#1117)
Variable MCYen._transferStandard(address,address,uint256).rReflectFee (#1117) is too similar to MCYen._transferStandard(address,address,uint256).tReflectFee (#1117)
Variable MCYen._transferToExcluded(address,address,uint256).rReflectFee (#1137) is too similar to MCYen._transferToExcluded(address,address,uint256).tReflectFee (#1137)
Variable MCYen._transferToExcluded(address,address,uint256).rReflectFee (#1137) is too similar to MCYen._transferStandard(address,address,uint256).tReflectFee (#1117)
Variable MCYen._transferBothExcluded(address,address,uint256).rReflectFee (#1179) is too similar to MCYen._transferBothExcluded(address,address,uint256).tReflectFee (#1179)
Variable MCYen._transferBothExcluded(address,address,uint256).rReflectFee (#1179) is too similar to MCYen._transferToExcluded(address,address,uint256).tReflectFee (#1137)
Variable MCYen._transferToExcluded(address,address,uint256).rTransferAmount (#1142) is too similar to MCYen._transferStandard(address,address,uint256).tTransferAmount (#1121)
Variable MCYen._transferToExcluded(address,address,uint256).rTransferAmount (#1142) is too similar to MCYen._transferToExcluded(address,address,uint256).tTransferAmount (#1141)
Variable MCYen._transferBothExcluded(address,address,uint256).rTransferAmount (#1184) is too similar to MCYen._transferFromExcluded(address,address,uint256).tTransferAmount (#1162)
Variable MCYen._transferStandard(address,address,uint256).rTransferAmount (#1122) is too similar to MCYen._transferFromExcluded(address,address,uint256).tTransferAmount (#1162)
Variable MCYen._transferBothExcluded(address,address,uint256).rTransferAmount (#1184) is too similar to MCYen._transferBothExcluded(address,address,uint256).tTransferAmount (#1183)
Variable MCYen._transferBothExcluded(address,address,uint256).rTransferAmount (#1184) is too similar to MCYen._transferStandard(address,address,uint256).tTransferAmount (#1121)
Variable MCYen._transferBothExcluded(address,address,uint256).rTransferAmount (#1184) is too similar to MCYen._transferToExcluded(address,address,uint256).tTransferAmount (#1141)
Variable MCYen._transferStandard(address,address,uint256).rTransferAmount (#1122) is too similar to MCYen._transferStandard(address,address,uint256).tTransferAmount (#1121)
Variable MCYen._transferToExcluded(address,address,uint256).rTransferAmount (#1142) is too similar to MCYen._transferFromExcluded(address,address,uint256).tTransferAmount (#1162)
Variable MCYen._transferFromExcluded(address,address,uint256).rTransferAmount (#1163) is too similar to MCYen._transferFromExcluded(address,address,uint256).tTransferAmount (#1162)
Prevent variables from having similar names.
Additional information: link
MCYen._getReflectFee(uint256) (#919-924) performs a multiplication on the result of a division:
-tFee = tAmount.mul(_taxFee).div(100) (#921)
-rFee = tFee.mul(currentRate) (#922)
MCYen._takeLiquidity(address,uint256) (#938-948) performs a multiplication on the result of a division:
-tLiquidity = tAmount.mul(_liquidityFee).div(100) (#941)
-rLiquidity = tLiquidity.mul(currentRate) (#942)
MCYen._takeBurn(address,uint256) (#950-960) performs a multiplication on the result of a division:
-tBurn = tAmount.mul(_burnFee).div(100) (#953)
-rBurn = tBurn.mul(currentRate) (#954)
Consider ordering multiplication before division.
Additional information: link
Low level call in Address.sendValue(address,uint256) (#291-297):
- (success) = recipient.call{value: amount}() (#295)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#357-378):
- (success,returndata) = target.call{value: weiValue}(data) (#361)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Variable Ownable._owner (#394) is not in mixedCase
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#495) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#496) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#513) is not in mixedCase
Function IUniswapV2Router01.WETH() (#535) is not in mixedCase
Parameter MCYen.setSwapAndLiquifyEnabled(bool)._enabled (#892) is not in mixedCase
Parameter MCYen.calculateTaxFee(uint256)._amount (#967) is not in mixedCase
Parameter MCYen.calculateLiquidityFee(uint256)._amount (#973) is not in mixedCase
Variable MCYen._taxFee (#695) is not in mixedCase
Variable MCYen._liquidityFee (#698) is not in mixedCase
Variable MCYen._burnFee (#701) is not in mixedCase
Variable MCYen._maxTxAmount (#713) is not in mixedCase
Variable MCYen._maxWalletAmount (#715) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#237)" inContext (#231-240)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in MCYen._transfer(address,address,uint256) (#1009-1063):
External calls:
- swapAndLiquify(contractTokenBalance) (#1050)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,marketingWallet,block.timestamp) (#1079-1085)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- _rOwned[deadWallet] = _rOwned[deadWallet].add(rBurn) (#955)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#943)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1123)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1143)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1124)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1165)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1186)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1145)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1166)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1188)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- _rTotal = _rTotal.sub(rFee) (#905)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- _tOwned[deadWallet] = _tOwned[deadWallet].add(tBurn) (#956)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#944)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1164)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1185)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1144)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1187)
Apply the check-effects-interactions pattern.
Additional information: link
MCYen._transferToExcluded(address,address,uint256).tFeeAmount (#1134) is a local variable never initialized
MCYen._transferFromExcluded(address,address,uint256).tFeeAmount (#1155) is a local variable never initialized
MCYen._transferStandard(address,address,uint256).tFeeAmount (#1114) is a local variable never initialized
MCYen._transferBothExcluded(address,address,uint256).tFeeAmount (#1176) is a local variable never initialized
Initialize all the variables. If a variable is meant to be initialized to zero, explicitly set it to zero to improve code readability.
Additional information: link
MCYen.allowance(address,address).owner (#789) shadows:
- Ownable.owner() (#404-406) (function)
MCYen._approve(address,address,uint256).owner (#1001) shadows:
- Ownable.owner() (#404-406) (function)
Rename the local variables that shadow another component.
Additional information: link
MCYen.setTaxFeePercent(uint256) (#868-870) should emit an event for:
- _taxFee = taxFee (#869)
MCYen.setLiquidityFeePercent(uint256) (#872-874) should emit an event for:
- _liquidityFee = liquidityFee (#873)
MCYen.setBurnFeePercent(uint256) (#876-878) should emit an event for:
- _burnFee = burnFee (#877)
MCYen.setNumTokensSellToAddToLiquidity(uint256) (#880-882) should emit an event for:
- numTokensSellToAddToLiquidity = swapNumber * 10 ** _decimals (#881)
MCYen.setMaxTxPercent(uint256) (#884-886) should emit an event for:
- _maxTxAmount = maxTxPercent * 10 ** _decimals (#885)
MCYen.setMaxWalletAmount(uint256) (#888-890) should emit an event for:
- _maxWalletAmount = maxWalletAmount * 10 ** _decimals (#889)
Emit an event for critical parameter changes.
Additional information: link
MCYen.setMarketingWalletAddress(address).marketingWalletAddress_ (#897) lacks a zero-check on :
- marketingWallet = marketingWalletAddress_ (#898)
Check that the address is not zero.
Additional information: link
Reentrancy in MCYen._transfer(address,address,uint256) (#1009-1063):
External calls:
- swapAndLiquify(contractTokenBalance) (#1050)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,marketingWallet,block.timestamp) (#1079-1085)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- _burnFee = _previousBurnFee (#994)
- _burnFee = 0 (#988)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- _liquidityFee = _previousLiquidityFee (#993)
- _liquidityFee = 0 (#987)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- _previousBurnFee = _burnFee (#984)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- _previousLiquidityFee = _liquidityFee (#983)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- _previousTaxFee = _taxFee (#982)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- _tFeeTotal = _tFeeTotal.add(tFee) (#906)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- _taxFee = _previousTaxFee (#992)
- _taxFee = 0 (#986)
Reentrancy in MCYen.constructor() (#726-760):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#747-748)
State variables written after the call(s):
- _isExcludedFromFee[tokenOwner] = true (#754)
- _isExcludedFromFee[address(this)] = true (#755)
- _isExcludedFromFee[address(_uniswapV2Router)] = true (#756)
- _owner = tokenOwner (#758)
- uniswapV2Router = _uniswapV2Router (#751)
Reentrancy in MCYen.transferFrom(address,address,uint256) (#798-802):
External calls:
- _transfer(sender,recipient,amount) (#799)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,marketingWallet,block.timestamp) (#1079-1085)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#800)
- _allowances[owner][spender] = amount (#1005)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in MCYen._transfer(address,address,uint256) (#1009-1063):
External calls:
- swapAndLiquify(contractTokenBalance) (#1050)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,marketingWallet,block.timestamp) (#1079-1085)
Event emitted after the call(s):
- Transfer(sender,deadWallet,tBurn) (#958)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- Transfer(sender,address(this),tLiquidity) (#946)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- Transfer(sender,recipient,tTransferAmount) (#1126)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- Transfer(sender,recipient,tTransferAmount) (#1168)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- Transfer(sender,recipient,tTransferAmount) (#1147)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
- Transfer(sender,recipient,tTransferAmount) (#1190)
- _tokenTransfer(from,to,amount,takeFee) (#1062)
Reentrancy in MCYen.constructor() (#726-760):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#747-748)
Event emitted after the call(s):
- Transfer(address(0),tokenOwner,_tTotal) (#759)
Reentrancy in MCYen.transferFrom(address,address,uint256) (#798-802):
External calls:
- _transfer(sender,recipient,amount) (#799)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,marketingWallet,block.timestamp) (#1079-1085)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1006)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#800)
Apply the check-effects-interactions pattern.
Additional information: link
Ownable.unlock() (#451-456) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(now > _lockTime,Contract is locked until 7 days) (#453)
Avoid relying on block.timestamp.
Additional information: link
Address.isContract(address) (#264-273) uses assembly
- INLINE ASM (#271)
Address._functionCallWithValue(address,bytes,uint256,string) (#357-378) uses assembly
- INLINE ASM (#370-373)
Do not use evm assembly.
Additional information: link
MCYen.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) (#357-378) is never used and should be removed
Address.functionCall(address,bytes) (#317-319) is never used and should be removed
Address.functionCall(address,bytes,string) (#327-329) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#342-344) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#352-355) is never used and should be removed
Address.isContract(address) (#264-273) is never used and should be removed
Address.sendValue(address,uint256) (#291-297) is never used and should be removed
Context._msgData() (#236-239) is never used and should be removed
MCYen.calculateLiquidityFee(uint256) (#973-977) is never used and should be removed
MCYen.calculateTaxFee(uint256) (#967-971) is never used and should be removed
SafeMath.mod(uint256,uint256) (#209-211) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#225-228) is never used and should be removed
Remove unused functions.
Additional information: link
MCYen.slitherConstructorVariables() (#673-1193) uses literals with too many digits:
- deadWallet = address(0x000000000000000000000000000000000000dEaD) (#707)
Use: Ether suffix, Time suffix, or The scientific notation
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