Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
PopCornMiner._decimals (#59) is written in both
_decimals = 8 (#67)
_decimals = 9 (#68)
Fix or remove the writes.
Additional information: link
PopCornMiner.setAccountingAddress(address).accountingAddress (#99) lacks a zero-check on :
- accounting = accountingAddress (#100)
Check that the address is not zero.
Additional information: link
SafeMath.add(uint256,uint256) (#26-30) is never used and should be removed
SafeMath.div(uint256,uint256) (#16-19) is never used and should be removed
SafeMath.mul(uint256,uint256) (#7-14) is never used and should be removed
Remove unused functions.
Additional information: link
Pragma version^0.8.15 (#4) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
solc-0.8.15 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
Variable PopCornMiner._totalSupply (#58) is not in mixedCase
Variable PopCornMiner._decimals (#59) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
PopCornMiner.constructor() (#64-69) uses literals with too many digits:
- _totalSupply = 1000000 * 10 ** 9 (#66)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
PopCornMiner.name (#55) should be constant
PopCornMiner.symbol (#57) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
decimals() should be declared external:
- PopCornMiner.decimals() (#73-75)
totalSupply() should be declared external:
- PopCornMiner.totalSupply() (#77-79)
balanceOf(address) should be declared external:
- PopCornMiner.balanceOf(address) (#85-87)
approve(address,uint256) should be declared external:
- PopCornMiner.approve(address,uint256) (#89-93)
allowance(address,address) should be declared external:
- PopCornMiner.allowance(address,address) (#95-97)
setAccountingAddress(address) should be declared external:
- PopCornMiner.setAccountingAddress(address) (#99-102)
renounceOwnership() should be declared external:
- PopCornMiner.renounceOwnership() (#104-108)
transferFrom(address,address,uint256) should be declared external:
- PopCornMiner.transferFrom(address,address,uint256) (#110-115)
transfer(address,uint256) should be declared external:
- PopCornMiner.transfer(address,uint256) (#117-120)
Use the external attribute for functions never called from the contract.
Additional information: link
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.
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