CertiK $SMRAT

CertiK audit report on $SMRAT

Due to the CertiK audit of $SMRAT we had to change our $MNRY token contract.

Certik Audit: https://www.certik.org/projects/moonratfinance

$MNRT contract: https://github.com/moonery-io/moonery-contracts/blob/main/contracts/Moonery.sol

Findings

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