LNR [Lunar] Token

Reentrancy in Lunar._transfer(address,address,uint256) (#705-750):
External calls:
- buyBackTokens(balance.div(100)) (#741)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
- swapAndLiquify(swapTokensAtAmount) (#744)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#849-855)
External calls sending eth:
- buyBackTokens(balance.div(100)) (#741)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
- swapAndLiquify(swapTokensAtAmount) (#744)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
- address(operationsAddress).transfer(operationsAmt) (#821)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
- _rOwned[address(this)] += rLiquidity (#619)
- _rOwned[address(this)] += rBuyback (#609)
- _rOwned[marketingAddress] += rMarketing (#629)
- _rOwned[address(this)] += rOperations (#598)
- _rOwned[sender] = _rOwned[sender] - s.rAmount (#765)
- _rOwned[recipient] = _rOwned[recipient] + s.rTransferAmount (#766)
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
- _rTotal -= rRfi (#589)
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
- _tOwned[address(this)] += tOperations (#596)
- _tOwned[marketingAddress] += tMarketing (#627)
- _tOwned[sender] = _tOwned[sender] - tAmount (#759)
- _tOwned[address(this)] += tLiquidity (#617)
- _tOwned[address(this)] += tBuyback (#607)
- _tOwned[recipient] = _tOwned[recipient] + s.tTransferAmount (#762)
- swapAndLiquify(swapTokensAtAmount) (#744)
- swapping = true (#417)
- swapping = false (#419)
Apply the check-effects-interactions pattern.

Additional information: link

Lunar.swapETHForTokens(uint256) (#784-797) sends eth to arbitrary user
Dangerous calls:
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
Lunar.addLiquidity(uint256,uint256) (#825-838) sends eth to arbitrary user
Dangerous calls:
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
Ensure that an arbitrary user cannot withdraw unauthorized funds.

Additional information: link

Lunar.rescueBEP20Tokens(address) (#908-910) ignores return value by IERC20(tokenAddress).transfer(msg.sender,IERC20(tokenAddress).balanceOf(address(this))) (#909)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.

Additional information: link

Combination 1: Reentrancy vulnerabilities + Functions that send Ether to arbitraty destination. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.

Combination 2: Unchecked transfer + Functions that send Ether to arbitraty destination. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.

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.

Lunar._transfer(address,address,uint256).isSale (#746) is a local variable never initialized
Initialize all the variables. If a variable is meant to be initialized to zero, explicitly set it to zero to improve code readability.

Additional information: link

Lunar.slitherConstructorVariables() (#320-929) uses literals with too many digits:
- _tTotal = 1000000000000000 * 10 ** _decimals (#345)
Lunar.slitherConstructorVariables() (#320-929) uses literals with too many digits:
- swapTokensAtAmount = 500000000000 * 10 ** _decimals (#350)
Lunar.slitherConstructorVariables() (#320-929) uses literals with too many digits:
- _maxWalletSize = 20000000000000 * 10 ** 9 (#351)
Lunar.slitherConstructorConstantVariables() (#320-929) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#356)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

Lunar._tTotal (#345) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

Lunar.includeInReward(address) (#537-548) has costly operations inside a loop:
- _excluded.pop() (#544)
Use a local variable to hold the loop computation result.

Additional information: link

Redundant expression "this (#139)" inContext (#133-142)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Lunar.constructor(address)._pair (#424-425) lacks a zero-check on :
- pair = _pair (#428)
Check that the address is not zero.

Additional information: link

Reentrancy in Lunar._transfer(address,address,uint256) (#705-750):
External calls:
- buyBackTokens(balance.div(100)) (#741)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
- swapAndLiquify(swapTokensAtAmount) (#744)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#849-855)
External calls sending eth:
- buyBackTokens(balance.div(100)) (#741)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
- swapAndLiquify(swapTokensAtAmount) (#744)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
- address(operationsAddress).transfer(operationsAmt) (#821)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#702)
- swapAndLiquify(swapTokensAtAmount) (#744)
- Transfer(sender,recipient,s.tTransferAmount) (#772)
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
- Transfer(sender,address(this),s.tLiquidity + s.tOperations + s.tBuyback) (#773)
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
- Transfer(sender,marketingAddress,s.tMarketing) (#774)
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
Reentrancy in Lunar.constructor(address) (#422-436):
External calls:
- _pair = IFactory(_router.factory()).createPair(address(this),_router.WETH()) (#424-425)
Event emitted after the call(s):
- Transfer(address(0),owner(),_tTotal) (#435)
Reentrancy in Lunar.swapAndLiquify(uint256) (#799-823):
External calls:
- swapTokensForBNB(toSwap) (#807)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#849-855)
- addLiquidity(tokensToAddLiquidityWith,bnbToAddLiquidityWith) (#815)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
External calls sending eth:
- addLiquidity(tokensToAddLiquidityWith,bnbToAddLiquidityWith) (#815)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#702)
- addLiquidity(tokensToAddLiquidityWith,bnbToAddLiquidityWith) (#815)
Reentrancy in Lunar.transferFrom(address,address,uint256) (#473-477):
External calls:
- _transfer(sender,recipient,amount) (#474)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#849-855)
External calls sending eth:
- _transfer(sender,recipient,amount) (#474)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
- address(operationsAddress).transfer(operationsAmt) (#821)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#702)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#475)
Apply the check-effects-interactions pattern.

Additional information: link

Address._verifyCallResult(bool,bytes,string) (#226-244) is never used and should be removed
Address.functionCall(address,bytes) (#163-165) is never used and should be removed
Address.functionCall(address,bytes,string) (#167-173) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#175-181) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#183-194) is never used and should be removed
Address.functionDelegateCall(address,bytes) (#211-213) is never used and should be removed
Address.functionDelegateCall(address,bytes,string) (#215-224) is never used and should be removed
Address.functionStaticCall(address,bytes) (#196-198) is never used and should be removed
Address.functionStaticCall(address,bytes,string) (#200-209) is never used and should be removed
Address.isContract(address) (#147-154) is never used and should be removed
Address.sendValue(address,uint256) (#156-161) is never used and should be removed
Context._msgData() (#138-141) is never used and should be removed
SafeMath.div(uint256,uint256,string) (#108-117) is never used and should be removed
SafeMath.mod(uint256,uint256) (#93-95) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#119-128) is never used and should be removed
SafeMath.tryAdd(uint256,uint256) (#39-45) is never used and should be removed
SafeMath.tryDiv(uint256,uint256) (#63-68) is never used and should be removed
SafeMath.tryMod(uint256,uint256) (#70-75) is never used and should be removed
SafeMath.tryMul(uint256,uint256) (#54-61) is never used and should be removed
SafeMath.trySub(uint256,uint256) (#47-52) is never used and should be removed
Remove unused functions.

Additional information: link

Lunar._rTotal (#346) is set pre-construction with a non-constant function or state variable:
- (MAX - (MAX % _tTotal))
Lunar.maxBuyAmount (#348) is set pre-construction with a non-constant function or state variable:
- _tTotal.mul(2).div(100)
Lunar.maxSellAmount (#349) is set pre-construction with a non-constant function or state variable:
- _tTotal.mul(1).div(100)
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

Lunar.swapAndLiquify(uint256) (#799-823) performs a multiplication on the result of a division:
-unitBalance = deltaBalance / (denominator - feeRates.liquidity) (#810)
-bnbToAddLiquidityWith = unitBalance * feeRates.liquidity (#811)
Lunar.swapAndLiquify(uint256) (#799-823) performs a multiplication on the result of a division:
-unitBalance = deltaBalance / (denominator - feeRates.liquidity) (#810)
-operationsAmt = unitBalance * 2 * feeRates.operations (#819)
Consider ordering multiplication before division.

Additional information: link

Reentrancy in Lunar.setRouterAddress(address) (#913-924):
External calls:
- pair = IFactory(_newRouter.factory()).createPair(address(this),_newRouter.WETH()) (#918)
State variables written after the call(s):
- router = _newRouter (#923)
Apply the check-effects-interactions pattern.

Additional information: link

Address.isContract(address) (#147-154) uses assembly
- INLINE ASM (#150-152)
Address._verifyCallResult(bool,bytes,string) (#226-244) uses assembly
- INLINE ASM (#236-239)
Do not use evm assembly.

Additional information: link

Pragma version^0.8.0 (#11) allows old versions
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

Lunar.addLiquidity(uint256,uint256) (#825-838) ignores return value by router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
Ensure that all the return values of the function calls are used.

Additional information: link

Lunar.allowance(address,address).owner (#464) shadows:
- Ownable.owner() (#259-261) (function)
Lunar._approve(address,address,uint256).owner (#698) shadows:
- Ownable.owner() (#259-261) (function)
Rename the local variables that shadow another component.

Additional information: link

Lunar.setMaxWalletPercent(uint256) (#564-568) should emit an event for:
- _maxWalletSize = _tTotal.mul(maxWallPercent).div(10 ** 2) (#565-567)
Lunar.setMaxBuyAndSellAmount(uint256,uint256) (#872-875) should emit an event for:
- maxBuyAmount = _maxBuyamount * 10 ** 9 (#873)
- maxSellAmount = _maxSellAmount * 10 ** 9 (#874)
Lunar.updateSwapTokensAtAmount(uint256) (#877-879) should emit an event for:
- swapTokensAtAmount = amount * 10 ** _decimals (#878)
Lunar.setBuybackUpperLimit(uint256) (#894-896) should emit an event for:
- buyBackUpperLimit = buyBackLimit * 10 ** 15 (#895)
Emit an event for critical parameter changes.

Additional information: link

Reentrancy in Lunar._transfer(address,address,uint256) (#705-750):
External calls:
- buyBackTokens(balance.div(100)) (#741)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
- swapAndLiquify(swapTokensAtAmount) (#744)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#849-855)
External calls sending eth:
- buyBackTokens(balance.div(100)) (#741)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
- swapAndLiquify(swapTokensAtAmount) (#744)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
- address(operationsAddress).transfer(operationsAmt) (#821)
State variables written after the call(s):
- swapAndLiquify(swapTokensAtAmount) (#744)
- _allowances[owner][spender] = amount (#701)
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
- totFeesPaid.marketing += tMarketing (#623)
- totFeesPaid.operations += tOperations (#594)
- totFeesPaid.liquidity += tLiquidity (#613)
- totFeesPaid.buyBack += tBuyback (#603)
- totFeesPaid.rfi += tRfi (#590)
Reentrancy in Lunar.constructor(address) (#422-436):
External calls:
- _pair = IFactory(_router.factory()).createPair(address(this),_router.WETH()) (#424-425)
State variables written after the call(s):
- _isExcludedFromFee[owner()] = true (#431)
- _isExcludedFromFee[marketingAddress] = true (#432)
- _isExcludedFromFee[operationsAddress] = true (#433)
- _rOwned[owner()] = _rTotal (#430)
- pair = _pair (#428)
- router = _router (#427)
Reentrancy in Lunar.swapAndLiquify(uint256) (#799-823):
External calls:
- swapTokensForBNB(toSwap) (#807)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#849-855)
- addLiquidity(tokensToAddLiquidityWith,bnbToAddLiquidityWith) (#815)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
External calls sending eth:
- addLiquidity(tokensToAddLiquidityWith,bnbToAddLiquidityWith) (#815)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
State variables written after the call(s):
- addLiquidity(tokensToAddLiquidityWith,bnbToAddLiquidityWith) (#815)
- _allowances[owner][spender] = amount (#701)
Reentrancy in Lunar.transferFrom(address,address,uint256) (#473-477):
External calls:
- _transfer(sender,recipient,amount) (#474)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#849-855)
External calls sending eth:
- _transfer(sender,recipient,amount) (#474)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
- address(operationsAddress).transfer(operationsAmt) (#821)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#475)
- _allowances[owner][spender] = amount (#701)
Apply the check-effects-interactions pattern.

Additional information: link

Low level call in Address.sendValue(address,uint256) (#156-161):
- (success) = recipient.call{value: amount}() (#159)
Low level call in Address.functionCallWithValue(address,bytes,uint256,string) (#183-194):
- (success,returndata) = target.call{value: value}(data) (#192)
Low level call in Address.functionStaticCall(address,bytes,string) (#200-209):
- (success,returndata) = target.staticcall(data) (#207)
Low level call in Address.functionDelegateCall(address,bytes,string) (#215-224):
- (success,returndata) = target.delegatecall(data) (#222)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

Function IRouter.WETH() (#291) is not in mixedCase
Struct Lunar.feeRatesStruct (#362-368) is not in CapWords
Struct Lunar.valuesFromGetValues (#396-410) is not in CapWords
Parameter Lunar.setFeeRates(uint256,uint256,uint256,uint256,uint256)._rfi (#570) is not in mixedCase
Parameter Lunar.setFeeRates(uint256,uint256,uint256,uint256,uint256)._operations (#570) is not in mixedCase
Parameter Lunar.setFeeRates(uint256,uint256,uint256,uint256,uint256)._marketing (#570) is not in mixedCase
Parameter Lunar.setFeeRates(uint256,uint256,uint256,uint256,uint256)._liquidity (#570) is not in mixedCase
Parameter Lunar.setFeeRates(uint256,uint256,uint256,uint256,uint256)._buyback (#570) is not in mixedCase
Parameter Lunar.setSellFeeRates(uint256,uint256,uint256,uint256,uint256)._rfi (#579) is not in mixedCase
Parameter Lunar.setSellFeeRates(uint256,uint256,uint256,uint256,uint256)._operations (#579) is not in mixedCase
Parameter Lunar.setSellFeeRates(uint256,uint256,uint256,uint256,uint256)._marketing (#579) is not in mixedCase
Parameter Lunar.setSellFeeRates(uint256,uint256,uint256,uint256,uint256)._liquidity (#579) is not in mixedCase
Parameter Lunar.setSellFeeRates(uint256,uint256,uint256,uint256,uint256)._buyback (#579) is not in mixedCase
Parameter Lunar.setMaxBuyAndSellAmount(uint256,uint256)._maxBuyamount (#872) is not in mixedCase
Parameter Lunar.setMaxBuyAndSellAmount(uint256,uint256)._maxSellAmount (#872) is not in mixedCase
Parameter Lunar.updateSwapEnabled(bool)._enabled (#881) is not in mixedCase
Parameter Lunar.updateBuybackEnabled(bool)._enabled (#885) is not in mixedCase
Parameter Lunar.setAntibot(address,bool)._bot (#889) is not in mixedCase
Constant Lunar._decimals (#342) is not in UPPER_CASE_WITH_UNDERSCORES
Variable Lunar._maxWalletSize (#351) is not in mixedCase
Constant Lunar.deadAddress (#356) is not in UPPER_CASE_WITH_UNDERSCORES
Constant Lunar._name (#358) is not in UPPER_CASE_WITH_UNDERSCORES
Constant Lunar._symbol (#359) is not in UPPER_CASE_WITH_UNDERSCORES
Follow the Solidity naming convention.

Additional information: link

Reentrancy in Lunar._transfer(address,address,uint256) (#705-750):
External calls:
- swapAndLiquify(swapTokensAtAmount) (#744)
- address(operationsAddress).transfer(operationsAmt) (#821)
External calls sending eth:
- buyBackTokens(balance.div(100)) (#741)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
- swapAndLiquify(swapTokensAtAmount) (#744)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
- address(operationsAddress).transfer(operationsAmt) (#821)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
- _rOwned[address(this)] += rLiquidity (#619)
- _rOwned[address(this)] += rBuyback (#609)
- _rOwned[marketingAddress] += rMarketing (#629)
- _rOwned[address(this)] += rOperations (#598)
- _rOwned[sender] = _rOwned[sender] - s.rAmount (#765)
- _rOwned[recipient] = _rOwned[recipient] + s.rTransferAmount (#766)
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
- _rTotal -= rRfi (#589)
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
- _tOwned[address(this)] += tOperations (#596)
- _tOwned[marketingAddress] += tMarketing (#627)
- _tOwned[sender] = _tOwned[sender] - tAmount (#759)
- _tOwned[address(this)] += tLiquidity (#617)
- _tOwned[address(this)] += tBuyback (#607)
- _tOwned[recipient] = _tOwned[recipient] + s.tTransferAmount (#762)
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
- totFeesPaid.marketing += tMarketing (#623)
- totFeesPaid.operations += tOperations (#594)
- totFeesPaid.liquidity += tLiquidity (#613)
- totFeesPaid.buyBack += tBuyback (#603)
- totFeesPaid.rfi += tRfi (#590)
Event emitted after the call(s):
- Transfer(sender,recipient,s.tTransferAmount) (#772)
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
- Transfer(sender,address(this),s.tLiquidity + s.tOperations + s.tBuyback) (#773)
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
- Transfer(sender,marketingAddress,s.tMarketing) (#774)
- _tokenTransfer(from,to,amount,! (_isExcludedFromFee[from] || _isExcludedFromFee[to]),isSale) (#749)
Reentrancy in Lunar.transferFrom(address,address,uint256) (#473-477):
External calls:
- _transfer(sender,recipient,amount) (#474)
- address(operationsAddress).transfer(operationsAmt) (#821)
External calls sending eth:
- _transfer(sender,recipient,amount) (#474)
- router.addLiquidityETH{value: bnbAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#830-837)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#791-796)
- address(operationsAddress).transfer(operationsAmt) (#821)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#475)
- _allowances[owner][spender] = amount (#701)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#702)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#475)
Apply the check-effects-interactions pattern.

Additional information: link

Variable Lunar.setFeeRates(uint256,uint256,uint256,uint256,uint256)._operations (#570) is too similar to Lunar._takeOperations(uint256,uint256).rOperations (#593)
Variable Lunar.setSellFeeRates(uint256,uint256,uint256,uint256,uint256)._operations (#579) is too similar to Lunar._takeOperations(uint256,uint256).rOperations (#593)
Variable Lunar.setFeeRates(uint256,uint256,uint256,uint256,uint256)._operations (#570) is too similar to Lunar._getRValues(Lunar.valuesFromGetValues,uint256,bool,uint256).rOperations (#665)
Variable Lunar.setSellFeeRates(uint256,uint256,uint256,uint256,uint256)._operations (#579) is too similar to Lunar._getRValues(Lunar.valuesFromGetValues,uint256,bool,uint256).rOperations (#665)
Variable Lunar.setFeeRates(uint256,uint256,uint256,uint256,uint256)._operations (#570) is too similar to Lunar._takeOperations(uint256,uint256).tOperations (#593)
Variable Lunar.setSellFeeRates(uint256,uint256,uint256,uint256,uint256)._operations (#579) is too similar to Lunar._takeOperations(uint256,uint256).tOperations (#593)
Variable Lunar._takeOperations(uint256,uint256).rOperations (#593) is too similar to Lunar._takeOperations(uint256,uint256).tOperations (#593)
Variable Lunar._getRValues(Lunar.valuesFromGetValues,uint256,bool,uint256).rOperations (#665) is too similar to Lunar._takeOperations(uint256,uint256).tOperations (#593)
Prevent variables from having similar names.

Additional information: link

renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#268-270)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#272-275)
name() should be declared external:
- Lunar.name() (#439-441)
symbol() should be declared external:
- Lunar.symbol() (#442-444)
decimals() should be declared external:
- Lunar.decimals() (#445-447)
totalSupply() should be declared external:
- Lunar.totalSupply() (#450-452)
transfer(address,uint256) should be declared external:
- Lunar.transfer(address,uint256) (#459-462)
allowance(address,address) should be declared external:
- Lunar.allowance(address,address) (#464-466)
approve(address,uint256) should be declared external:
- Lunar.approve(address,uint256) (#468-471)
transferFrom(address,address,uint256) should be declared external:
- Lunar.transferFrom(address,address,uint256) (#473-477)
increaseAllowance(address,uint256) should be declared external:
- Lunar.increaseAllowance(address,uint256) (#479-482)
decreaseAllowance(address,uint256) should be declared external:
- Lunar.decreaseAllowance(address,uint256) (#484-487)
isExcludedFromReward(address) should be declared external:
- Lunar.isExcludedFromReward(address) (#489-491)
deliver(uint256) should be declared external:
- Lunar.deliver(uint256) (#493-500)
reflectionFromToken(uint256,bool) should be declared external:
- Lunar.reflectionFromToken(uint256,bool) (#503-512)
excludeFromReward(address) should be declared external:
- Lunar.excludeFromReward(address) (#528-535)
excludeFromFee(address) should be declared external:
- Lunar.excludeFromFee(address) (#551-553)
includeInFee(address) should be declared external:
- Lunar.includeInFee(address) (#555-557)
isExcludedFromFee(address) should be declared external:
- Lunar.isExcludedFromFee(address) (#560-562)
isBot(address) should be declared external:
- Lunar.isBot(address) (#898-900)
Use the external attribute for functions never called from the contract.

Additional information: link