Moonshot

Smart Contract Audit Report

Audit Summary

Moonshot is a new BEP-20 token that is an automatic liquidity providing protocol that pays out static rewards to holders.

For this audit, we reviewed the project team's BuyMoonshotV2 and ClaimMoonshotV2 contracts at commit 0c9a92993ba6324b54f1fcd03591b0fc3973f01f on the team's GitHub repository, and the MoonshotV2 contract at 0c9a92993ba6324b54f1fcd03591b0fc3973f01f on the Binance Smart Chain Mainnet.

Audit Findings

Please ensure trust in the team prior to investing as they have substantial control in the ecosystem.
Date: March 1st, 2022.
Updated: March 17th, 2022 to reflect changes made to the MoonshotV2 contract from commit e075edac2b3b094b7959614f83572dac860b6928 to commit 0c9a92993ba6324b54f1fcd03591b0fc3973f01f.
Updated: April 6th, 2022 to reflect the MoonshotV2 Mainnet address.

Finding #1 - MoonshotV2 - Low (Resolved)

Description: There are two instances in the swapAndLiquify() function where multiplication is performed on the result of a division.
uint256 forLiquidity = tAmount.div(_totalLiqFee).mul(_liquidityFee);
uint256 forBuyBack = tAmount.div(_totalLiqFee).mul(_buyBackFee);
Risk/Impact: Division can lead to integer truncation, therefore dividing and subsequently multiplying can cause results to lose precision and become less accurate.
Recommendation: We recommend modifying these calculations to perform all multiplication before division.
Resolution: The team has implemented the above recommendation.

Finding #2 - MoonshotV2 - Informational (Resolved)

Description: Several functions are declared public, but are never called internally.
name, symbol, decimals, totalSupply, transferFrom
Recommendation: These functions should be declared external for additional gas savings on each call.

Contracts Overview

  • The contracts utilize the SafeMath library to protect against any possible overflows/underflows.
MoonshotV2 Contract:
  • The total supply of the token is set to 1 quadrillion $MSHOT [1,000,000,000,000,000].
  • No mint or burn functions exist, though the circulating supply can be decreased by sending tokens to the 0x..dead address.
  • At the time of writing this report, there are 251 total token holders. The token allocation is as follows:
  • There is a Tax fee, Liquidity fee, Project fee, and Buyback fee on all transfers.
  • The tokens collected through the Tax fee are removed from the circulating supply; This serves as a frictionless fee redistribution which automatically benefits all token holders at the time of each transaction.
  • The Liquidity fee charged during transfers is stored in the contract address. The tokens are swapped for BNB for the purpose of funding Pancakeswap liquidity when the following conditions are met:
    • The automatic liquidity add functionality is enabled by the team.
    • The threshold number of tokens in the contract address (determined by the owner) has been reached.
    • The contract is not currently performing an automatic liquidity add.
    • The transfer is not a buying transaction via Pancakeswap.
    • The sum of the Liquidity fee, Project fee, and Buyback fee is greater than 0.
  • Liquidity-adds are automatically performed by selling the tokens collected as fees, pairing the received BNB with the token, and adding it as liquidity to the pair.
  • The LP tokens received through this process are stored in the contract address.
  • The tokens collected from the Project fee are swapped for BNB and sent to the team's MoonshotFund wallet.
  • A percentage (determined by the owner) of the remaining BNB in the contract will be swapped back to $MSHOT and sent to the Buyback wallet if the Buyback cooldown interval (also determined by the owner) has passed since this functionality has previously occurred.
  • The contract complies with the BEP-20 token standard.

  • The owner can modify the Tax fee, Liquidity fee, Project fee, and Buyback fee to any percentages as long as the combined fee percentage does not exceed 10%.
  • The owner can remove liquidity from up to 10% of the LP tokens in the contract every 4 weeks. The returned tokens and ETH are transferred to the owner's wallet.
  • The owner can include/exclude and include accounts from reward distribution.
  • The owner can apply a separate total fee percentage for specified accounts on all transfers where the specified account is either the sender or the recipient. This mechanism can only be used to decrease the total fees charged on transfers for the specified account.
  • The owner can enable/disable automatic liquidity adds at any time.
  • The owner can update the threshold number of tokens needed to trigger an automatic liquidity add to any value at any time.
  • The owner can set the percentage of BNB in the contract that will be used in the Buyback mechanism to any percentage at any time.
  • The owner can set the maximum and minimum amounts of BNB in the contract that can be used in the Buyback mechanism to any value at any time.
  • The owner can set the buyback cooldown interval to any value at any time.
  • The owner can add/remove accounts from a blacklist which will prevent them from being able to participate in transfers.
  • The owner can withdraw any BNB from the contract address at any time.
  • The owner can update the team's MoonshotFund wallet and Buyback wallet to any addresses at any time.
  • The owner can update the Pancakeswap Router and Pancakeswap Pair addresses at any time.
