The Arcona Ecosystem creates a Digital Land - a layer of Augmented Reality uniting the physical and virtual worlds into a single information environment perfectly linked to our real world. It is designed for everyday user interactive experience with augmented, virtual and mixed reality multimedia content in real world locations.
Unable to verify that contract auditor is trusted: Certik, Quantstamp, Hacken, Solidity, Paladinsec, Openzeppelin, Verichains
Reentrancy in ARCONA.exchange(uint256) (contracts/ARCONA.sol#33-41):
External calls:
- require(bool,string)(exchangeToken.transferFrom(msg.sender,address(this),_exchangeTokensAmount),[E-87] - Failed to transfer token.) (contracts/ARCONA.sol#39)
State variables written after the call(s):
- _mint(msg.sender,_tokensToMint) (contracts/ARCONA.sol#40)
- _totalSupply += amount (@openzeppelin/contracts/token/ERC20/ERC20.sol#238)
Apply the check-effects-interactions pattern.
Additional information: link
ARCONA.setTokenPrice(uint256) (contracts/ARCONA.sol#27-29) should emit an event for:
- tokenPrice = _newPrice (contracts/ARCONA.sol#28)
Emit an event for critical parameter changes.
Additional information: link
Reentrancy in ARCONA.exchange(uint256) (contracts/ARCONA.sol#33-41):
External calls:
- require(bool,string)(exchangeToken.transferFrom(msg.sender,address(this),_exchangeTokensAmount),[E-87] - Failed to transfer token.) (contracts/ARCONA.sol#39)
State variables written after the call(s):
- _mint(msg.sender,_tokensToMint) (contracts/ARCONA.sol#40)
- _balances[account] += amount (@openzeppelin/contracts/token/ERC20/ERC20.sol#239)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in ARCONA.exchange(uint256) (contracts/ARCONA.sol#33-41):
External calls:
- require(bool,string)(exchangeToken.transferFrom(msg.sender,address(this),_exchangeTokensAmount),[E-87] - Failed to transfer token.) (contracts/ARCONA.sol#39)
Event emitted after the call(s):
- Transfer(address(0),account,amount) (@openzeppelin/contracts/token/ERC20/ERC20.sol#240)
- _mint(msg.sender,_tokensToMint) (contracts/ARCONA.sol#40)
Apply the check-effects-interactions pattern.
Additional information: link
Different versions of Solidity is used:
- Version used: ['0.8.0', '^0.8.0']
- ^0.8.0 (@openzeppelin/contracts/access/Ownable.sol#3)
- ^0.8.0 (@openzeppelin/contracts/token/ERC20/ERC20.sol#3)
- ^0.8.0 (@openzeppelin/contracts/token/ERC20/IERC20.sol#3)
- ^0.8.0 (@openzeppelin/contracts/utils/Context.sol#3)
- ^0.8.0 (@openzeppelin/contracts/utils/math/SafeMath.sol#3)
- 0.8.0 (contracts/ARCONA.sol#2)
Use one Solidity version.
Additional information: link
Context._msgData() (@openzeppelin/contracts/utils/Context.sol#20-23) is never used and should be removed
ERC20._burn(address,uint256) (@openzeppelin/contracts/token/ERC20/ERC20.sol#254-265) is never used and should be removed
SafeMath.div(uint256,uint256,string) (@openzeppelin/contracts/utils/math/SafeMath.sol#190-195) is never used and should be removed
SafeMath.mod(uint256,uint256) (@openzeppelin/contracts/utils/math/SafeMath.sol#150-152) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (@openzeppelin/contracts/utils/math/SafeMath.sol#212-217) is never used and should be removed
SafeMath.sub(uint256,uint256) (@openzeppelin/contracts/utils/math/SafeMath.sol#106-108) is never used and should be removed
SafeMath.sub(uint256,uint256,string) (@openzeppelin/contracts/utils/math/SafeMath.sol#167-172) is never used and should be removed
SafeMath.tryAdd(uint256,uint256) (@openzeppelin/contracts/utils/math/SafeMath.sol#21-27) is never used and should be removed
SafeMath.tryDiv(uint256,uint256) (@openzeppelin/contracts/utils/math/SafeMath.sol#63-68) is never used and should be removed
SafeMath.tryMod(uint256,uint256) (@openzeppelin/contracts/utils/math/SafeMath.sol#75-80) is never used and should be removed
SafeMath.tryMul(uint256,uint256) (@openzeppelin/contracts/utils/math/SafeMath.sol#46-56) is never used and should be removed
SafeMath.trySub(uint256,uint256) (@openzeppelin/contracts/utils/math/SafeMath.sol#34-39) is never used and should be removed
Remove unused functions.
Additional information: link
Pragma version^0.8.0 (@openzeppelin/contracts/access/Ownable.sol#3) allows old versions
Pragma version^0.8.0 (@openzeppelin/contracts/token/ERC20/ERC20.sol#3) allows old versions
Pragma version^0.8.0 (@openzeppelin/contracts/token/ERC20/IERC20.sol#3) allows old versions
Pragma version^0.8.0 (@openzeppelin/contracts/utils/Context.sol#3) allows old versions
Pragma version^0.8.0 (@openzeppelin/contracts/utils/math/SafeMath.sol#3) allows old versions
Pragma version0.8.0 (contracts/ARCONA.sol#2) allows old versions
solc-0.8.0 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
Parameter ARCONA.setTokenPrice(uint256)._newPrice (contracts/ARCONA.sol#27) is not in mixedCase
Parameter ARCONA.exchange(uint256)._exchangeTokensAmount (contracts/ARCONA.sol#33) is not in mixedCase
Parameter ARCONA.calculateTokensAmountAfterExchange(uint256)._exchangeTokensAmount (contracts/ARCONA.sol#45) is not in mixedCase
Parameter ARCONA.mint(address,uint256)._account (contracts/ARCONA.sol#52) is not in mixedCase
Parameter ARCONA.mint(address,uint256)._amount (contracts/ARCONA.sol#52) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (@openzeppelin/contracts/utils/Context.sol#21)" inContext (@openzeppelin/contracts/utils/Context.sol#15-24)
Remove redundant statements if they congest code but offer no value.
Additional information: link
ARCONA.calculateTokensAmountAfterExchange(uint256) (contracts/ARCONA.sol#45-47) uses literals with too many digits:
- _exchangeTokensAmount.mul(1000000000000000000).div(tokenPrice) (contracts/ARCONA.sol#46)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
ARCONA.maxSupply (contracts/ARCONA.sol#18) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (@openzeppelin/contracts/access/Ownable.sol#54-57)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (@openzeppelin/contracts/access/Ownable.sol#63-67)
name() should be declared external:
- ERC20.name() (@openzeppelin/contracts/token/ERC20/ERC20.sol#59-61)
symbol() should be declared external:
- ERC20.symbol() (@openzeppelin/contracts/token/ERC20/ERC20.sol#67-69)
decimals() should be declared external:
- ERC20.decimals() (@openzeppelin/contracts/token/ERC20/ERC20.sol#84-86)
balanceOf(address) should be declared external:
- ERC20.balanceOf(address) (@openzeppelin/contracts/token/ERC20/ERC20.sol#98-100)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (@openzeppelin/contracts/token/ERC20/ERC20.sol#110-113)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (@openzeppelin/contracts/token/ERC20/ERC20.sol#118-120)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (@openzeppelin/contracts/token/ERC20/ERC20.sol#129-132)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (@openzeppelin/contracts/token/ERC20/ERC20.sol#147-155)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (@openzeppelin/contracts/token/ERC20/ERC20.sol#169-172)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (@openzeppelin/contracts/token/ERC20/ERC20.sol#188-194)
Use the external attribute for functions never called from the contract.
Additional information: link
Young tokens have high risks of scam / price dump / death
Young tokens have high risks of price dump / death
Last post in Twitter was more than 30 days ago