NEFTiPEDiA - Smart Contract Audit Report

Summary

NEFTiPEDiA Audit Report NEFTiPEDiA is a new BEP20 token that is an automatic liquidity providing protocol that pays out static rewards to holders.

For this audit, we reviewed the NEFTiPEDiA contract at commit 45fb8d07504183ba922d935caa178aca9c686728 on the team's GitHub Repository.

Notes on the Contract:
  • The total supply of the token will be set by the deployer upon initialization.
  • The owner can mint any amount of tokens at any time.
  • Any user can burn their own tokens to reduce the total supply.
  • There was no token allocation for our team to analyze as the contract has yet to be deployed to the mainnet.

  • There is a tax fee, liquidity fee, and marketing fee on all transfers where neither the sender nor the recipient is excluded from fees.
  • The tokens collected from 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 tokens collected from the liquidity fee during transfers are stored in the contract address balance. Once the threshold number of tokens (determined by the owner) is met, a swap will occur for the purpose of funding Pancakeswap liquidity.
  • Liquidity-adds are funded by selling a portion of the tokens collected as fees (after the threshold number of tokens is met), then pairing the received BNB with the token, and adding it as liquidity to the BNB pair.
  • The recipient of the newly created LP tokens is the owner. We recommend that the team locks these newly acquired LP tokens.
  • The tokens collected from the marketing fee are sent to the team's Marketing wallet.
  • The contract includes functionality to interact with an external antibot contract. This contract was out of scope for the purpose of this audit.
  • The contract is upgradable, meaning the team can swap out the current contract for a new one at any time.
  • Although the Safemath library is utilized, the contract is deployed with Solidity v0.8.9 which has built-in overflow checks. Safemath could be safely removed to reduce contract size and increase gas savings.

  • Ownership Controls:
  • Ownership has not been renounced.
  • The owner can modify the tax fee and liquidity fee to any percentages at any time. The marketing fee is set upon initialization and can only be changed if the contract is upgraded.
  • The owner can exclude and include accounts from transfer fees and reward distribution.
  • The owner can set and update a maximum transaction amount at any time, which will impose a limit to the number of tokens that can be transferred during any given transaction. The owner can also exclude accounts from this restriction.
  • The owner can call the recallTax() function to undo the frictionless fee redistribution mechanism of the contract at any time. The number of tokens that have been removed from the circulating supply through the tax fee will be minted to the team's Support address. This will reduce the token balance of all holders.
  • The owner can withdraw any BEP20 tokens from the contract address at any time.
  • The owner can enable/disable automatic liquidity adds at any time.
  • The owner can add/remove accounts from a blacklist which prevents them from being able to participate in transfers.
  • The owner can update the team's Marketing wallet, and Support wallet to any addresses at any time.
  • The owner can update the Pancakeswap Router address at any time.
  • The owner can use the "lock" function in order to temporarily set ownership to address(0). Ownership is restored after the duration of time determined by the owner has passed and they use the 'unlock' function.
  • The unlock function has the potential to be used after ownership is renounced, which will restore ownership to the original owner that initially created the ownership lock. This can be used in a nefarious way by the project team to restore ownership and change fee structures.
  • We recommend that the unlock function is modified to set the "previous owner" = "address(0)" at the end of the unlock function to prevent it from being used more than once per lock.
Audit Findings Summary
  • No external threats were identified.
  • Please ensure trust in the team prior to investing as they have substantial control in the ecosystem and can undo the frictionless fee redistribution mechanism at any time.
  • Further, ensure trust in the team as the contract is upgradeable and can be swapped out for a new contract at any time.
  • Date: December 22nd, 2021

Audit Results

