CertiK $SMRAT
CertiK audit report on $SMRAT
Due to the CertiK audit of $SMRAT we had to change our $MNRY token contract.
ID | Category | Severity | Location | Changes |
MRC-01 Incorrect error message | Logical Issue | 3 | | |
MRC-02 Redundant code | Logical Issue | 🔵 | | |
MRC-03 Missing Range Check for Input Variable | Centralization / Privilege | 3 | Multiple locations | changed to taxFee, liquidityFee and maxTxAmount |
MRC-04 Centralized risk in addLiquidity | Centralization / Privilege | 🔴 | | |
MRC-05 Inappropriate Variable Initialization | Logical Issue | 2 | Moonery.sol:475 | By init: _maxTxAmount= _tTotal.mul(5).div(10000) |
MRC-06 Tautology or Contradiction Issue | Gas Optimization | 3 | Moonery.sol:586 | changed to: require(balanceOf(receiver) > 0, "CL2"); |
MRC-07 Variable Typo | Coding Style | 🔵 | | |
MRC-08 Missing Events for Significant Transactions | Coding Style | 🔵 | | |
MRC-09 Inappropriate Location of Constant Declaration | Coding Style | 🔵 | Moonery.sol:471~487 | Declared constants at the beginning of the contract. |
MRC-10 Redundant Setting | Coding Style | 🔵 | Multiple locations | Appropriate values set once |
MRC-11 Missing Check for Reentrancy Attack | Logical Issue | 2 | Moonery.sol:197~211 | Changed and tested on test net. But these changes could lock all transfers of $MNRY. So we reverted the changes. |
| | | | |
Last modified 2yr ago