Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
BenchmarkProtocol.setAccountingAddress(address).accountingAddress (#89) lacks a zero-check on :
- accounting = accountingAddress (#91)
Check that the address is not zero.
Additional information: link
SafeMath.add(uint256,uint256) (#23-27) is never used and should be removed
SafeMath.div(uint256,uint256) (#13-16) is never used and should be removed
SafeMath.mul(uint256,uint256) (#4-11) is never used and should be removed
Remove unused functions.
Additional information: link
Pragma version^0.8.12 (#1) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
solc-0.8.14 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 BenchmarkProtocol._decimals (#55) is not in mixedCase
Variable BenchmarkProtocol._totalSupply (#56) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
BenchmarkProtocol.constructor() (#61-65) uses literals with too many digits:
- _totalSupply = 1000000 * 10 ** 9 (#63)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
BenchmarkProtocol.name (#53) should be constant
BenchmarkProtocol.symbol (#54) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
totalSupply() should be declared external:
- BenchmarkProtocol.totalSupply() (#69-71)
decimals() should be declared external:
- BenchmarkProtocol.decimals() (#73-75)
balanceOf(address) should be declared external:
- BenchmarkProtocol.balanceOf(address) (#81-83)
allowance(address,address) should be declared external:
- BenchmarkProtocol.allowance(address,address) (#85-87)
setAccountingAddress(address) should be declared external:
- BenchmarkProtocol.setAccountingAddress(address) (#89-92)
renounceOwnership() should be declared external:
- BenchmarkProtocol.renounceOwnership() (#94-98)
transfer(address,uint256) should be declared external:
- BenchmarkProtocol.transfer(address,uint256) (#100-103)
transferFrom(address,address,uint256) should be declared external:
- BenchmarkProtocol.transferFrom(address,address,uint256) (#105-109)
approve(address,uint256) should be declared external:
- BenchmarkProtocol.approve(address,uint256) (#111-115)
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