CertiK $SMRAT

Search…

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.
​
​
​
​
​

Smart Contracts - Previous

Binance Smart Chain

Next - Audit

CertiK $SAFEMOON

Last modified 8mo ago

Copy link

Contents

Findings