Audits that take no more than 20 days to complete, now start within 10 days 

In the contract `MoonNation` the role `_owner` has authority over the functions shown in the diagram below.
Any compromise to the `_owner` account may allow the hacker to take advantage of this authority.
![](https://accelerator-tasks-prod.s3.amazonaws.com/3eecc6d3ec63452b925060fca9c59e7e/diagrams/centralization_MoonNation__owner.svg)

In the contract `Ownable` the role `_owner` has authority over the functions shown in the diagram below.
Any compromise to the `_owner` account may allow the hacker to take advantage of this authority.
![](https://accelerator-tasks-prod.s3.amazonaws.com/3eecc6d3ec63452b925060fca9c59e7e/diagrams/centralization_Ownable__owner.svg)

In the contract `Ownable` the role `_previousOwner` has authority over the functions shown in the diagram below.
Any compromise to the `_previousOwner` account may allow the hacker to take advantage of this authority.
![](https://accelerator-tasks-prod.s3.amazonaws.com/3eecc6d3ec63452b925060fca9c59e7e/diagrams/centralization_Ownable__previousOwner.svg)

Centralization Risks in MoonNationV2.sol

`384400000 * 10**9` tokens were sent to the `msg.sender` when deploying the contract. 
This could be a centralization risk as the deployer can distribute tokens without obtaining the consensus of the community.

Initial Token Distribution

When an account is included in the reward list through `includeInReward()`, the account's `rOwned` balance is not updated to reflect the change in `rTotal`. This may lead to a miscalculation of the account's deserved reward. For example, when an account on the reward list is excluded, it's `tOwned` balance would be locked for a period of time before the account is included back again. However, in that situation, the universal `rate` is likely to decrease in that period so that by reflecting the `rOwned` the account would receive more tokens than deserved which in effect cancels out that excluded period. So the `_rOwned[account]` should be updated to `reflectionFromToken(_tOwned[account])` instead of `_tOwned[account]`.

Incorrect `_rOwned[account]` Update

The `_previousTaxFee`, `_previousBurnFee` and `_previousMarketingAndDev` are used to restore the origin value of `_taxFee`, `_burnFee` and `marketingAndDevelopment` when the transfer is free. However, these variables are used to record the value before the update. This will cause `_taxFee`, `_burnFee` and `marketingAndDevelopment` to be updated to the pre-update value instead of the latest modified value once the transfer is free. So these variables should be updated in the function `removeAllFee()` instead of `setTaxFeePercent()`, `setBurnFeePercent()` and `setMarketingAndDevFeePercent()`.

Incorrect `_previousTaxFee`, `_previousBurnFee`  and `_previousMarketingAndDev` Update

An owner is possible to gain ownership of the contract even if he calls the function `transferOwnership` to transfer the ownership. This can be achieved by performing the following operations:
1. Call `lock` to lock the contract. The variable `_previousOwner` is set to the current owner.
2. Call `unlock` to unlock the contract.
3. Call `transferOwnership` to leave the contract without an owner. 
4. Call `unlock` to regain ownership.


Possible To Gain Ownership After Transferring The Contract Ownership

Addresses should be checked before assignment or external calls to make sure they are not zero addresses.

Missing Zero Address Validation

The variables `_taxFee`, `_burnFee`, `_maxTxAmount`, `_maxWalletToken`, `marketingAndDevelopment` do not have reasonable boundaries, so they can be given arbitrary values after deploying.

Lack of Reasonable Boundary

According to logic, only excluded accounts have `_tOwned`. So the `_burn` and `_marketingAndDev` should check if they are excluded before updating `_tOwned` to prevent unpredictable mistakes.

No Check Account  Is Excluded

There should always be events emitted in the sensitive functions that are controlled by centralization roles.

Missing Emit Events

`public` functions that are never called by the contract could be declared as `external`. `external` functions are more efficient than `public` functions.

Improper Usage of `public` and `external` Type

The linked variables assigned in the constructor can be declared as `immutable`. Immutable state variables can be assigned during contract creation but will remain constant throughout the lifetime of a deployed contract. A big advantage of immutable variables is that reading them is significantly cheaper than reading from regular state variables since they will not be stored in storage.

Variables That Could Be Declared as Immutable

The linked statements do not affect the functionality of the codebase and appear to be either leftovers from test code or older functionality.

Redundant Code Components

Literals with many digits are difficult to read and review.

Too Many Digits

It is recommended to add `indexed` keyword for parameters in events, which makes it easier for users to navigate event logs.

Address Type Could Be Indexed In Events

Moon Nation V2 Addendum

MoonNationV2.sol

Possible To Gain Ownership After Renouncing The Contract Ownership

Typos In The Contract

Set `constant` to Variables

Third Party Dependencies

Hardcode Address

The Purpose Of Function `deliver`

`rOwned` Not Updated

Return Value Not Handled

Centralized Risk In `addLiquidity`

Redundant Code

Contract Gains Non-withdrawable BNB Via The `swapAndLiquify` Function

Potential Sandwich Attacks

Lack of Input Validation

Function Visibility Optimization

Centralization Risk

Lack of Zero Address Validation

Lack of Check `takeFee`

Logic Issue of Function `removeAllFee()`

Visibility Specifiers Missing

Moon Nation Game

projects

contracts

MoonNation.sol

MoonNationGame

Moon Nation Game 

Binance receives green light for crypto services in Dubai

FTX Token price risks 30% plunge as a 23M FTT 'part' moves to Binance - Crypto Capital Today

Hundreds of Millions of XRP Moved from Binance, Here's What Happened Here

Binance Coin Shows Weakness As Price Sits On Key Support; Is $200 Possible?

Binance intends to acquire Bankman-Fried's FTX amid 'significant liquidity crunch'

Sam Bankman-Fried Apologises, Tells Binance ‘You Won’

FTX’s ETH Reserves Drop to the Lowest Level Since November 2020 After Binance Begins Dumping all FTX Tokens on its Books

Binance confirms it's backing out of deal to buy FTX assets - MarketWatch

Singapore defends its treatment of Binance after FTX ‘debacle’

Crypto News Summary: Binance Launches Oracle, UvToken and Team Finance Exploited, Blockchain.com Debuts Visa Card

Saudi Crown Prince Mohammed bin Salman Named Prime Minister - WSJ

Luna Classic, Remnant of Terra Collapse, Jumps 60% as Binance Unveils Burn Scheme

Binance CEO met with Abu Dhabi investors about industry recovery fund: Bloomberg

Binance Pool Starts $500 Million Fund to Support Bitcoin Mining

Crypto Exchange Binance Receives License to Operate in Kazakhstan

FTX Acquisition And Crypto Crash Is Bad For The Community, Says CZ Binance

Tron TRX Price Prediction as Binance Adds TRX to Index – Time to Buy?

Binance Dumps FTX, SHIB Burn Rate up 5,800%, U.S. Lawyer Predicts Date of Ripple Case Resolution: Crypto News Digest by U.Today

Binance Pulls Out of FTX Rescue, Putting Company on the Brink of Collapse

Bitcoin Notches Highest Trading Volume In Over 3 Months, Binance Data Shows

Binance deploys $1 billion to keep crypto industry afloat after FTX collapse

Moon Nation Game

Highlights & Alerts

This is just a placeholder

highlight placeholder lorem Ipsum...

highlight lorem

This is a alert demo

Alert demo

Code Audit History

14

0

2

0

12