ALTO [ALLIANCE] Token

Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)

Reentrancy in BuyBackToken._transfer(address,address,uint256) (#666-704):
External calls:
- swapTokens(contractTokenBalance) (#684)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#735-741)
- buyBackTokens(balance.div(100)) (#692)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
External calls sending eth:
- swapTokens(contractTokenBalance) (#684)
- recipient.transfer(amount) (#988)
- buyBackTokens(balance.div(100)) (#692)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _liquidityFee = _previousLiquidityFee (#915)
- _liquidityFee = 0 (#908)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#882)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#798)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#807)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#818)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#828)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#799)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#809)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#819)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#830)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _rTotal = _rTotal.sub(rFee) (#837)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#884)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#817)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#827)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#808)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#829)
- buyBackTokens(balance.div(100)) (#692)
- inSwapAndLiquify = true (#493)
- inSwapAndLiquify = false (#495)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- marketingFee = previousMarketingFee (#917)
- marketingFee = 0 (#910)
Apply the check-effects-interactions pattern.

Additional information: link

Contract ownership is not renounced (belongs to a wallet)

Redundant expression "this (#20)" inContext (#14-23)
Remove redundant statements if they congest code but offer no value.

Additional information: link

BuyBackToken.addLiquidity(uint256,uint256) (#763-776) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#768-775)
Ensure that all the return values of the function calls are used.

Additional information: link

Reentrancy in BuyBackToken._transfer(address,address,uint256) (#666-704):
External calls:
- swapTokens(contractTokenBalance) (#684)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#735-741)
- buyBackTokens(balance.div(100)) (#692)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
External calls sending eth:
- swapTokens(contractTokenBalance) (#684)
- recipient.transfer(amount) (#988)
- buyBackTokens(balance.div(100)) (#692)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
Event emitted after the call(s):
- SwapETHForTokens(amount,path) (#760)
- buyBackTokens(balance.div(100)) (#692)
- Transfer(sender,recipient,tTransferAmount) (#802)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- Transfer(sender,recipient,tTransferAmount) (#822)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- Transfer(sender,recipient,tTransferAmount) (#812)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- Transfer(sender,recipient,tTransferAmount) (#833)
- _tokenTransfer(from,to,amount,takeFee) (#703)
Reentrancy in BuyBackToken.constructor(string,string,uint8,uint256,uint256,uint256,uint256,address) (#498-539):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#529-530)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_tTotal) (#538)
Reentrancy in BuyBackToken.swapETHForTokens(uint256) (#746-761):
External calls:
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
Event emitted after the call(s):
- SwapETHForTokens(amount,path) (#760)
Reentrancy in BuyBackToken.swapTokensForEth(uint256) (#726-744):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#735-741)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#743)
Reentrancy in BuyBackToken.transferFrom(address,address,uint256) (#576-580):
External calls:
- _transfer(sender,recipient,amount) (#577)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#735-741)
External calls sending eth:
- _transfer(sender,recipient,amount) (#577)
- recipient.transfer(amount) (#988)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#663)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#578)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in BuyBackToken._transfer(address,address,uint256) (#666-704):
External calls:
- swapTokens(contractTokenBalance) (#684)
- recipient.transfer(amount) (#988)
External calls sending eth:
- swapTokens(contractTokenBalance) (#684)
- recipient.transfer(amount) (#988)
- buyBackTokens(balance.div(100)) (#692)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _liquidityFee = _previousLiquidityFee (#915)
- _liquidityFee = 0 (#908)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _previousLiquidityFee = _liquidityFee (#903)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _previousTaxFee = _taxFee (#902)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#882)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#798)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#807)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#818)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#828)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#799)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#809)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#819)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#830)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _rTotal = _rTotal.sub(rFee) (#837)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _tFeeTotal = _tFeeTotal.add(tFee) (#838)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#884)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#817)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#827)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#808)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#829)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _taxFee = _previousTaxFee (#914)
- _taxFee = 0 (#907)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- buybackFee = previousBuybackFee (#916)
- buybackFee = 0 (#909)
- buyBackTokens(balance.div(100)) (#692)
- inSwapAndLiquify = true (#493)
- inSwapAndLiquify = false (#495)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- marketingFee = previousMarketingFee (#917)
- marketingFee = 0 (#910)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- previousBuybackFee = buybackFee (#904)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- previousMarketingFee = marketingFee (#905)
Event emitted after the call(s):
- SwapETHForTokens(amount,path) (#760)
- buyBackTokens(balance.div(100)) (#692)
- Transfer(sender,recipient,tTransferAmount) (#802)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- Transfer(sender,recipient,tTransferAmount) (#812)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- Transfer(sender,recipient,tTransferAmount) (#822)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- Transfer(sender,recipient,tTransferAmount) (#833)
- _tokenTransfer(from,to,amount,takeFee) (#703)
Reentrancy in BuyBackToken.transferFrom(address,address,uint256) (#576-580):
External calls:
- _transfer(sender,recipient,amount) (#577)
- recipient.transfer(amount) (#988)
External calls sending eth:
- _transfer(sender,recipient,amount) (#577)
- recipient.transfer(amount) (#988)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#578)
- _allowances[owner][spender] = amount (#662)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#663)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#578)
Apply the check-effects-interactions pattern.

Additional information: link

Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#288) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#289)
Variable BuyBackToken._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#858) is too similar to BuyBackToken._transferStandard(address,address,uint256).tTransferAmount (#797)
Variable BuyBackToken._getValues(uint256).rTransferAmount (#843) is too similar to BuyBackToken._getTValues(uint256).tTransferAmount (#850)
Variable BuyBackToken._transferToExcluded(address,address,uint256).rTransferAmount (#806) is too similar to BuyBackToken._getValues(uint256).tTransferAmount (#842)
Variable BuyBackToken.reflectionFromToken(uint256,bool).rTransferAmount (#624) is too similar to BuyBackToken._getTValues(uint256).tTransferAmount (#850)
Variable BuyBackToken._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#858) is too similar to BuyBackToken._transferBothExcluded(address,address,uint256).tTransferAmount (#826)
Variable BuyBackToken.reflectionFromToken(uint256,bool).rTransferAmount (#624) is too similar to BuyBackToken._getValues(uint256).tTransferAmount (#842)
Variable BuyBackToken._transferFromExcluded(address,address,uint256).rTransferAmount (#816) is too similar to BuyBackToken._transferFromExcluded(address,address,uint256).tTransferAmount (#816)
Variable BuyBackToken._getValues(uint256).rTransferAmount (#843) is too similar to BuyBackToken._getValues(uint256).tTransferAmount (#842)
Variable BuyBackToken._transferBothExcluded(address,address,uint256).rTransferAmount (#826) is too similar to BuyBackToken._transferToExcluded(address,address,uint256).tTransferAmount (#806)
Variable BuyBackToken._transferFromExcluded(address,address,uint256).rTransferAmount (#816) is too similar to BuyBackToken._getTValues(uint256).tTransferAmount (#850)
Variable BuyBackToken._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#858) is too similar to BuyBackToken._transferToExcluded(address,address,uint256).tTransferAmount (#806)
Variable BuyBackToken._transferFromExcluded(address,address,uint256).rTransferAmount (#816) is too similar to BuyBackToken._getValues(uint256).tTransferAmount (#842)
Variable BuyBackToken._getValues(uint256).rTransferAmount (#843) is too similar to BuyBackToken._transferStandard(address,address,uint256).tTransferAmount (#797)
Variable BuyBackToken.reflectionFromToken(uint256,bool).rTransferAmount (#624) is too similar to BuyBackToken._transferStandard(address,address,uint256).tTransferAmount (#797)
Variable BuyBackToken._getValues(uint256).rTransferAmount (#843) is too similar to BuyBackToken._transferBothExcluded(address,address,uint256).tTransferAmount (#826)
Variable BuyBackToken._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#858) is too similar to BuyBackToken._transferFromExcluded(address,address,uint256).tTransferAmount (#816)
Variable BuyBackToken.reflectionFromToken(uint256,bool).rTransferAmount (#624) is too similar to BuyBackToken._transferBothExcluded(address,address,uint256).tTransferAmount (#826)
Variable BuyBackToken._transferBothExcluded(address,address,uint256).rTransferAmount (#826) is too similar to BuyBackToken._getValues(uint256).tTransferAmount (#842)
Variable BuyBackToken._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#858) is too similar to BuyBackToken._getTValues(uint256).tTransferAmount (#850)
Variable BuyBackToken._transferToExcluded(address,address,uint256).rTransferAmount (#806) is too similar to BuyBackToken._transferToExcluded(address,address,uint256).tTransferAmount (#806)
Variable BuyBackToken._transferFromExcluded(address,address,uint256).rTransferAmount (#816) is too similar to BuyBackToken._transferStandard(address,address,uint256).tTransferAmount (#797)
Variable BuyBackToken._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#858) is too similar to BuyBackToken._getValues(uint256).tTransferAmount (#842)
Variable BuyBackToken._transferFromExcluded(address,address,uint256).rTransferAmount (#816) is too similar to BuyBackToken._transferBothExcluded(address,address,uint256).tTransferAmount (#826)
Variable BuyBackToken._getValues(uint256).rTransferAmount (#843) is too similar to BuyBackToken._transferToExcluded(address,address,uint256).tTransferAmount (#806)
Variable BuyBackToken.reflectionFromToken(uint256,bool).rTransferAmount (#624) is too similar to BuyBackToken._transferToExcluded(address,address,uint256).tTransferAmount (#806)
Variable BuyBackToken._transferBothExcluded(address,address,uint256).rTransferAmount (#826) is too similar to BuyBackToken._transferBothExcluded(address,address,uint256).tTransferAmount (#826)
Variable BuyBackToken._getValues(uint256).rTransferAmount (#843) is too similar to BuyBackToken._transferFromExcluded(address,address,uint256).tTransferAmount (#816)
Variable BuyBackToken._transferFromExcluded(address,address,uint256).rTransferAmount (#816) is too similar to BuyBackToken._transferToExcluded(address,address,uint256).tTransferAmount (#806)
Variable BuyBackToken._transferBothExcluded(address,address,uint256).rTransferAmount (#826) is too similar to BuyBackToken._transferStandard(address,address,uint256).tTransferAmount (#797)
Variable BuyBackToken._transferStandard(address,address,uint256).rTransferAmount (#797) is too similar to BuyBackToken._transferFromExcluded(address,address,uint256).tTransferAmount (#816)
Variable BuyBackToken._transferStandard(address,address,uint256).rTransferAmount (#797) is too similar to BuyBackToken._transferBothExcluded(address,address,uint256).tTransferAmount (#826)
Variable BuyBackToken._transferStandard(address,address,uint256).rTransferAmount (#797) is too similar to BuyBackToken._transferStandard(address,address,uint256).tTransferAmount (#797)
Variable BuyBackToken._transferToExcluded(address,address,uint256).rTransferAmount (#806) is too similar to BuyBackToken._transferFromExcluded(address,address,uint256).tTransferAmount (#816)
Variable BuyBackToken._transferToExcluded(address,address,uint256).rTransferAmount (#806) is too similar to BuyBackToken._transferStandard(address,address,uint256).tTransferAmount (#797)
Variable BuyBackToken._transferToExcluded(address,address,uint256).rTransferAmount (#806) is too similar to BuyBackToken._transferBothExcluded(address,address,uint256).tTransferAmount (#826)
Variable BuyBackToken._transferBothExcluded(address,address,uint256).rTransferAmount (#826) is too similar to BuyBackToken._getTValues(uint256).tTransferAmount (#850)
Variable BuyBackToken._transferStandard(address,address,uint256).rTransferAmount (#797) is too similar to BuyBackToken._getTValues(uint256).tTransferAmount (#850)
Variable BuyBackToken._transferStandard(address,address,uint256).rTransferAmount (#797) is too similar to BuyBackToken._transferToExcluded(address,address,uint256).tTransferAmount (#806)
Variable BuyBackToken._transferToExcluded(address,address,uint256).rTransferAmount (#806) is too similar to BuyBackToken._getTValues(uint256).tTransferAmount (#850)
Variable BuyBackToken.reflectionFromToken(uint256,bool).rTransferAmount (#624) is too similar to BuyBackToken._transferFromExcluded(address,address,uint256).tTransferAmount (#816)
Variable BuyBackToken._transferBothExcluded(address,address,uint256).rTransferAmount (#826) is too similar to BuyBackToken._transferFromExcluded(address,address,uint256).tTransferAmount (#816)
Variable BuyBackToken._transferStandard(address,address,uint256).rTransferAmount (#797) is too similar to BuyBackToken._getValues(uint256).tTransferAmount (#842)
Prevent variables from having similar names.

Additional information: link

BuyBackToken.slitherConstructorVariables() (#420-994) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#426)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#175-178)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#180-184)
getUnlockTime() should be declared external:
- Ownable.getUnlockTime() (#186-188)
getTime() should be declared external:
- Ownable.getTime() (#190-192)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#194-199)
unlock() should be declared external:
- Ownable.unlock() (#201-206)
name() should be declared external:
- BuyBackToken.name() (#541-543)
symbol() should be declared external:
- BuyBackToken.symbol() (#545-547)
decimals() should be declared external:
- BuyBackToken.decimals() (#549-551)
transfer(address,uint256) should be declared external:
- BuyBackToken.transfer(address,uint256) (#562-565)
allowance(address,address) should be declared external:
- BuyBackToken.allowance(address,address) (#567-569)
approve(address,uint256) should be declared external:
- BuyBackToken.approve(address,uint256) (#571-574)
transferFrom(address,address,uint256) should be declared external:
- BuyBackToken.transferFrom(address,address,uint256) (#576-580)
increaseAllowance(address,uint256) should be declared external:
- BuyBackToken.increaseAllowance(address,uint256) (#582-585)
decreaseAllowance(address,uint256) should be declared external:
- BuyBackToken.decreaseAllowance(address,uint256) (#587-590)
isExcludedFromReward(address) should be declared external:
- BuyBackToken.isExcludedFromReward(address) (#592-594)
totalFees() should be declared external:
- BuyBackToken.totalFees() (#596-598)
minimumTokensBeforeSwapAmount() should be declared external:
- BuyBackToken.minimumTokensBeforeSwapAmount() (#600-602)
buyBackUpperLimitAmount() should be declared external:
- BuyBackToken.buyBackUpperLimitAmount() (#604-606)
deliver(uint256) should be declared external:
- BuyBackToken.deliver(uint256) (#608-615)
reflectionFromToken(uint256,bool) should be declared external:
- BuyBackToken.reflectionFromToken(uint256,bool) (#618-627)
excludeFromReward(address) should be declared external:
- BuyBackToken.excludeFromReward(address) (#635-643)
isExcludedFromFee(address) should be declared external:
- BuyBackToken.isExcludedFromFee(address) (#920-922)
excludeFromFee(address) should be declared external:
- BuyBackToken.excludeFromFee(address) (#924-926)
includeInFee(address) should be declared external:
- BuyBackToken.includeInFee(address) (#928-930)
setBuyBackEnabled(bool) should be declared external:
- BuyBackToken.setBuyBackEnabled(bool) (#968-971)
Use the external attribute for functions never called from the contract.

Additional information: link

BuyBackToken.constructor(string,string,uint8,uint256,uint256,uint256,uint256,address) (#498-539) performs a multiplication on the result of a division:
-_maxTxAmount = _tTotal.div(1000).mul(3) (#520)
BuyBackToken.constructor(string,string,uint8,uint256,uint256,uint256,uint256,address) (#498-539) performs a multiplication on the result of a division:
-minimumTokensBeforeSwap = _tTotal.div(10000).mul(2) (#522)
BuyBackToken.swapTokens(uint256) (#706-717) performs a multiplication on the result of a division:
-transferToAddressETH(dappbuilderAddress,transferredBalance.div(_liquidityFee).mul(dappbuilderFee)) (#714)
BuyBackToken.swapTokens(uint256) (#706-717) performs a multiplication on the result of a division:
-transferToAddressETH(marketingAddress,transferredBalance.div(_liquidityFee).mul(marketingFee.sub(dappbuilderFee))) (#715)
Consider ordering multiplication before division.

Additional information: link

BuyBackToken.allowance(address,address).owner (#567) shadows:
- Ownable.owner() (#166-168) (function)
BuyBackToken._approve(address,address,uint256).owner (#658) shadows:
- Ownable.owner() (#166-168) (function)
Rename the local variables that shadow another component.

Additional information: link

BuyBackToken.setTaxFee(uint256) (#932-934) should emit an event for:
- _taxFee = taxFee (#933)
BuyBackToken.setBuybackFee(uint256) (#936-939) should emit an event for:
- buybackFee = _buybackFee (#937)
- _liquidityFee = buybackFee.add(marketingFee) (#938)
BuyBackToken.setMaxTxAmount(uint256) (#941-943) should emit an event for:
- _maxTxAmount = maxTxAmount (#942)
BuyBackToken.setMarketingFee(uint256) (#945-949) should emit an event for:
- marketingFee = _marketingFee (#947)
- _liquidityFee = buybackFee.add(marketingFee) (#948)
BuyBackToken.setNumTokensSellToAddToLiquidity(uint256) (#951-953) should emit an event for:
- minimumTokensBeforeSwap = _minimumTokensBeforeSwap (#952)
BuyBackToken.setBuybackUpperLimit(uint256) (#955-957) should emit an event for:
- buyBackUpperLimit = buyBackLimit (#956)
Emit an event for critical parameter changes.

Additional information: link

BuyBackToken.constructor(string,string,uint8,uint256,uint256,uint256,uint256,address)._ma (#498) lacks a zero-check on :
- marketingAddress = address(_ma) (#508)
BuyBackToken.setMarketingAddress(address)._marketingAddress (#959) lacks a zero-check on :
- marketingAddress = address(_marketingAddress) (#960)
Check that the address is not zero.

Additional information: link

Reentrancy in BuyBackToken._transfer(address,address,uint256) (#666-704):
External calls:
- swapTokens(contractTokenBalance) (#684)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#735-741)
- buyBackTokens(balance.div(100)) (#692)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
External calls sending eth:
- swapTokens(contractTokenBalance) (#684)
- recipient.transfer(amount) (#988)
- buyBackTokens(balance.div(100)) (#692)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _previousLiquidityFee = _liquidityFee (#903)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _previousTaxFee = _taxFee (#902)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _tFeeTotal = _tFeeTotal.add(tFee) (#838)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- _taxFee = _previousTaxFee (#914)
- _taxFee = 0 (#907)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- buybackFee = previousBuybackFee (#916)
- buybackFee = 0 (#909)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- previousBuybackFee = buybackFee (#904)
- _tokenTransfer(from,to,amount,takeFee) (#703)
- previousMarketingFee = marketingFee (#905)
Reentrancy in BuyBackToken.constructor(string,string,uint8,uint256,uint256,uint256,uint256,address) (#498-539):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#529-530)
State variables written after the call(s):
- _isExcludedFromFee[owner()] = true (#535)
- _isExcludedFromFee[address(this)] = true (#536)
- uniswapV2Router = _uniswapV2Router (#532)
Reentrancy in BuyBackToken.transferFrom(address,address,uint256) (#576-580):
External calls:
- _transfer(sender,recipient,amount) (#577)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#735-741)
External calls sending eth:
- _transfer(sender,recipient,amount) (#577)
- recipient.transfer(amount) (#988)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#753-758)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#578)
- _allowances[owner][spender] = amount (#662)
Apply the check-effects-interactions pattern.

Additional information: link

Ownable.unlock() (#201-206) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp > _lockTime,Contract is locked until 7 days) (#203)
Avoid relying on block.timestamp.

Additional information: link

Address.isContract(address) (#96-105) uses assembly
- INLINE ASM (#103)
Address._functionCallWithValue(address,bytes,uint256,string) (#133-150) uses assembly
- INLINE ASM (#142-145)
Do not use evm assembly.

Additional information: link

BuyBackToken.includeInReward(address) (#645-656) has costly operations inside a loop:
- _excluded.pop() (#652)
Use a local variable to hold the loop computation result.

Additional information: link

Address._functionCallWithValue(address,bytes,uint256,string) (#133-150) is never used and should be removed
Address.functionCall(address,bytes) (#116-118) is never used and should be removed
Address.functionCall(address,bytes,string) (#120-122) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#124-126) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#128-131) is never used and should be removed
Address.isContract(address) (#96-105) is never used and should be removed
Address.sendValue(address,uint256) (#107-113) is never used and should be removed
BuyBackToken.addLiquidity(uint256,uint256) (#763-776) is never used and should be removed
Context._msgData() (#19-22) is never used and should be removed
SafeMath.mod(uint256,uint256) (#84-86) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#88-91) is never used and should be removed
Remove unused functions.

Additional information: link

Low level call in Address.sendValue(address,uint256) (#107-113):
- (success) = recipient.call{value: amount}() (#111)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#133-150):
- (success,returndata) = target.call{value: weiValue}(data) (#136)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#245) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#246) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#262) is not in mixedCase
Function IUniswapV2Router01.WETH() (#283) is not in mixedCase
Parameter BuyBackToken.calculateTaxFee(uint256)._amount (#887) is not in mixedCase
Parameter BuyBackToken.calculateLiquidityFee(uint256)._amount (#893) is not in mixedCase
Parameter BuyBackToken.setBuybackFee(uint256)._buybackFee (#936) is not in mixedCase
Parameter BuyBackToken.setMarketingFee(uint256)._marketingFee (#945) is not in mixedCase
Parameter BuyBackToken.setNumTokensSellToAddToLiquidity(uint256)._minimumTokensBeforeSwap (#951) is not in mixedCase
Parameter BuyBackToken.setMarketingAddress(address)._marketingAddress (#959) is not in mixedCase
Parameter BuyBackToken.setSwapAndLiquifyEnabled(bool)._enabled (#963) is not in mixedCase
Parameter BuyBackToken.setBuyBackEnabled(bool)._enabled (#968) is not in mixedCase
Parameter BuyBackToken.presale(bool)._presale (#973) is not in mixedCase
Variable BuyBackToken._taxFee (#446) is not in mixedCase
Variable BuyBackToken._maxTxAmount (#460) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Average 30d PancakeSwap liquidity is less than $100. Token is either dead or inactive.

Average 30d PancakeSwap volume is less than $100. Token is either dead or inactive.

Average PancakeSwap trading volume, liqudity, number of swaps are extremely low. Token seems to be dead.

Contract has 10% buy tax and 5% 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

Unable to find token on CoinMarketCap

Additional information: link

Unable to find token contract audit

Unable to find audit link on the website

Unable to find token on CoinHunt

Additional information: link

Alexa traffic rank is very low

Additional information: link

Young tokens have high risks of scam / price dump / death

Token has no active CoinGecko listing / rank

Token has no active CoinMarketCap listing / rank

Young tokens have high risks of price dump / death

Young tokens have high risks of price dump / death