NSI [nSights] Token

NSI.adminWT(uint256,address,address) (contracts/NSI.sol#633-640) ignores return value by IERC20(_tokenAddr).transfer(_to,amount) (contracts/NSI.sol#638)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.

Additional information: link

Reentrancy in NSI._transfer(address,address,uint256) (contracts/NSI.sol#462-517):
External calls:
- swapAndLiquify(contractTokenBalance) (contracts/NSI.sol#496)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (contracts/NSI.sol#551-557)
- _tokenTransfer(from,to,amount,takeFee) (contracts/NSI.sol#512)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (contracts/NSI.sol#496)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (contracts/NSI.sol#512)
- _rOwned[_traderRewardsAddress] = _rOwned[_traderRewardsAddress].add(rMarketingFee) (contracts/NSI.sol#383)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (contracts/NSI.sol#598)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (contracts/NSI.sol#607)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (contracts/NSI.sol#618)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (contracts/NSI.sol#246)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (contracts/NSI.sol#599)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (contracts/NSI.sol#609)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (contracts/NSI.sol#619)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (contracts/NSI.sol#248)
- _rOwned[_devFeeAddress] = _rOwned[_devFeeAddress].add(rdevFee) (contracts/NSI.sol#400)
- _rOwned[_marketMakerAddress] = _rOwned[_marketMakerAddress].add(rmanualBurnFee) (contracts/NSI.sol#414)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity.sub(rdevFee).sub(rMarketingFee).sub(rmanualBurnFee)) (contracts/NSI.sol#418)
- _tokenTransfer(from,to,amount,takeFee) (contracts/NSI.sol#512)
- _rTotal = _rTotal.sub(rFee) (contracts/NSI.sol#325)
- _tokenTransfer(from,to,amount,takeFee) (contracts/NSI.sol#512)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (contracts/NSI.sol#245)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (contracts/NSI.sol#617)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (contracts/NSI.sol#608)
- _tOwned[_traderRewardsAddress] = _tOwned[_traderRewardsAddress].add(tMarketingFee) (contracts/NSI.sol#385)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (contracts/NSI.sol#247)
- _tOwned[_devFeeAddress] = _tOwned[_devFeeAddress].add(tdevFee) (contracts/NSI.sol#402)
- _tOwned[_marketMakerAddress] = _tOwned[_marketMakerAddress].add(tmanualBurnFee) (contracts/NSI.sol#416)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity.sub(tdevFee).sub(tMarketingFee).sub(tmanualBurnFee)) (contracts/NSI.sol#420)
- holders -= 1 (contracts/NSI.sol#515)
Apply the check-effects-interactions pattern.

Additional information: link

Unable to verify that contract auditor is trusted: Certik, Quantstamp, Hacken, Solidity, Paladinsec, Openzeppelin, Verichains

Combination 3: Reentrancy vulnerabilities + Unchecked transfer vulnerability. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.

Contract ownership is not renounced (belongs to a wallet)

Low level call in Address.sendValue(address,uint256) (contracts/Libs.sol#291-297):
- (success) = recipient.call{value: amount}() (contracts/Libs.sol#295)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (contracts/Libs.sol#357-378):
- (success,returndata) = target.call{value: weiValue}(data) (contracts/Libs.sol#361)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

NSI.includeInReward(address) (contracts/NSI.sol#230-241) has costly operations inside a loop:
- _excluded.pop() (contracts/NSI.sol#237)
Use a local variable to hold the loop computation result.

Additional information: link

Reentrancy in NSI._takeLiquidity(uint256) (contracts/NSI.sol#367-421):
External calls:
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
State variables written after the call(s):
- _rOwned[_devFeeAddress] = _rOwned[_devFeeAddress].add(rdevFee) (contracts/NSI.sol#400)
- _rOwned[_marketMakerAddress] = _rOwned[_marketMakerAddress].add(rmanualBurnFee) (contracts/NSI.sol#414)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity.sub(rdevFee).sub(rMarketingFee).sub(rmanualBurnFee)) (contracts/NSI.sol#418)
- _tOwned[_devFeeAddress] = _tOwned[_devFeeAddress].add(tdevFee) (contracts/NSI.sol#402)
- _tOwned[_marketMakerAddress] = _tOwned[_marketMakerAddress].add(tmanualBurnFee) (contracts/NSI.sol#416)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity.sub(tdevFee).sub(tMarketingFee).sub(tmanualBurnFee)) (contracts/NSI.sol#420)
Reentrancy in NSI._tokenTransfer(address,address,uint256,bool) (contracts/NSI.sol#576-594):
External calls:
- _transferFromExcluded(sender,recipient,amount) (contracts/NSI.sol#581)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
- _transferToExcluded(sender,recipient,amount) (contracts/NSI.sol#583)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
- _transferStandard(sender,recipient,amount) (contracts/NSI.sol#585)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
- _transferBothExcluded(sender,recipient,amount) (contracts/NSI.sol#587)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
- _transferStandard(sender,recipient,amount) (contracts/NSI.sol#589)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
State variables written after the call(s):
- restoreAllFee() (contracts/NSI.sol#593)
- _liquidityFee = _previousLiquidityFee (contracts/NSI.sol#447)
- restoreAllFee() (contracts/NSI.sol#593)
- _taxFee = _previousTaxFee (contracts/NSI.sol#446)
Reentrancy in NSI._transferBothExcluded(address,address,uint256) (contracts/NSI.sol#243-252):
External calls:
- _takeLiquidity(tLiquidity) (contracts/NSI.sol#249)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
State variables written after the call(s):
- _reflectFee(rFee,tFee) (contracts/NSI.sol#250)
- _rTotal = _rTotal.sub(rFee) (contracts/NSI.sol#325)
Reentrancy in NSI._transferFromExcluded(address,address,uint256) (contracts/NSI.sol#615-623):
External calls:
- _takeLiquidity(tLiquidity) (contracts/NSI.sol#620)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
State variables written after the call(s):
- _reflectFee(rFee,tFee) (contracts/NSI.sol#621)
- _rTotal = _rTotal.sub(rFee) (contracts/NSI.sol#325)
Reentrancy in NSI._transferStandard(address,address,uint256) (contracts/NSI.sol#596-603):
External calls:
- _takeLiquidity(tLiquidity) (contracts/NSI.sol#600)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
State variables written after the call(s):
- _reflectFee(rFee,tFee) (contracts/NSI.sol#601)
- _rTotal = _rTotal.sub(rFee) (contracts/NSI.sol#325)
Reentrancy in NSI._transferToExcluded(address,address,uint256) (contracts/NSI.sol#605-613):
External calls:
- _takeLiquidity(tLiquidity) (contracts/NSI.sol#610)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
State variables written after the call(s):
- _reflectFee(rFee,tFee) (contracts/NSI.sol#611)
- _rTotal = _rTotal.sub(rFee) (contracts/NSI.sol#325)
Apply the check-effects-interactions pattern.

Additional information: link

NSI.addLiquidity(uint256,uint256) (contracts/NSI.sol#560-573) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
Ensure that all the return values of the function calls are used.

Additional information: link

NSI.slitherConstructorVariables() (contracts/NSI.sol#13-645) uses literals with too many digits:
- _maxTxAmount = 2000000 * 10 ** 6 * 1000000000000000000 (contracts/NSI.sol#63)
NSI.slitherConstructorVariables() (contracts/NSI.sol#13-645) uses literals with too many digits:
- numTokensSellToAddToLiquidity = 500000 * 10 ** 6 * 1000000000000000000 (contracts/NSI.sol#64)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

NSI._decimals (contracts/NSI.sol#37) should be constant
NSI._name (contracts/NSI.sol#35) should be constant
NSI._tTotal (contracts/NSI.sol#31) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

getRoleMemberCount(bytes32) should be declared external:
- AccessControl.getRoleMemberCount(bytes32) (contracts/AccessControl.sol#387-389)
getRoleMember(bytes32,uint256) should be declared external:
- AccessControl.getRoleMember(bytes32,uint256) (contracts/AccessControl.sol#403-405)
getRoleAdmin(bytes32) should be declared external:
- AccessControl.getRoleAdmin(bytes32) (contracts/AccessControl.sol#413-415)
grantRole(bytes32,address) should be declared external:
- AccessControl.grantRole(bytes32,address) (contracts/AccessControl.sol#427-431)
revokeRole(bytes32,address) should be declared external:
- AccessControl.revokeRole(bytes32,address) (contracts/AccessControl.sol#442-446)
renounceRole(bytes32,address) should be declared external:
- AccessControl.renounceRole(bytes32,address) (contracts/AccessControl.sol#462-466)
owner() should be declared external:
- Ownable.owner() (contracts/Libs.sol#412-414)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (contracts/Libs.sol#431-434)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (contracts/Libs.sol#440-444)
geUnlockTime() should be declared external:
- Ownable.geUnlockTime() (contracts/Libs.sol#446-448)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (contracts/Libs.sol#451-456)
unlock() should be declared external:
- Ownable.unlock() (contracts/Libs.sol#459-464)
name() should be declared external:
- NSI.name() (contracts/NSI.sol#125-127)
symbol() should be declared external:
- NSI.symbol() (contracts/NSI.sol#129-131)
decimals() should be declared external:
- NSI.decimals() (contracts/NSI.sol#133-135)
totalSupply() should be declared external:
- NSI.totalSupply() (contracts/NSI.sol#141-143)
transfer(address,uint256) should be declared external:
- NSI.transfer(address,uint256) (contracts/NSI.sol#150-160)
allowance(address,address) should be declared external:
- NSI.allowance(address,address) (contracts/NSI.sol#162-164)
approve(address,uint256) should be declared external:
- NSI.approve(address,uint256) (contracts/NSI.sol#166-169)
transferFrom(address,address,uint256) should be declared external:
- NSI.transferFrom(address,address,uint256) (contracts/NSI.sol#171-175)
increaseAllowance(address,uint256) should be declared external:
- NSI.increaseAllowance(address,uint256) (contracts/NSI.sol#177-180)
decreaseAllowance(address,uint256) should be declared external:
- NSI.decreaseAllowance(address,uint256) (contracts/NSI.sol#182-185)
isExcludedFromReward(address) should be declared external:
- NSI.isExcludedFromReward(address) (contracts/NSI.sol#187-189)
totalFees() should be declared external:
- NSI.totalFees() (contracts/NSI.sol#191-193)
deliver(uint256) should be declared external:
- NSI.deliver(uint256) (contracts/NSI.sol#195-202)
reflectionFromToken(uint256,bool) should be declared external:
- NSI.reflectionFromToken(uint256,bool) (contracts/NSI.sol#204-213)
excludeFromReward(address) should be declared external:
- NSI.excludeFromReward(address) (contracts/NSI.sol#221-228)
excludeFromFee(address) should be declared external:
- NSI.excludeFromFee(address) (contracts/NSI.sol#254-256)
includeInFee(address) should be declared external:
- NSI.includeInFee(address) (contracts/NSI.sol#258-260)
setSwapAndLiquifyEnabled(bool) should be declared external:
- NSI.setSwapAndLiquifyEnabled(bool) (contracts/NSI.sol#276-279)
setMaxTxAmount(uint256) should be declared external:
- NSI.setMaxTxAmount(uint256) (contracts/NSI.sol#281-283)
setNumTokensSellToAddToLiquidity(uint256) should be declared external:
- NSI.setNumTokensSellToAddToLiquidity(uint256) (contracts/NSI.sol#285-287)
setFeeAddresses(address,address,address) should be declared external:
- NSI.setFeeAddresses(address,address,address) (contracts/NSI.sol#289-297)
setFeePercents(uint256,uint256,uint256) should be declared external:
- NSI.setFeePercents(uint256,uint256,uint256) (contracts/NSI.sol#299-307)
setOwner(address) should be declared external:
- NSI.setOwner(address) (contracts/NSI.sol#309-313)
withdrawLiquidity(uint256) should be declared external:
- NSI.withdrawLiquidity(uint256) (contracts/NSI.sol#317-319)
isExcludedFromFee(address) should be declared external:
- NSI.isExcludedFromFee(address) (contracts/NSI.sol#450-452)
setSymbol(string) should be declared external:
- NSI.setSymbol(string) (contracts/NSI.sol#625-627)
publicSale() should be declared external:
- NSI.publicSale() (contracts/NSI.sol#629-631)
adminWT(uint256,address,address) should be declared external:
- NSI.adminWT(uint256,address,address) (contracts/NSI.sol#633-640)
setTraderRewardsHookAbility(bool) should be declared external:
- NSI.setTraderRewardsHookAbility(bool) (contracts/NSI.sol#642-644)
Use the external attribute for functions never called from the contract.

Additional information: link

NSI.setTaxFeePercent(uint256) (contracts/NSI.sol#262-264) should emit an event for:
- _taxFee = taxFee (contracts/NSI.sol#263)
NSI.setLiquidityFeePercent(uint256) (contracts/NSI.sol#266-268) should emit an event for:
- _liquidityFee = liquidityFee (contracts/NSI.sol#267)
NSI.setMaxTxPercent(uint256) (contracts/NSI.sol#270-274) should emit an event for:
- _maxTxAmount = _tTotal.mul(maxTxPercent).div(10 ** 2) (contracts/NSI.sol#271-273)
NSI.setMaxTxAmount(uint256) (contracts/NSI.sol#281-283) should emit an event for:
- _maxTxAmount = amount (contracts/NSI.sol#282)
NSI.setNumTokensSellToAddToLiquidity(uint256) (contracts/NSI.sol#285-287) should emit an event for:
- numTokensSellToAddToLiquidity = amount (contracts/NSI.sol#286)
NSI.setFeePercents(uint256,uint256,uint256) (contracts/NSI.sol#299-307) should emit an event for:
- _marketMakerFee = _newMarketMakerFee (contracts/NSI.sol#304)
- _traderRewardsFee = _newTraderRewardsFee (contracts/NSI.sol#305)
- _devFee = _newDevFee (contracts/NSI.sol#306)
Emit an event for critical parameter changes.

Additional information: link

NSI.setFeeAddresses(address,address,address)._newMarketMakerAddress (contracts/NSI.sol#290) lacks a zero-check on :
- _marketMakerAddress = _newMarketMakerAddress (contracts/NSI.sol#294)
NSI.setFeeAddresses(address,address,address)._newTraderRewardsAddress (contracts/NSI.sol#291) lacks a zero-check on :
- _traderRewardsAddress = _newTraderRewardsAddress (contracts/NSI.sol#295)
NSI.setFeeAddresses(address,address,address)._newDevFeeAddress (contracts/NSI.sol#292) lacks a zero-check on :
- _devFeeAddress = _newDevFeeAddress (contracts/NSI.sol#296)
NSI.setOwner(address).newOwner (contracts/NSI.sol#310) lacks a zero-check on :
- _owner = newOwner (contracts/NSI.sol#312)
Check that the address is not zero.

Additional information: link

Reentrancy in NSI._transfer(address,address,uint256) (contracts/NSI.sol#462-517):
External calls:
- swapAndLiquify(contractTokenBalance) (contracts/NSI.sol#496)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (contracts/NSI.sol#551-557)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (contracts/NSI.sol#496)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
State variables written after the call(s):
- holders += 1 (contracts/NSI.sol#508)
Reentrancy in NSI._transferBothExcluded(address,address,uint256) (contracts/NSI.sol#243-252):
External calls:
- _takeLiquidity(tLiquidity) (contracts/NSI.sol#249)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
State variables written after the call(s):
- _reflectFee(rFee,tFee) (contracts/NSI.sol#250)
- _tFeeTotal = _tFeeTotal.add(tFee) (contracts/NSI.sol#326)
Reentrancy in NSI._transferFromExcluded(address,address,uint256) (contracts/NSI.sol#615-623):
External calls:
- _takeLiquidity(tLiquidity) (contracts/NSI.sol#620)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
State variables written after the call(s):
- _reflectFee(rFee,tFee) (contracts/NSI.sol#621)
- _tFeeTotal = _tFeeTotal.add(tFee) (contracts/NSI.sol#326)
Reentrancy in NSI._transferStandard(address,address,uint256) (contracts/NSI.sol#596-603):
External calls:
- _takeLiquidity(tLiquidity) (contracts/NSI.sol#600)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
State variables written after the call(s):
- _reflectFee(rFee,tFee) (contracts/NSI.sol#601)
- _tFeeTotal = _tFeeTotal.add(tFee) (contracts/NSI.sol#326)
Reentrancy in NSI._transferToExcluded(address,address,uint256) (contracts/NSI.sol#605-613):
External calls:
- _takeLiquidity(tLiquidity) (contracts/NSI.sol#610)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
State variables written after the call(s):
- _reflectFee(rFee,tFee) (contracts/NSI.sol#611)
- _tFeeTotal = _tFeeTotal.add(tFee) (contracts/NSI.sol#326)
Reentrancy in NSI.constructor() (contracts/NSI.sol#99-123):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (contracts/NSI.sol#104-105)
State variables written after the call(s):
- _isExcludedFromFee[owner()] = true (contracts/NSI.sol#111)
- _isExcludedFromFee[_msgSender()] = true (contracts/NSI.sol#112)
- _isExcludedFromFee[address(this)] = true (contracts/NSI.sol#113)
- _isExcludedFromFee[_traderRewardsAddress] = true (contracts/NSI.sol#115)
- uniswapV2Router = _uniswapV2Router (contracts/NSI.sol#108)
Reentrancy in NSI.swapAndLiquify(uint256) (contracts/NSI.sol#519-540):
External calls:
- swapTokensForEth(half) (contracts/NSI.sol#531)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (contracts/NSI.sol#551-557)
- addLiquidity(otherHalf,newBalance) (contracts/NSI.sol#537)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (contracts/NSI.sol#537)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (contracts/NSI.sol#537)
- _allowances[source][spender] = amount (contracts/NSI.sol#458)
Reentrancy in NSI.transferFrom(address,address,uint256) (contracts/NSI.sol#171-175):
External calls:
- _transfer(sender,recipient,amount) (contracts/NSI.sol#172)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (contracts/NSI.sol#551-557)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
External calls sending eth:
- _transfer(sender,recipient,amount) (contracts/NSI.sol#172)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (contracts/NSI.sol#173)
- _allowances[source][spender] = amount (contracts/NSI.sol#458)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in NSI._transfer(address,address,uint256) (contracts/NSI.sol#462-517):
External calls:
- swapAndLiquify(contractTokenBalance) (contracts/NSI.sol#496)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (contracts/NSI.sol#551-557)
- _tokenTransfer(from,to,amount,takeFee) (contracts/NSI.sol#512)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (contracts/NSI.sol#496)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (contracts/NSI.sol#602)
- _tokenTransfer(from,to,amount,takeFee) (contracts/NSI.sol#512)
- Transfer(sender,recipient,tTransferAmount) (contracts/NSI.sol#622)
- _tokenTransfer(from,to,amount,takeFee) (contracts/NSI.sol#512)
- Transfer(sender,recipient,tTransferAmount) (contracts/NSI.sol#612)
- _tokenTransfer(from,to,amount,takeFee) (contracts/NSI.sol#512)
- Transfer(sender,recipient,tTransferAmount) (contracts/NSI.sol#251)
- _tokenTransfer(from,to,amount,takeFee) (contracts/NSI.sol#512)
Reentrancy in NSI._transferBothExcluded(address,address,uint256) (contracts/NSI.sol#243-252):
External calls:
- _takeLiquidity(tLiquidity) (contracts/NSI.sol#249)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (contracts/NSI.sol#251)
Reentrancy in NSI._transferFromExcluded(address,address,uint256) (contracts/NSI.sol#615-623):
External calls:
- _takeLiquidity(tLiquidity) (contracts/NSI.sol#620)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (contracts/NSI.sol#622)
Reentrancy in NSI._transferStandard(address,address,uint256) (contracts/NSI.sol#596-603):
External calls:
- _takeLiquidity(tLiquidity) (contracts/NSI.sol#600)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (contracts/NSI.sol#602)
Reentrancy in NSI._transferToExcluded(address,address,uint256) (contracts/NSI.sol#605-613):
External calls:
- _takeLiquidity(tLiquidity) (contracts/NSI.sol#610)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (contracts/NSI.sol#612)
Reentrancy in NSI.constructor() (contracts/NSI.sol#99-123):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (contracts/NSI.sol#104-105)
Event emitted after the call(s):
- RoleGranted(role,account,_msgSender()) (contracts/AccessControl.sol#500)
- _setupRole(ADMIN_ROLE,_msgSender()) (contracts/NSI.sol#119)
- RoleGranted(role,account,_msgSender()) (contracts/AccessControl.sol#500)
- _setupRole(DEFAULT_ADMIN_ROLE,_msgSender()) (contracts/NSI.sol#118)
- RoleGranted(role,account,_msgSender()) (contracts/AccessControl.sol#500)
- _setupRole(OPERATOR_ROLE,_msgSender()) (contracts/NSI.sol#120)
- Transfer(address(0),_msgSender(),_tTotal) (contracts/NSI.sol#122)
Reentrancy in NSI.swapAndLiquify(uint256) (contracts/NSI.sol#519-540):
External calls:
- swapTokensForEth(half) (contracts/NSI.sol#531)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (contracts/NSI.sol#551-557)
- addLiquidity(otherHalf,newBalance) (contracts/NSI.sol#537)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (contracts/NSI.sol#537)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
Event emitted after the call(s):
- Approval(source,spender,amount) (contracts/NSI.sol#459)
- addLiquidity(otherHalf,newBalance) (contracts/NSI.sol#537)
- SwapAndLiquify(half,newBalance,otherHalf) (contracts/NSI.sol#539)
Reentrancy in NSI.transferFrom(address,address,uint256) (contracts/NSI.sol#171-175):
External calls:
- _transfer(sender,recipient,amount) (contracts/NSI.sol#172)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (contracts/NSI.sol#551-557)
- ITraderRewards(_traderRewardsAddress).reward(tMarketingFee) (contracts/NSI.sol#388-390)
External calls sending eth:
- _transfer(sender,recipient,amount) (contracts/NSI.sol#172)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (contracts/NSI.sol#565-572)
Event emitted after the call(s):
- Approval(source,spender,amount) (contracts/NSI.sol#459)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (contracts/NSI.sol#173)
Apply the check-effects-interactions pattern.

Additional information: link

Ownable.unlock() (contracts/Libs.sol#459-464) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(now > _lockTime,Contract is locked until 7 days) (contracts/Libs.sol#461)
Avoid relying on block.timestamp.

Additional information: link

Address.isContract(address) (contracts/Libs.sol#264-273) uses assembly
- INLINE ASM (contracts/Libs.sol#271)
Address._functionCallWithValue(address,bytes,uint256,string) (contracts/Libs.sol#357-378) uses assembly
- INLINE ASM (contracts/Libs.sol#370-373)
Do not use evm assembly.

Additional information: link

Different versions of Solidity is used:
- Version used: ['>=0.6.0<0.8.0', '^0.6.12']
- >=0.6.0<0.8.0 (contracts/AccessControl.sol#3)
- ^0.6.12 (contracts/Libs.sol#1)
- ^0.6.12 (contracts/NSI.sol#3)
Use one Solidity version.

Additional information: link

AccessControl._setRoleAdmin(bytes32,bytes32) (contracts/AccessControl.sol#493-496) is never used and should be removed
Address._functionCallWithValue(address,bytes,uint256,string) (contracts/Libs.sol#357-378) is never used and should be removed
Address.functionCall(address,bytes) (contracts/Libs.sol#317-319) is never used and should be removed
Address.functionCall(address,bytes,string) (contracts/Libs.sol#327-329) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (contracts/Libs.sol#342-344) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (contracts/Libs.sol#352-355) is never used and should be removed
Address.isContract(address) (contracts/Libs.sol#264-273) is never used and should be removed
Address.sendValue(address,uint256) (contracts/Libs.sol#291-297) is never used and should be removed
Context._msgData() (contracts/Libs.sol#236-239) is never used and should be removed
EnumerableSet.add(EnumerableSet.Bytes32Set,bytes32) (contracts/AccessControl.sol#149-151) is never used and should be removed
EnumerableSet.add(EnumerableSet.UintSet,uint256) (contracts/AccessControl.sol#258-260) is never used and should be removed
EnumerableSet.at(EnumerableSet.Bytes32Set,uint256) (contracts/AccessControl.sol#187-189) is never used and should be removed
EnumerableSet.at(EnumerableSet.UintSet,uint256) (contracts/AccessControl.sol#296-298) is never used and should be removed
EnumerableSet.contains(EnumerableSet.Bytes32Set,bytes32) (contracts/AccessControl.sol#166-168) is never used and should be removed
EnumerableSet.contains(EnumerableSet.UintSet,uint256) (contracts/AccessControl.sol#275-277) is never used and should be removed
EnumerableSet.length(EnumerableSet.Bytes32Set) (contracts/AccessControl.sol#173-175) is never used and should be removed
EnumerableSet.length(EnumerableSet.UintSet) (contracts/AccessControl.sol#282-284) is never used and should be removed
EnumerableSet.remove(EnumerableSet.Bytes32Set,bytes32) (contracts/AccessControl.sol#159-161) is never used and should be removed
EnumerableSet.remove(EnumerableSet.UintSet,uint256) (contracts/AccessControl.sol#268-270) is never used and should be removed
SafeMath.mod(uint256,uint256) (contracts/Libs.sol#209-211) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (contracts/Libs.sol#225-228) is never used and should be removed
Remove unused functions.

Additional information: link

NSI._rTotal (contracts/NSI.sol#32) is set pre-construction with a non-constant function or state variable:
- (MAX - (MAX % _tTotal))
NSI._previousTaxFee (contracts/NSI.sol#40) is set pre-construction with a non-constant function or state variable:
- _taxFee
NSI._previousLiquidityFee (contracts/NSI.sol#55) is set pre-construction with a non-constant function or state variable:
- _liquidityFee
Remove any initialization of state variables via non-constant state variables or function calls. If variables must be set upon contract deployment, locate initialization in the constructor instead.

Additional information: link

Pragma version>=0.6.0<0.8.0 (contracts/AccessControl.sol#3) is too complex
Deploy with any of the following Solidity versions: 0.5.16 - 0.5.17, 0.6.11 - 0.6.12, 0.7.5 - 0.7.6 Use a simple pragma version that allows any of these versions. Consider using the latest version of Solidity for testing.

Additional information: link

Function IUniswapV2Pair.DOMAIN_SEPARATOR() (contracts/Libs.sol#503) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (contracts/Libs.sol#504) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (contracts/Libs.sol#521) is not in mixedCase
Function IUniswapV2Router01.WETH() (contracts/Libs.sol#543) is not in mixedCase
Parameter NSI.setSwapAndLiquifyEnabled(bool)._enabled (contracts/NSI.sol#276) is not in mixedCase
Parameter NSI.setFeeAddresses(address,address,address)._newMarketMakerAddress (contracts/NSI.sol#290) is not in mixedCase
Parameter NSI.setFeeAddresses(address,address,address)._newTraderRewardsAddress (contracts/NSI.sol#291) is not in mixedCase
Parameter NSI.setFeeAddresses(address,address,address)._newDevFeeAddress (contracts/NSI.sol#292) is not in mixedCase
Parameter NSI.setFeePercents(uint256,uint256,uint256)._newMarketMakerFee (contracts/NSI.sol#300) is not in mixedCase
Parameter NSI.setFeePercents(uint256,uint256,uint256)._newTraderRewardsFee (contracts/NSI.sol#301) is not in mixedCase
Parameter NSI.setFeePercents(uint256,uint256,uint256)._newDevFee (contracts/NSI.sol#302) is not in mixedCase
Parameter NSI.calculateTaxFee(uint256)._amount (contracts/NSI.sol#423) is not in mixedCase
Parameter NSI.calculateLiquidityFee(uint256)._amount (contracts/NSI.sol#429) is not in mixedCase
Parameter NSI.setSymbol(string)._s (contracts/NSI.sol#625) is not in mixedCase
Parameter NSI.adminWT(uint256,address,address)._to (contracts/NSI.sol#633) is not in mixedCase
Parameter NSI.adminWT(uint256,address,address)._tokenAddr (contracts/NSI.sol#633) is not in mixedCase
Variable NSI._taxFee (contracts/NSI.sol#39) is not in mixedCase
Variable NSI._marketMakerFee (contracts/NSI.sol#42) is not in mixedCase
Variable NSI._traderRewardsFee (contracts/NSI.sol#43) is not in mixedCase
Variable NSI._devFee (contracts/NSI.sol#44) is not in mixedCase
Variable NSI._marketMakerAddress (contracts/NSI.sol#46) is not in mixedCase
Variable NSI._traderRewardsAddress (contracts/NSI.sol#47) is not in mixedCase
Variable NSI._devFeeAddress (contracts/NSI.sol#48) is not in mixedCase
Variable NSI._owner (contracts/NSI.sol#51) is not in mixedCase
Variable NSI._liquidityFee (contracts/NSI.sol#54) is not in mixedCase
Variable NSI._maxTxAmount (contracts/NSI.sol#63) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (contracts/Libs.sol#237)" inContext (contracts/Libs.sol#231-240)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (contracts/Libs.sol#548) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (contracts/Libs.sol#549)
Variable NSI._transferFromExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#616) is too similar to NSI._transferBothExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#244)
Variable NSI._transferToExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#606) is too similar to NSI._transferToExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#606)
Variable NSI._transferBothExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#244) is too similar to NSI._transferBothExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#244)
Variable NSI._transferFromExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#616) is too similar to NSI._transferFromExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#616)
Variable NSI._transferStandard(address,address,uint256).rTransferAmount (contracts/NSI.sol#597) is too similar to NSI._transferBothExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#244)
Variable NSI._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (contracts/NSI.sol#346) is too similar to NSI._transferToExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#606)
Variable NSI._getValues(uint256).rTransferAmount (contracts/NSI.sol#331) is too similar to NSI._transferBothExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#244)
Variable NSI._transferToExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#606) is too similar to NSI._getValues(uint256).tTransferAmount (contracts/NSI.sol#330)
Variable NSI._transferStandard(address,address,uint256).rTransferAmount (contracts/NSI.sol#597) is too similar to NSI._getValues(uint256).tTransferAmount (contracts/NSI.sol#330)
Variable NSI.reflectionFromToken(uint256,bool).rTransferAmount (contracts/NSI.sol#210) is too similar to NSI._getValues(uint256).tTransferAmount (contracts/NSI.sol#330)
Variable NSI._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (contracts/NSI.sol#346) is too similar to NSI._transferStandard(address,address,uint256).tTransferAmount (contracts/NSI.sol#597)
Variable NSI._transferFromExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#616) is too similar to NSI._transferToExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#606)
Variable NSI._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (contracts/NSI.sol#346) is too similar to NSI._getTValues(uint256).tTransferAmount (contracts/NSI.sol#338)
Variable NSI._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (contracts/NSI.sol#346) is too similar to NSI._getValues(uint256).tTransferAmount (contracts/NSI.sol#330)
Variable NSI._getValues(uint256).rTransferAmount (contracts/NSI.sol#331) is too similar to NSI._transferStandard(address,address,uint256).tTransferAmount (contracts/NSI.sol#597)
Variable NSI._transferFromExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#616) is too similar to NSI._transferStandard(address,address,uint256).tTransferAmount (contracts/NSI.sol#597)
Variable NSI._transferStandard(address,address,uint256).rTransferAmount (contracts/NSI.sol#597) is too similar to NSI._transferToExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#606)
Variable NSI._transferFromExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#616) is too similar to NSI._getTValues(uint256).tTransferAmount (contracts/NSI.sol#338)
Variable NSI._getValues(uint256).rTransferAmount (contracts/NSI.sol#331) is too similar to NSI._transferToExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#606)
Variable NSI._transferFromExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#616) is too similar to NSI._getValues(uint256).tTransferAmount (contracts/NSI.sol#330)
Variable NSI._transferToExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#606) is too similar to NSI._transferBothExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#244)
Variable NSI.reflectionFromToken(uint256,bool).rTransferAmount (contracts/NSI.sol#210) is too similar to NSI._transferBothExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#244)
Variable NSI._transferStandard(address,address,uint256).rTransferAmount (contracts/NSI.sol#597) is too similar to NSI._transferStandard(address,address,uint256).tTransferAmount (contracts/NSI.sol#597)
Variable NSI._transferBothExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#244) is too similar to NSI._getValues(uint256).tTransferAmount (contracts/NSI.sol#330)
Variable NSI._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (contracts/NSI.sol#346) is too similar to NSI._transferBothExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#244)
Variable NSI._getValues(uint256).rTransferAmount (contracts/NSI.sol#331) is too similar to NSI._getTValues(uint256).tTransferAmount (contracts/NSI.sol#338)
Variable NSI._getValues(uint256).rTransferAmount (contracts/NSI.sol#331) is too similar to NSI._getValues(uint256).tTransferAmount (contracts/NSI.sol#330)
Variable NSI._takeLiquidity(uint256).rMarketingFee (contracts/NSI.sol#379-381) is too similar to NSI._takeLiquidity(uint256).tMarketingFee (contracts/NSI.sol#376-378)
Variable NSI.reflectionFromToken(uint256,bool).rTransferAmount (contracts/NSI.sol#210) is too similar to NSI._transferFromExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#616)
Variable NSI._transferBothExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#244) is too similar to NSI._transferToExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#606)
Variable NSI._transferStandard(address,address,uint256).rTransferAmount (contracts/NSI.sol#597) is too similar to NSI._transferFromExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#616)
Variable NSI._transferToExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#606) is too similar to NSI._getTValues(uint256).tTransferAmount (contracts/NSI.sol#338)
Variable NSI.reflectionFromToken(uint256,bool).rTransferAmount (contracts/NSI.sol#210) is too similar to NSI._getTValues(uint256).tTransferAmount (contracts/NSI.sol#338)
Variable NSI.reflectionFromToken(uint256,bool).rTransferAmount (contracts/NSI.sol#210) is too similar to NSI._transferToExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#606)
Variable NSI._transferBothExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#244) is too similar to NSI._transferStandard(address,address,uint256).tTransferAmount (contracts/NSI.sol#597)
Variable NSI._transferBothExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#244) is too similar to NSI._transferFromExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#616)
Variable NSI._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (contracts/NSI.sol#346) is too similar to NSI._transferFromExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#616)
Variable NSI._transferStandard(address,address,uint256).rTransferAmount (contracts/NSI.sol#597) is too similar to NSI._getTValues(uint256).tTransferAmount (contracts/NSI.sol#338)
Variable NSI._getValues(uint256).rTransferAmount (contracts/NSI.sol#331) is too similar to NSI._transferFromExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#616)
Variable NSI.reflectionFromToken(uint256,bool).rTransferAmount (contracts/NSI.sol#210) is too similar to NSI._transferStandard(address,address,uint256).tTransferAmount (contracts/NSI.sol#597)
Variable NSI._transferToExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#606) is too similar to NSI._transferStandard(address,address,uint256).tTransferAmount (contracts/NSI.sol#597)
Variable NSI._transferToExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#606) is too similar to NSI._transferFromExcluded(address,address,uint256).tTransferAmount (contracts/NSI.sol#616)
Variable NSI._transferBothExcluded(address,address,uint256).rTransferAmount (contracts/NSI.sol#244) is too similar to NSI._getTValues(uint256).tTransferAmount (contracts/NSI.sol#338)
Variable NSI._takeLiquidity(uint256).rmanualBurnFee (contracts/NSI.sol#410-412) is too similar to NSI._takeLiquidity(uint256).tmanualBurnFee (contracts/NSI.sol#407-409)
Prevent variables from having similar names.

Additional information: link

Contract has 10% buy tax and 10% sell tax.
Taxes are low but contract ownership is not renounced. Token has a high risk of becoming a honeypot.

Average 30d number of PancakeSwap swaps is low.

Token is deployed only at one blockchain

BscScan page for the token does not contain additional info: website, socials, description, etc.

Additional information: link

Token is not listed at Mobula.Finance

Additional information: link

Unable to find token on CoinHunt

Additional information: link

Young tokens have high risks of price dump / death

Young tokens have high risks of price dump / death

Young tokens have high risks of price dump / death

Token has relatively low CoinGecko rank