Vulnerability Category Notes Result
Arbitrary Storage Write N/A PASS
Arbitrary Jump N/A PASS
Centralization of Control
  • The owner can set the tax fee and liquidity fee up to 100%.
  • The owner can mint any amount of tokens at any time.
  • The owner can undo the frictionless fee redistribution mechanism which would reduce the token balances of all holders.
  • The contract can be swapped out for another at any time.
  • WARNING
    Delegate Call to Untrusted Contract N/A PASS
    Dependence on Predictable Variables N/A PASS
    Deprecated Opcodes N/A PASS
    Ether Thief N/A PASS
    Exceptions N/A PASS
    External Calls N/A PASS
    Flash Loans N/A PASS
    Integer Over/Underflow N/A PASS
    Multiple Sends N/A PASS
    Oracles N/A PASS
    Suicide N/A PASS
    State Change External Calls N/A PASS
    Unchecked Retval N/A PASS
    User Supplied Assertion N/A PASS
    Critical Solidity Compiler N/A PASS
    Overall Contract Safety   PASS

    Function Graph

    ERC20 Token Graph


    Inheritance Chart

    Multi-file Token


    Functions Overview

    
     ($) = payable function
     # = non-constant function
     
     + [Lib] SafeMath 
        - [Int] tryAdd
        - [Int] trySub
        - [Int] tryMul
        - [Int] tryDiv
        - [Int] tryMod
        - [Int] add
        - [Int] sub
        - [Int] mul
        - [Int] div
        - [Int] mod
        - [Int] sub
        - [Int] div
        - [Int] mod
    
     + [Lib] AddressUpgradeable 
        - [Int] isContract
        - [Int] sendValue #
        - [Int] functionCall #
        - [Int] functionCall #
        - [Int] functionCallWithValue #
        - [Int] functionCallWithValue #
        - [Int] functionStaticCall
        - [Int] functionStaticCall
        - [Int] verifyCallResult
    
     + [Int] IERC20Upgradeable 
        - [Ext] totalSupply
        - [Ext] balanceOf
        - [Ext] transfer #
        - [Ext] allowance
        - [Ext] approve #
        - [Ext] transferFrom #
    
     + [Int] IERC20MetadataUpgradeable (IERC20Upgradeable)
        - [Ext] name
        - [Ext] symbol
        - [Ext] decimals
    
     + [Int] IPancakeFactory 
        - [Ext] feeTo
        - [Ext] feeToSetter
        - [Ext] getPair
        - [Ext] allPairs
        - [Ext] allPairsLength
        - [Ext] createPair #
        - [Ext] setFeeTo #
        - [Ext] setFeeToSetter #
    
     + [Int] IPancakeRouter01 
        - [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] IPancakeRouter02 (IPancakeRouter01)
        - [Ext] removeLiquidityETHSupportingFeeOnTransferTokens #
        - [Ext] removeLiquidityETHWithPermitSupportingFeeOnTransferTokens #
        - [Ext] swapExactTokensForTokensSupportingFeeOnTransferTokens #
        - [Ext] swapExactETHForTokensSupportingFeeOnTransferTokens ($)
        - [Ext] swapExactTokensForETHSupportingFeeOnTransferTokens #
    
     + [Int] IPancakePair 
        - [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] IPinkAntiBot 
        - [Ext] setTokenOwner #
        - [Ext] onPreTransferCheck #
    
     +  Initializable 
    
     +  ContextUpgradeable (Initializable)
        - [Int] __Context_init #
           - modifiers: initializer
        - [Int] __Context_init_unchained #
           - modifiers: initializer
        - [Int] _msgSender
        - [Int] _msgData
    
     +  OwnableUpgradeable (Initializable, ContextUpgradeable)
        - [Int] __Ownable_init #
           - modifiers: initializer
        - [Int] __Ownable_init_unchained #
           - modifiers: initializer
        - [Pub] authorize #
           - modifiers: onlyOwner
        - [Pub] unauthorize #
           - modifiers: onlyOwner
        - [Pub] owner
        - [Pub] isOwner
        - [Prv] _setOwner #
        - [Pub] isAuthorized
        - [Pub] transferOwnership #
           - modifiers: onlyOwner
    
     +  ReentrancyGuardUpgradeable (Initializable)
        - [Int] __ReentrancyGuard_init #
           - modifiers: initializer
        - [Int] __ReentrancyGuard_init_unchained #
           - modifiers: initializer
    
     +  NEFTiPEDiA (OwnableUpgradeable, IERC20MetadataUpgradeable, ReentrancyGuardUpgradeable)
        - [Pub] initialize #
           - modifiers: initializer
        - [Int] __NEFTiPEDiA_init #
           - modifiers: initializer
        - [Int] initVars #
           - modifiers: initializer
        - [Ext] setVersion #
           - modifiers: onlyOwner
        - [Ext] setEnableAntiBot #
           - modifiers: onlyOwner
        - [Pub] name
        - [Pub] symbol
        - [Pub] decimals
        - [Pub] totalSupply
        - [Pub] balanceOf
        - [Pub] reflectOf
        - [Pub] setSupportAccounts #
           - modifiers: onlyOwner
        - [Pub] transfer #
        - [Pub] allowance
        - [Pub] approve #
        - [Pub] transferFrom #
        - [Pub] increaseAllowance #
        - [Pub] decreaseAllowance #
        - [Pub] suspend #
           - modifiers: onlyOwner
        - [Pub] isSuspended
        - [Pub] isExcludedFromReward
        - [Pub] totalFees
        - [Pub] deliver #
        - [Pub] reflectionFromToken
        - [Pub] tokenFromReflection
        - [Pub] excludeFromReward #
           - modifiers: onlyOwner
        - [Ext] includeInReward #
           - modifiers: onlyOwner
        - [Prv] _transferBothExcluded #
        - [Pub] excludeFromFee #
           - modifiers: onlyOwner
        - [Pub] includeInFee #
           - modifiers: onlyOwner
        - [Pub] setExcludeFromMaxTx #
           - modifiers: onlyOwner
        - [Pub] isExcludedFromMaxTx
        - [Ext] setTaxFeePercent #
           - modifiers: onlyOwner
        - [Ext] setLiquidityFeePercent #
           - modifiers: onlyOwner
        - [Ext] setMaxTxPercent #
           - modifiers: onlyOwner
        - [Pub] setSwapAndLiquifyEnabled #
           - modifiers: onlyOwner
        - [Ext]  ($)
        - [Pub] mint #
           - modifiers: onlyOwner
        - [Int] _burn #
        - [Pub] burn #
        - [Pub] burnFrom #
        - [Prv] _reflectFee #
        - [Prv] _getValues
        - [Prv] _getTValues
        - [Prv] _getRValues
        - [Prv] _getRate
        - [Prv] _getCurrentSupply
        - [Prv] _takeLiquidity #
        - [Prv] _takeMarketingFee #
        - [Prv] calcTaxFee
        - [Prv] calcLiquidityFee
        - [Prv] calcMarketingFee
        - [Prv] removeAllFee #
        - [Prv] restoreAllFee #
        - [Pub] isExcludedFromFee
        - [Prv] _approve #
        - [Prv] _transfer #
        - [Prv] swapAndLiquify #
           - modifiers: lockTheSwap
        - [Prv] swapTokensForEth #
        - [Prv] addLiquidity #
        - [Pub] recallTo #
           - modifiers: onlyOwner
        - [Pub] recallTax #
           - modifiers: onlyOwner
        - [Prv] _tokenTransfer #
        - [Prv] _transferStandard #
        - [Prv] _transferToExcluded #
        - [Prv] _transferFromExcluded #