SOLDAO Token

SOLDAO.swapBNBForSOLANA(uint256) (#1959-1974) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: tokenAmount}(0,path,address(this),block.timestamp) (#1968-1973)
Ensure that an arbitrary user cannot withdraw unauthorized funds.

Additional information: link

Reentrancy in SOLDAO._transfer(address,address,uint256) (#1814-1936):
External calls:
- swapTokensForEth(tokenAmount) (#1887)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1949-1955)
- swapAndSendDividends(sellTokens) (#1902)
- success = IERC20(SOLANA).transfer(address(dividendTracker),dividends) (#1979)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: tokenAmount}(0,path,address(this),block.timestamp) (#1968-1973)
- dividendTracker.distributeSOLANADividends(dividends) (#1982)
External calls sending eth:
- address(_DevWalletAddress).transfer(BNBAmountForApp) (#1897)
- address(_FoudWalletAddress).transfer(BNBAmountForFoud) (#1898)
- swapAndSendDividends(sellTokens) (#1902)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: tokenAmount}(0,path,address(this),block.timestamp) (#1968-1973)
State variables written after the call(s):
- super._transfer(from,address(this),fees_scope_1) (#1918)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#518)
- _balances[recipient] = _balances[recipient].add(amount) (#519)
- super._transfer(from,to,amount) (#1921)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#518)
- _balances[recipient] = _balances[recipient].add(amount) (#519)
- swapping = false (#1905)
Apply the check-effects-interactions pattern.

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.

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

SOLDAO._transfer(address,address,uint256).iterations (#1929) is a local variable never initialized
SOLDAO._transfer(address,address,uint256).claims (#1929) is a local variable never initialized
SOLDAO._transfer(address,address,uint256).lastProcessedIndex (#1929) 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

SafeMathInt.MAX_INT256 (#927) is never used in SafeMathInt (#925-983)
Remove unused state variables.

Additional information: link

name() should be declared external:
- ERC20.name() (#360-362)
symbol() should be declared external:
- ERC20.symbol() (#368-370)
decimals() should be declared external:
- ERC20.decimals() (#385-387)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (#411-414)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (#419-421)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (#430-433)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (#448-456)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (#470-473)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (#489-492)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#653-656)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#662-666)
distributeSOLANADividends(uint256) should be declared external:
- DividendPayingToken.distributeSOLANADividends(uint256) (#1033-1044)
withdrawDividend() should be declared external:
- DividendPayingToken.withdrawDividend() (#1048-1050)
- SolanaDaoDividendTracker.withdrawDividend() (#1259-1261)
dividendOf(address) should be declared external:
- DividendPayingToken.dividendOf(address) (#1075-1077)
withdrawnDividendOf(address) should be declared external:
- DividendPayingToken.withdrawnDividendOf(address) (#1089-1091)
get(IterableMapping.Map,address) should be declared external:
- IterableMapping.get(IterableMapping.Map,address) (#1170-1172)
getIndexOfKey(IterableMapping.Map,address) should be declared external:
- IterableMapping.getIndexOfKey(IterableMapping.Map,address) (#1174-1179)
getKeyAtIndex(IterableMapping.Map,uint256) should be declared external:
- IterableMapping.getKeyAtIndex(IterableMapping.Map,uint256) (#1181-1183)
size(IterableMapping.Map) should be declared external:
- IterableMapping.size(IterableMapping.Map) (#1187-1189)
getAccountAtIndex(uint256) should be declared external:
- SolanaDaoDividendTracker.getAccountAtIndex(uint256) (#1334-1351)
process(uint256) should be declared external:
- SolanaDaoDividendTracker.process(uint256) (#1378-1423)
updateDividendTracker(address) should be declared external:
- SOLDAO.updateDividendTracker(address) (#1628-1643)
updateUniswapV2Router(address) should be declared external:
- SOLDAO.updateUniswapV2Router(address) (#1645-1652)
excludeMultipleAccountsFromFees(address[],bool) should be declared external:
- SOLDAO.excludeMultipleAccountsFromFees(address[],bool) (#1661-1667)
setFee(uint256,uint256) should be declared external:
- SOLDAO.setFee(uint256,uint256) (#1677-1683)
setAutomatedMarketMakerPair(address,bool) should be declared external:
- SOLDAO.setAutomatedMarketMakerPair(address,bool) (#1685-1689)
snipe(address) should be declared external:
- SOLDAO.snipe(address) (#1695-1698)
rescueToken(address,uint256) should be declared external:
- SOLDAO.rescueToken(address,uint256) (#1700-1706)
rescueBNB(address) should be declared external:
- SOLDAO.rescueBNB(address) (#1708-1710)
updateGasForProcessing(uint256) should be declared external:
- SOLDAO.updateGasForProcessing(uint256) (#1725-1730)
isExcludedFromFees(address) should be declared external:
- SOLDAO.isExcludedFromFees(address) (#1744-1746)
withdrawableDividendOf(address) should be declared external:
- SOLDAO.withdrawableDividendOf(address) (#1748-1750)
dividendTokenBalanceOf(address) should be declared external:
- SOLDAO.dividendTokenBalanceOf(address) (#1752-1754)
Use the external attribute for functions never called from the contract.

Additional information: link

Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#684) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#685)
Variable DividendPayingToken._withdrawDividendOfUser(address)._withdrawableDividend (#1055) is too similar to SolanaDaoDividendTracker.getAccount(address).withdrawableDividends (#1293)
Variable SOLDAO._transfer(address,address,uint256).fees_scope_0 (#1861) is too similar to SOLDAO._transfer(address,address,uint256).fees_scope_1 (#1916)
Prevent variables from having similar names.

Additional information: link

Reentrancy in DividendPayingToken._withdrawDividendOfUser(address) (#1054-1070):
External calls:
- success = IERC20(SOLANA).transfer(user,_withdrawableDividend) (#1059)
State variables written after the call(s):
- withdrawnDividends[user] = withdrawnDividends[user].sub(_withdrawableDividend) (#1062)
Reentrancy in SOLDAO.updateDividendTracker(address) (#1628-1643):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (#1635)
- newDividendTracker.excludeFromDividends(address(this)) (#1636)
- newDividendTracker.excludeFromDividends(owner()) (#1637)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (#1638)
State variables written after the call(s):
- dividendTracker = newDividendTracker (#1642)
Apply the check-effects-interactions pattern.

Additional information: link

SOLDAO.claim() (#1791-1793) ignores return value by dividendTracker.processAccount(msg.sender,false) (#1792)
SOLDAO._transfer(address,address,uint256) (#1814-1936) ignores return value by dividendTracker.process(gas) (#1929-1934)
Ensure that all the return values of the function calls are used.

Additional information: link

DividendPayingToken.constructor(string,string)._name (#1028) shadows:
- ERC20._name (#340) (state variable)
DividendPayingToken.constructor(string,string)._symbol (#1028) shadows:
- ERC20._symbol (#341) (state variable)
DividendPayingToken.dividendOf(address)._owner (#1075) shadows:
- Ownable._owner (#618) (state variable)
DividendPayingToken.withdrawableDividendOf(address)._owner (#1082) shadows:
- Ownable._owner (#618) (state variable)
DividendPayingToken.withdrawnDividendOf(address)._owner (#1089) shadows:
- Ownable._owner (#618) (state variable)
DividendPayingToken.accumulativeDividendOf(address)._owner (#1099) shadows:
- Ownable._owner (#618) (state variable)
Rename the local variables that shadow another component.

Additional information: link

SOLDAO.setFee(uint256,uint256) (#1677-1683) should emit an event for:
- SOLANARewardsFee = _SOLANARewardsFee (#1681)
- SOLANARewardsShare = _SOLANARewardsShare (#1682)
SOLDAO.setSnipeBlocks(uint8) (#1803-1806) should emit an event for:
- snipeBlocks = _blocks (#1805)
SOLDAO.setSnipeTime(uint256) (#1808-1811) should emit an event for:
- snipeTime = _snipeTime (#1810)
Emit an event for critical parameter changes.

Additional information: link

SOLDAO.updateUniswapV2Router(address)._uniswapV2Pair (#1649-1650) lacks a zero-check on :
- uniswapV2Pair = _uniswapV2Pair (#1651)
SOLDAO.setDevWallet(address).wallet (#1669) lacks a zero-check on :
- _DevWalletAddress = wallet (#1670)
SOLDAO.setFoudWalletAddress(address).wallet (#1673) lacks a zero-check on :
- _FoudWalletAddress = wallet (#1674)
SOLDAO.rescueBNB(address)._recipient (#1708) lacks a zero-check on :
- _recipient.transfer(address(this).balance) (#1709)
Check that the address is not zero.

Additional information: link

DividendPayingToken._withdrawDividendOfUser(address) (#1054-1070) has external calls inside a loop: success = IERC20(SOLANA).transfer(user,_withdrawableDividend) (#1059)
Favor pull over push strategy for external calls.

Additional information: link

Variable 'SOLDAO._transfer(address,address,uint256).claims (#1929)' in SOLDAO._transfer(address,address,uint256) (#1814-1936) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1930)
Variable 'SOLDAO._transfer(address,address,uint256).iterations (#1929)' in SOLDAO._transfer(address,address,uint256) (#1814-1936) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1930)
Variable 'SOLDAO._transfer(address,address,uint256).lastProcessedIndex (#1929)' in SOLDAO._transfer(address,address,uint256) (#1814-1936) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1930)
Move all variable declarations prior to any usage of the variable, and ensure that reaching a variable declaration does not depend on some conditional if it is used unconditionally.

Additional information: link

Reentrancy in SOLDAO._transfer(address,address,uint256) (#1814-1936):
External calls:
- swapTokensForEth(tokenAmount) (#1887)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1949-1955)
- swapAndSendDividends(sellTokens) (#1902)
- success = IERC20(SOLANA).transfer(address(dividendTracker),dividends) (#1979)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: tokenAmount}(0,path,address(this),block.timestamp) (#1968-1973)
- dividendTracker.distributeSOLANADividends(dividends) (#1982)
External calls sending eth:
- address(_DevWalletAddress).transfer(BNBAmountForApp) (#1897)
- address(_FoudWalletAddress).transfer(BNBAmountForFoud) (#1898)
- swapAndSendDividends(sellTokens) (#1902)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: tokenAmount}(0,path,address(this),block.timestamp) (#1968-1973)
State variables written after the call(s):
- swapAndSendDividends(sellTokens) (#1902)
- _allowances[owner][spender] = amount (#584)
Reentrancy in SOLDAO.constructor() (#1591-1624):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1598-1599)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (#1602)
- uniswapV2Router = _uniswapV2Router (#1601)
Reentrancy in SOLDAO.constructor() (#1591-1624):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1598-1599)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1604)
- dividendTracker.excludeFromDividends(pair) (#1718)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (#1607)
- dividendTracker.excludeFromDividends(address(this)) (#1608)
- dividendTracker.excludeFromDividends(owner()) (#1609)
- dividendTracker.excludeFromDividends(deadWallet) (#1610)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (#1611)
State variables written after the call(s):
- _mint(owner(),1000000000 * (10 ** 18)) (#1623)
- _balances[account] = _balances[account].add(amount) (#538)
- excludeFromFees(owner(),true) (#1614)
- _isExcludedFromFees[account] = excluded (#1656)
- excludeFromFees(_DevWalletAddress,true) (#1615)
- _isExcludedFromFees[account] = excluded (#1656)
- excludeFromFees(_FoudWalletAddress,true) (#1616)
- _isExcludedFromFees[account] = excluded (#1656)
- excludeFromFees(address(this),true) (#1617)
- _isExcludedFromFees[account] = excluded (#1656)
- _mint(owner(),1000000000 * (10 ** 18)) (#1623)
- _totalSupply = _totalSupply.add(amount) (#537)
Reentrancy in SolanaDaoDividendTracker.processAccount(address,bool) (#1425-1435):
External calls:
- amount = _withdrawDividendOfUser(account) (#1426)
- success = IERC20(SOLANA).transfer(user,_withdrawableDividend) (#1059)
State variables written after the call(s):
- lastClaimTimes[account] = block.timestamp (#1429)
Reentrancy in SOLDAO.updateUniswapV2Router(address) (#1645-1652):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(uniswapV2Router.factory()).createPair(address(this),uniswapV2Router.WETH()) (#1649-1650)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (#1651)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in SOLDAO._setAutomatedMarketMakerPair(address,bool) (#1713-1722):
External calls:
- dividendTracker.excludeFromDividends(pair) (#1718)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (#1721)
Reentrancy in SOLDAO._transfer(address,address,uint256) (#1814-1936):
External calls:
- swapTokensForEth(tokenAmount) (#1887)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1949-1955)
- swapAndSendDividends(sellTokens) (#1902)
- success = IERC20(SOLANA).transfer(address(dividendTracker),dividends) (#1979)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: tokenAmount}(0,path,address(this),block.timestamp) (#1968-1973)
- dividendTracker.distributeSOLANADividends(dividends) (#1982)
External calls sending eth:
- address(_DevWalletAddress).transfer(BNBAmountForApp) (#1897)
- address(_FoudWalletAddress).transfer(BNBAmountForFoud) (#1898)
- swapAndSendDividends(sellTokens) (#1902)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: tokenAmount}(0,path,address(this),block.timestamp) (#1968-1973)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#585)
- swapAndSendDividends(sellTokens) (#1902)
- SendDividends(tokens,dividends) (#1983)
- swapAndSendDividends(sellTokens) (#1902)
- Transfer(sender,recipient,amount) (#520)
- super._transfer(from,address(this),fees_scope_1) (#1918)
- Transfer(sender,recipient,amount) (#520)
- super._transfer(from,to,amount) (#1921)
Reentrancy in SOLDAO._transfer(address,address,uint256) (#1814-1936):
External calls:
- swapTokensForEth(tokenAmount) (#1887)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1949-1955)
- swapAndSendDividends(sellTokens) (#1902)
- success = IERC20(SOLANA).transfer(address(dividendTracker),dividends) (#1979)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: tokenAmount}(0,path,address(this),block.timestamp) (#1968-1973)
- dividendTracker.distributeSOLANADividends(dividends) (#1982)
- dividendTracker.setBalance(address(from),balanceOf(from)) (#1923)
- dividendTracker.setBalance(address(to),balanceOf(to)) (#1924)
- dividendTracker.process(gas) (#1929-1934)
External calls sending eth:
- address(_DevWalletAddress).transfer(BNBAmountForApp) (#1897)
- address(_FoudWalletAddress).transfer(BNBAmountForFoud) (#1898)
- swapAndSendDividends(sellTokens) (#1902)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: tokenAmount}(0,path,address(this),block.timestamp) (#1968-1973)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1930)
Reentrancy in SOLDAO.constructor() (#1591-1624):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1598-1599)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1604)
- dividendTracker.excludeFromDividends(pair) (#1718)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (#1721)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1604)
Reentrancy in SOLDAO.constructor() (#1591-1624):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1598-1599)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1604)
- dividendTracker.excludeFromDividends(pair) (#1718)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (#1607)
- dividendTracker.excludeFromDividends(address(this)) (#1608)
- dividendTracker.excludeFromDividends(owner()) (#1609)
- dividendTracker.excludeFromDividends(deadWallet) (#1610)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (#1611)
Event emitted after the call(s):
- ExcludeFromFees(account,excluded) (#1658)
- excludeFromFees(_FoudWalletAddress,true) (#1616)
- ExcludeFromFees(account,excluded) (#1658)
- excludeFromFees(_DevWalletAddress,true) (#1615)
- ExcludeFromFees(account,excluded) (#1658)
- excludeFromFees(owner(),true) (#1614)
- ExcludeFromFees(account,excluded) (#1658)
- excludeFromFees(address(this),true) (#1617)
- Transfer(address(0),account,amount) (#539)
- _mint(owner(),1000000000 * (10 ** 18)) (#1623)
Reentrancy in SolanaDaoDividendTracker.processAccount(address,bool) (#1425-1435):
External calls:
- amount = _withdrawDividendOfUser(account) (#1426)
- success = IERC20(SOLANA).transfer(user,_withdrawableDividend) (#1059)
Event emitted after the call(s):
- Claim(account,amount,automatic) (#1430)
Reentrancy in SOLDAO.processDividendTracker(uint256) (#1786-1789):
External calls:
- (iterations,claims,lastProcessedIndex) = dividendTracker.process(gas) (#1787)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,false,gas,tx.origin) (#1788)
Reentrancy in SOLDAO.swapAndSendDividends(uint256) (#1976-1985):
External calls:
- swapBNBForSOLANA(tokens) (#1977)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: tokenAmount}(0,path,address(this),block.timestamp) (#1968-1973)
- success = IERC20(SOLANA).transfer(address(dividendTracker),dividends) (#1979)
- dividendTracker.distributeSOLANADividends(dividends) (#1982)
External calls sending eth:
- swapBNBForSOLANA(tokens) (#1977)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: tokenAmount}(0,path,address(this),block.timestamp) (#1968-1973)
Event emitted after the call(s):
- SendDividends(tokens,dividends) (#1983)
Reentrancy in SOLDAO.updateDividendTracker(address) (#1628-1643):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (#1635)
- newDividendTracker.excludeFromDividends(address(this)) (#1636)
- newDividendTracker.excludeFromDividends(owner()) (#1637)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (#1638)
Event emitted after the call(s):
- UpdateDividendTracker(newAddress,address(dividendTracker)) (#1640)
Apply the check-effects-interactions pattern.

Additional information: link

SolanaDaoDividendTracker.getAccount(address) (#1288-1332) uses timestamp for comparisons
Dangerous comparisons:
- nextClaimTime > block.timestamp (#1329-1331)
SolanaDaoDividendTracker.canAutoClaim(uint256) (#1353-1359) uses timestamp for comparisons
Dangerous comparisons:
- lastClaimTime > block.timestamp (#1354)
- block.timestamp.sub(lastClaimTime) >= claimWait (#1358)
SOLDAO._transfer(address,address,uint256) (#1814-1936) uses timestamp for comparisons
Dangerous comparisons:
- from == address(_AppWalletAddress) && block.timestamp <= applastTime (#1829)
- from == address(_CrossWalletAddress) && block.timestamp <= crosslastTime (#1833)
- liquidityLaunched && block.timestamp <= lastSnipeTime && ! isFirstLaunch (#1860)
Avoid relying on block.timestamp.

Additional information: link

Context._msgData() (#111-114) is never used and should be removed
DividendPayingToken._transfer(address,address,uint256) (#1109-1115) is never used and should be removed
SafeMath.mod(uint256,uint256) (#277-279) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#293-296) is never used and should be removed
SafeMathInt.abs(int256) (#973-976) is never used and should be removed
SafeMathInt.div(int256,int256) (#944-950) is never used and should be removed
SafeMathInt.mul(int256,int256) (#932-939) is never used and should be removed
Remove unused functions.

Additional information: link

Function IUniswapV2Router01.WETH() (#679) is not in mixedCase
Parameter DividendPayingToken.dividendOf(address)._owner (#1075) is not in mixedCase
Parameter DividendPayingToken.withdrawableDividendOf(address)._owner (#1082) is not in mixedCase
Parameter DividendPayingToken.withdrawnDividendOf(address)._owner (#1089) is not in mixedCase
Parameter DividendPayingToken.accumulativeDividendOf(address)._owner (#1099) is not in mixedCase
Variable DividendPayingToken.SOLANA (#1003) is not in mixedCase
Constant DividendPayingToken.magnitude (#1008) is not in UPPER_CASE_WITH_UNDERSCORES
Parameter SolanaDaoDividendTracker.getAccount(address)._account (#1288) is not in mixedCase
Parameter SOLDAO.setFee(uint256,uint256)._SOLANARewardsFee (#1678) is not in mixedCase
Parameter SOLDAO.setFee(uint256,uint256)._SOLANARewardsShare (#1679) is not in mixedCase
Parameter SOLDAO.rescueBNB(address)._recipient (#1708) is not in mixedCase
Parameter SOLDAO.setSnipeBlocks(uint8)._blocks (#1803) is not in mixedCase
Parameter SOLDAO.setSnipeTime(uint256)._snipeTime (#1808) is not in mixedCase
Variable SOLDAO._DevWalletAddress (#1525) is not in mixedCase
Variable SOLDAO._FoudWalletAddress (#1526) is not in mixedCase
Variable SOLDAO.SOLANA (#1528) is not in mixedCase
Variable SOLDAO._isBlacklisted (#1532) is not in mixedCase
Variable SOLDAO.SOLANARewardsFee (#1534) is not in mixedCase
Variable SOLDAO.SOLANARewardsShare (#1535) is not in mixedCase
Variable SOLDAO._AppWalletAddress (#1549) is not in mixedCase
Variable SOLDAO._CrossWalletAddress (#1550) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#112)" inContext (#106-115)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Reentrancy in SOLDAO._transfer(address,address,uint256) (#1814-1936):
External calls:
- address(_DevWalletAddress).transfer(BNBAmountForApp) (#1897)
- address(_FoudWalletAddress).transfer(BNBAmountForFoud) (#1898)
External calls sending eth:
- address(_DevWalletAddress).transfer(BNBAmountForApp) (#1897)
- address(_FoudWalletAddress).transfer(BNBAmountForFoud) (#1898)
- swapAndSendDividends(sellTokens) (#1902)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: tokenAmount}(0,path,address(this),block.timestamp) (#1968-1973)
State variables written after the call(s):
- swapAndSendDividends(sellTokens) (#1902)
- _allowances[owner][spender] = amount (#584)
- super._transfer(from,address(this),fees_scope_1) (#1918)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#518)
- _balances[recipient] = _balances[recipient].add(amount) (#519)
- super._transfer(from,to,amount) (#1921)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#518)
- _balances[recipient] = _balances[recipient].add(amount) (#519)
- swapping = false (#1905)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#585)
- swapAndSendDividends(sellTokens) (#1902)
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1930)
- SendDividends(tokens,dividends) (#1983)
- swapAndSendDividends(sellTokens) (#1902)
- Transfer(sender,recipient,amount) (#520)
- super._transfer(from,address(this),fees_scope_1) (#1918)
- Transfer(sender,recipient,amount) (#520)
- super._transfer(from,to,amount) (#1921)
Apply the check-effects-interactions pattern.

Additional information: link

SolanaDaoDividendTracker.constructor() (#1250-1253) uses literals with too many digits:
- minimumTokenBalanceForDividends = 500000 * (10 ** 18) (#1252)
SolanaDaoDividendTracker.getAccountAtIndex(uint256) (#1334-1351) uses literals with too many digits:
- (0x0000000000000000000000000000000000000000,- 1,- 1,0,0,0,0,0) (#1345)
SOLDAO.constructor() (#1591-1624) uses literals with too many digits:
- _mint(owner(),1000000000 * (10 ** 18)) (#1623)
SOLDAO.updateGasForProcessing(uint256) (#1725-1730) uses literals with too many digits:
- require(bool,string)(newValue >= 200000 && newValue <= 500000,SolanaDao: gasForProcessing must be between 200,000 and 500,000) (#1726)
SOLDAO.slitherConstructorVariables() (#1514-1987) uses literals with too many digits:
- deadWallet = 0x000000000000000000000000000000000000dEaD (#1524)
SOLDAO.slitherConstructorVariables() (#1514-1987) uses literals with too many digits:
- gasForProcessing = 300000 (#1546)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

SOLDAO._AppWalletAddress (#1549) should be constant
SOLDAO._CrossWalletAddress (#1550) should be constant
SOLDAO.appLockTime (#1553) should be constant
SOLDAO.crossLockTime (#1554) should be constant
SOLDAO.deadWallet (#1524) should be constant
SOLDAO.swapTokensAtAmount (#1530) should be constant
Add the constant attributes to state variables that never change.

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.

Token is deployed only at one blockchain

Token has only one trading pair

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

Additional information: link

Unable to find token on CoinGecko

Additional information: link

Unable to verify token contract address on the website

Unable to find audit link on the website

Token is not listed at Mobula.Finance

Additional information: link

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

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

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

Token has no active CoinGecko listing / rank

Token has relatively low CoinMarketCap rank