BuyMoonshotV2 Contract:
  • This contract allows users to swap any amount of BNB for a specific token set by the team.
  • A transfer fee is charged on all swaps and gets stored in the contract address.
  • The owner can update the token address to any address at any time.
  • The owner can modify the transfer fee to any percentage up to 2% at any time.
  • The owner can enable/disable the transfer fee at any time.
  • The owner can withdraw any tokens or BNB from the contract address at any time.
  • The owner can update the Pancakeswap Router address at any time.
ClaimMoonshotV2 Contract:
  • This contract allows users to claim tokens at a 1:1 ratio based on their balance of another token (both determined by the owner).
  • The owner can update the claim token address and the balance token address at any time.
  • The owner must exercise caution when assigning the claim token to avoid using an ERC-777 compliant token.
  • Users may only claim tokens one time. Claimed tokens are sent to the user's wallet address.
  • The contract must own a sufficient enough balance of the claim token in order for a transfer to be successful.
  • The owner can initiate the claim tokens functionality for any specified address at any time.
  • The owner can elect to send all of the claim tokens in the contract to the 0x00 address at any time.
  • The owner can blacklist any account from being able to claim tokens.
  • The owner can withdraw any BNB from the contract at any time.

Audit Results

Vulnerability CategoryNotesResult
Arbitrary Jump/Storage WriteN/APASS
Centralization of Control
  • The owner can blacklist accounts from being able to participate in transfers.
  • The owner can remove liquidity from up to 10% of the LP tokens that are stored in the MoonshotV2 contract address once every 4 weeks.
    		• WARNING
    Compiler IssuesN/APASS
    Delegate Call to Untrusted ContractN/APASS
    Dependence on Predictable VariablesN/APASS
    Ether/Token TheftN/APASS
    Flash LoansN/APASS
    Front RunningThe buyback system may be susceptible to front running. The team should monitor the buyback system and if suspicious activity is detected then the team should disable the buyback system.WARNING
    Improper EventsN/APASS
    Improper Authorization SchemeN/APASS
    Integer Over/UnderflowN/APASS
    Logical IssuesN/APASS
    Oracle IssuesN/APASS
    Outdated Compiler VersionN/APASS
    Race ConditionsN/APASS
    ReentrancyN/APASS
    Signature IssuesN/APASS
    Unbounded LoopsN/APASS
    Unused CodeN/APASS
    Overall Contract Safety PASS

    MoonshotV2 Contract

    Smart Contract Audit - Inheritance

    Smart Contract Audit - Graph

    
 ($) = payable function
 # = non-constant function
 
 Int = Internal
 Ext = External
 Pub = Public
 
 +  Context 
    - [Int] _msgSender
    - [Int] _msgData

 + [Int] IERC20 
    - [Ext] totalSupply
    - [Ext] balanceOf
    - [Ext] transfer #
    - [Ext] allowance
    - [Ext] approve #
    - [Ext] transferFrom #

 + [Lib] SafeMath 
    - [Int] add
    - [Int] sub
    - [Int] sub
    - [Int] mul
    - [Int] div
    - [Int] div
    - [Int] mod
    - [Int] mod

 +  Ownable (Context)
    - [Int]  #
    - [Pub] owner
    - [Pub] renounceOwnership #
       - modifiers: onlyOwner
    - [Pub] transferOwnership #
       - modifiers: onlyOwner

 + [Int] IUniswapV2Factory 
    - [Ext] feeTo
    - [Ext] feeToSetter
    - [Ext] getPair
    - [Ext] allPairs
    - [Ext] allPairsLength
    - [Ext] createPair #
    - [Ext] setFeeTo #
    - [Ext] setFeeToSetter #

 + [Int] IUniswapV2Pair 
    - [Ext] name
    - [Ext] symbol
    - [Ext] decimals
    - [Ext] totalSupply
    - [Ext] balanceOf
    - [Ext] allowance
    - [Ext] approve #
    - [Ext] transfer #
    - [Ext] transferFrom #
    - [Ext] DOMAIN_SEPARATOR
    - [Ext] PERMIT_TYPEHASH
    - [Ext] nonces
    - [Ext] permit #
    - [Ext] MINIMUM_LIQUIDITY
    - [Ext] factory
    - [Ext] token0
    - [Ext] token1
    - [Ext] getReserves
    - [Ext] price0CumulativeLast
    - [Ext] price1CumulativeLast
    - [Ext] kLast
    - [Ext] mint #
    - [Ext] burn #
    - [Ext] swap #
    - [Ext] skim #
    - [Ext] sync #
    - [Ext] initialize #

 + [Int] IUniswapV2Router01 
    - [Ext] factory
    - [Ext] WETH
    - [Ext] addLiquidity #
    - [Ext] addLiquidityETH ($)
    - [Ext] removeLiquidity #
    - [Ext] removeLiquidityETH #
    - [Ext] removeLiquidityWithPermit #
    - [Ext] removeLiquidityETHWithPermit #
    - [Ext] swapExactTokensForTokens #
    - [Ext] swapTokensForExactTokens #
    - [Ext] swapExactETHForTokens ($)
    - [Ext] swapTokensForExactETH #
    - [Ext] swapExactTokensForETH #
    - [Ext] swapETHForExactTokens ($)
    - [Ext] quote
    - [Ext] getAmountOut
    - [Ext] getAmountIn
    - [Ext] getAmountsOut
    - [Ext] getAmountsIn

 + [Int] IUniswapV2Router02 (IUniswapV2Router01)
    - [Ext] removeLiquidityETHSupportingFeeOnTransferTokens #
    - [Ext] removeLiquidityETHWithPermitSupportingFeeOnTransferTokens #
    - [Ext] swapExactTokensForTokensSupportingFeeOnTransferTokens #
    - [Ext] swapExactETHForTokensSupportingFeeOnTransferTokens ($)
    - [Ext] swapExactTokensForETHSupportingFeeOnTransferTokens #

 +  MoonshotV2 (Context, IERC20, Ownable)
    - [Pub]  #
    - [Pub] name
    - [Pub] symbol
    - [Pub] decimals
    - [Pub] totalSupply
    - [Pub] balanceOf
    - [Ext] transfer #
    - [Ext] allowance
    - [Ext] approve #
    - [Pub] transferFrom #
    - [Ext] increaseAllowance #
    - [Ext] decreaseAllowance #
    - [Ext] isExcludedFromReward
    - [Ext] totalFees
    - [Ext] reflectionFromToken
    - [Pub] tokenFromReflection
    - [Ext] isFeeForAddressEnabled
    - [Ext] getFeeForAddress
    - [Ext] setPancakeRouterAddress #
       - modifiers: onlyOwner
    - [Ext] setPancakePairAddress #
       - modifiers: onlyOwner
    - [Ext] setMoonshotFundAddress #
       - modifiers: onlyOwner
    - [Ext] setFees #
       - modifiers: onlyOwner
    - [Ext] setFee #
       - modifiers: onlyOwner
    - [Ext] setBuyBackConfiguration #
       - modifiers: onlyOwner
    - [Ext] setBuyBackAddress #
       - modifiers: onlyOwner
    - [Ext] excludeFromReward #
       - modifiers: onlyOwner
    - [Ext] includeInReward #
       - modifiers: onlyOwner
    - [Ext] setSwapAndLiquifyEnabled #
       - modifiers: onlyOwner
    - [Ext] addToBlackList #
       - modifiers: onlyOwner
    - [Ext] removeFromBlackList #
       - modifiers: onlyOwner
    - [Ext] isBlackListed
    - [Ext] setSwapAndLiquifyMaxAmountEnabled #
       - modifiers: onlyOwner
    - [Ext] setSwapAndLiquifyMaxAmount #
       - modifiers: onlyOwner
    - [Ext] rescueBNB #
       - modifiers: onlyOwner
    - [Ext] removeLiquidity #
       - modifiers: onlyOwner,lockTheSwap
    - [Prv] _reflectFee #
    - [Prv] _getValues
    - [Prv] _getTValues
    - [Prv] _getRValues
    - [Prv] _getRate
    - [Prv] _getCurrentSupply
    - [Prv] _takeLiquidity #
    - [Prv] calculateTaxFee
    - [Prv] calculateLiquidityFee
    - [Prv] saveAllFees #
    - [Prv] setSpecialFee #
    - [Prv] restoreAllFees #
    - [Prv] _approve #
    - [Prv] _transfer #
    - [Prv] swapAndLiquify #
       - modifiers: lockTheSwap
    - [Prv] swapTokensForBNB #
    - [Prv] swapForBuyback #
    - [Prv] transferToAddressBNB #
    - [Prv] addLiquidity #
    - [Prv] _tokenTransfer #
    - [Prv] _transferStandard #
    - [Prv] _transferToExcluded #
    - [Prv] _transferFromExcluded #
    - [Prv] _transferBothExcluded #
    - [Ext]  ($)

    BuyMoonshotV2 Contract

    Smart Contract Audit - Inheritance

    Smart Contract Audit - Graph

    
 ($) = payable function
 # = non-constant function
 
 Int = Internal
 Ext = External
 Pub = Public
 
 +  Context 
    - [Int] _msgSender
    - [Int] _msgData

 + [Int] IERC20 
    - [Ext] totalSupply
    - [Ext] balanceOf
    - [Ext] transfer #
    - [Ext] allowance
    - [Ext] approve #
    - [Ext] transferFrom #

 + [Lib] SafeMath 
    - [Int] add
    - [Int] sub
    - [Int] sub
    - [Int] mul
    - [Int] div
    - [Int] div
    - [Int] mod
    - [Int] mod

 +  Ownable (Context)
    - [Int]  #
    - [Pub] owner
    - [Pub] renounceOwnership #
       - modifiers: onlyOwner
    - [Pub] transferOwnership #
       - modifiers: onlyOwner

 + [Int] IUniswapV2Factory 
    - [Ext] feeTo
    - [Ext] feeToSetter
    - [Ext] getPair
    - [Ext] allPairs
    - [Ext] allPairsLength
    - [Ext] createPair #
    - [Ext] setFeeTo #
    - [Ext] setFeeToSetter #

 + [Int] IUniswapV2Pair 
    - [Ext] name
    - [Ext] symbol
    - [Ext] decimals
    - [Ext] totalSupply
    - [Ext] balanceOf
    - [Ext] allowance
    - [Ext] approve #
    - [Ext] transfer #
    - [Ext] transferFrom #
    - [Ext] DOMAIN_SEPARATOR
    - [Ext] PERMIT_TYPEHASH
    - [Ext] nonces
    - [Ext] permit #
    - [Ext] MINIMUM_LIQUIDITY
    - [Ext] factory
    - [Ext] token0
    - [Ext] token1
    - [Ext] getReserves
    - [Ext] price0CumulativeLast
    - [Ext] price1CumulativeLast
    - [Ext] kLast
    - [Ext] mint #
    - [Ext] burn #
    - [Ext] swap #
    - [Ext] skim #
    - [Ext] sync #
    - [Ext] initialize #

 + [Int] IUniswapV2Router01 
    - [Ext] factory
    - [Ext] WETH
    - [Ext] addLiquidity #
    - [Ext] addLiquidityETH ($)
    - [Ext] removeLiquidity #
    - [Ext] removeLiquidityETH #
    - [Ext] removeLiquidityWithPermit #
    - [Ext] removeLiquidityETHWithPermit #
    - [Ext] swapExactTokensForTokens #
    - [Ext] swapTokensForExactTokens #
    - [Ext] swapExactETHForTokens ($)
    - [Ext] swapTokensForExactETH #
    - [Ext] swapExactTokensForETH #
    - [Ext] swapETHForExactTokens ($)
    - [Ext] quote
    - [Ext] getAmountOut
    - [Ext] getAmountIn
    - [Ext] getAmountsOut
    - [Ext] getAmountsIn

 + [Int] IUniswapV2Router02 (IUniswapV2Router01)
    - [Ext] removeLiquidityETHSupportingFeeOnTransferTokens #
    - [Ext] removeLiquidityETHWithPermitSupportingFeeOnTransferTokens #
    - [Ext] swapExactTokensForTokensSupportingFeeOnTransferTokens #
    - [Ext] swapExactETHForTokensSupportingFeeOnTransferTokens ($)
    - [Ext] swapExactTokensForETHSupportingFeeOnTransferTokens #

 +  BuyMoonshotV2 (Context, Ownable)
    - [Pub]  ($)
    - [Ext] setTokenAddress #
       - modifiers: onlyOwner
    - [Ext] setRouterAddress #
       - modifiers: onlyOwner
    - [Pub] buyTokenWithBNB ($)
    - [Ext] setFeeEnabled #
       - modifiers: onlyOwner
    - [Ext] setFee #
       - modifiers: onlyOwner
    - [Ext] withdraw #
       - modifiers: onlyOwner
    - [Ext] withdrawBNB #
       - modifiers: onlyOwner
    - [Ext]  ($)

    ClaimMoonshotV2 Contract

    Smart Contract Audit - Inheritance

    Smart Contract Audit - Graph

    
 ($) = payable function
 # = non-constant function
 
 Int = Internal
 Ext = External
 Pub = Public
 
 + [Int] IERC20 
    - [Ext] totalSupply
    - [Ext] balanceOf
    - [Ext] transfer #
    - [Ext] allowance
    - [Ext] approve #
    - [Ext] transferFrom #

 +  Context 
    - [Int] _msgSender
    - [Int] _msgData

 +  Ownable (Context)
    - [Int]  #
    - [Pub] owner
    - [Pub] renounceOwnership #
       - modifiers: onlyOwner
    - [Pub] transferOwnership #
       - modifiers: onlyOwner

 +  ClaimMoonshotV2 (Context, Ownable)
    - [Pub]  #
    - [Ext] setFromTokenAddress #
       - modifiers: onlyOwner
    - [Ext] setToTokenAddress #
       - modifiers: onlyOwner
    - [Ext] hasClaimed
    - [Ext] getClaimAmount
    - [Ext] claim #
    - [Ext] claimByOwner #
       - modifiers: onlyOwner
    - [Ext] addToBlackList #
       - modifiers: onlyOwner
    - [Ext] removeFromBlackList #
       - modifiers: onlyOwner
    - [Ext] isBlackListed
    - [Ext] burn #
       - modifiers: onlyOwner
    - [Ext] rescueBNB #
       - modifiers: onlyOwner
    - [Ext]  ($)

    About SourceHat

    SourceHat has quickly grown to have one of the most experienced and well-equipped smart contract auditing teams in the industry. Our team has conducted 1800+ solidity smart contract audits covering all major project types and protocols, securing a total of over $50 billion U.S. dollars in on-chain value!
    Our firm is well-reputed in the community and is trusted as a top smart contract auditing company for the review of solidity code, no matter how complex. Our team of experienced solidity smart contract auditors performs audits for tokens, NFTs, crowdsales, marketplaces, gambling games, financial protocols, and more!

    Contact us today to get a free quote for a smart contract audit of your project!

    What is a SourceHat Audit?

    Typically, a smart contract audit is a comprehensive review process designed to discover logical errors, security vulnerabilities, and optimization opportunities within code. A SourceHat Audit takes this a step further by verifying economic logic to ensure the stability of smart contracts and and highlighting privileged functionality to create a report that is easy to understand for developers and community members alike.

    How Do I Interpret the Findings?

    Each of our Findings will be labeled with a Severity level. We always recommend the team resolve High, Medium, and Low severity findings prior to deploying the code to the mainnet. Here is a breakdown on what each Severity level means for the project:

    • High severity indicates that the issue puts a large number of users' funds at risk and has a high probability of exploitation, or the smart contract contains serious logical issues which can prevent the code from operating as intended.
    • Medium severity issues are those which place at least some users' funds at risk and has a medium to high probability of exploitation.
    • Low severity issues have a relatively minor risk association; these issues have a low probability of occuring or may have a minimal impact.
    • Informational issues pose no immediate risk, but inform the project team of opportunities for gas optimizations and following smart contract security best practices.