card.addLiquidity(uint256,uint256) (#1729-1744) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in card._transfer(address,address,uint256) (#1569-1654):
External calls:
- swapAndSendToFee(marketingTokens) (#1596)
- IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1662)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1720-1726)
- swapAndLiquify(swapTokens) (#1599)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1700-1706)
- swapAndSendDividends(sellTokens) (#1602)
- success = IERC20(BUSD).transfer(address(dividendTracker),dividends) (#1761)
- dividendTracker.distributeBUSDDividends(dividends) (#1764)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1720-1726)
External calls sending eth:
- swapAndLiquify(swapTokens) (#1599)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
State variables written after the call(s):
- super._transfer(from,address(this),fees) (#1631)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#1037)
- _balances[recipient] = _balances[recipient].add(amount) (#1038)
- super._transfer(from,deadWallet,burnShare) (#1635)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#1037)
- _balances[recipient] = _balances[recipient].add(amount) (#1038)
- super._transfer(from,to,amount) (#1639)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#1037)
- _balances[recipient] = _balances[recipient].add(amount) (#1038)
- swapping = false (#1604)
Apply the check-effects-interactions pattern.
Additional information: link
card.swapAndSendToFee(uint256) (#1656-1663) ignores return value by IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1662)
card.removeStuckToken(address) (#1751-1756) ignores return value by IERC20(_address).transfer(owner(),IERC20(_address).balanceOf(address(this))) (#1755)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.
Additional information: link
Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
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.
Contract ownership is not renounced (belongs to a wallet)
Reentrancy in DividendPayingToken._withdrawDividendOfUser(address) (#1204-1220):
External calls:
- success = IERC20(BUSD).transfer(user,_withdrawableDividend) (#1209)
State variables written after the call(s):
- withdrawnDividends[user] = withdrawnDividends[user].sub(_withdrawableDividend) (#1212)
Reentrancy in card.updateDividendTracker(address) (#1421-1436):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (#1428)
- newDividendTracker.excludeFromDividends(address(this)) (#1429)
- newDividendTracker.excludeFromDividends(owner()) (#1430)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (#1431)
State variables written after the call(s):
- dividendTracker = newDividendTracker (#1435)
Apply the check-effects-interactions pattern.
Additional information: link
card._transfer(address,address,uint256).iterations (#1647) is a local variable never initialized
card._transfer(address,address,uint256).claims (#1647) is a local variable never initialized
card._transfer(address,address,uint256).lastProcessedIndex (#1647) 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
card.claim() (#1556-1558) ignores return value by dividendTracker.processAccount(msg.sender,false) (#1557)
card._transfer(address,address,uint256) (#1569-1654) ignores return value by dividendTracker.process(gas) (#1647-1652)
card.addLiquidity(uint256,uint256) (#1729-1744) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
Ensure that all the return values of the function calls are used.
Additional information: link
DividendPayingToken.constructor(string,string)._name (#1178) shadows:
- ERC20._name (#859) (state variable)
DividendPayingToken.constructor(string,string)._symbol (#1178) shadows:
- ERC20._symbol (#860) (state variable)
DividendPayingToken.dividendOf(address)._owner (#1226) shadows:
- Ownable._owner (#652) (state variable)
DividendPayingToken.withdrawableDividendOf(address)._owner (#1233) shadows:
- Ownable._owner (#652) (state variable)
DividendPayingToken.withdrawnDividendOf(address)._owner (#1240) shadows:
- Ownable._owner (#652) (state variable)
DividendPayingToken.accumulativeDividendOf(address)._owner (#1250) shadows:
- Ownable._owner (#652) (state variable)
Rename the local variables that shadow another component.
Additional information: link
card.updateUniswapV2Router(address)._uniswapV2Pair (#1442-1443) lacks a zero-check on :
- uniswapV2Pair = _uniswapV2Pair (#1444)
card.setMarketingWallet(address).wallet (#1462) lacks a zero-check on :
- _marketingWalletAddress = wallet (#1463)
Check that the address is not zero.
Additional information: link
DividendPayingToken._withdrawDividendOfUser(address) (#1204-1220) has external calls inside a loop: success = IERC20(BUSD).transfer(user,_withdrawableDividend) (#1209)
Favor pull over push strategy for external calls.
Additional information: link
Variable 'card._transfer(address,address,uint256).claims (#1647)' in card._transfer(address,address,uint256) (#1569-1654) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1648)
Variable 'card._transfer(address,address,uint256).lastProcessedIndex (#1647)' in card._transfer(address,address,uint256) (#1569-1654) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1648)
Variable 'card._transfer(address,address,uint256).iterations (#1647)' in card._transfer(address,address,uint256) (#1569-1654) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1648)
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 card._transfer(address,address,uint256) (#1569-1654):
External calls:
- swapAndSendToFee(marketingTokens) (#1596)
- IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1662)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1720-1726)
- swapAndLiquify(swapTokens) (#1599)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1700-1706)
External calls sending eth:
- swapAndLiquify(swapTokens) (#1599)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
State variables written after the call(s):
- swapAndLiquify(swapTokens) (#1599)
- _allowances[owner][spender] = amount (#1103)
Reentrancy in card._transfer(address,address,uint256) (#1569-1654):
External calls:
- swapAndSendToFee(marketingTokens) (#1596)
- IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1662)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1720-1726)
- swapAndLiquify(swapTokens) (#1599)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1700-1706)
- swapAndSendDividends(sellTokens) (#1602)
- success = IERC20(BUSD).transfer(address(dividendTracker),dividends) (#1761)
- dividendTracker.distributeBUSDDividends(dividends) (#1764)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1720-1726)
External calls sending eth:
- swapAndLiquify(swapTokens) (#1599)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
State variables written after the call(s):
- swapAndSendDividends(sellTokens) (#1602)
- _allowances[owner][spender] = amount (#1103)
Reentrancy in card.constructor() (#1383-1415):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1390-1391)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (#1394)
- uniswapV2Router = _uniswapV2Router (#1393)
Reentrancy in card.constructor() (#1383-1415):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1390-1391)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1396)
- dividendTracker.excludeFromDividends(pair) (#1483)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (#1399)
- dividendTracker.excludeFromDividends(address(this)) (#1400)
- dividendTracker.excludeFromDividends(owner()) (#1401)
- dividendTracker.excludeFromDividends(deadWallet) (#1402)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (#1403)
State variables written after the call(s):
- _mint(owner(),100000000 * (10 ** 18)) (#1414)
- _balances[account] = _balances[account].add(amount) (#1057)
- excludeFromFees(owner(),true) (#1406)
- _isExcludedFromFees[account] = excluded (#1449)
- excludeFromFees(_marketingWalletAddress,true) (#1407)
- _isExcludedFromFees[account] = excluded (#1449)
- excludeFromFees(address(this),true) (#1408)
- _isExcludedFromFees[account] = excluded (#1449)
- _mint(owner(),100000000 * (10 ** 18)) (#1414)
- _totalSupply = _totalSupply.add(amount) (#1056)
Reentrancy in cardDividendTracker.processAccount(address,bool) (#1966-1976):
External calls:
- amount = _withdrawDividendOfUser(account) (#1967)
- success = IERC20(BUSD).transfer(user,_withdrawableDividend) (#1209)
State variables written after the call(s):
- lastClaimTimes[account] = block.timestamp (#1970)
Reentrancy in card.swapAndLiquify(uint256) (#1665-1686):
External calls:
- swapTokensForEth(half) (#1677)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1700-1706)
- addLiquidity(otherHalf,newBalance) (#1683)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1683)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (#1683)
- _allowances[owner][spender] = amount (#1103)
Reentrancy in card.updateUniswapV2Router(address) (#1438-1445):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(uniswapV2Router.factory()).createPair(address(this),uniswapV2Router.WETH()) (#1442-1443)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (#1444)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in card._setAutomatedMarketMakerPair(address,bool) (#1478-1487):
External calls:
- dividendTracker.excludeFromDividends(pair) (#1483)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (#1486)
Reentrancy in card._transfer(address,address,uint256) (#1569-1654):
External calls:
- swapAndSendToFee(marketingTokens) (#1596)
- IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1662)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1720-1726)
- swapAndLiquify(swapTokens) (#1599)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1700-1706)
External calls sending eth:
- swapAndLiquify(swapTokens) (#1599)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1104)
- swapAndLiquify(swapTokens) (#1599)
- SwapAndLiquify(half,newBalance,otherHalf) (#1685)
- swapAndLiquify(swapTokens) (#1599)
Reentrancy in card._transfer(address,address,uint256) (#1569-1654):
External calls:
- swapAndSendToFee(marketingTokens) (#1596)
- IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1662)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1720-1726)
- swapAndLiquify(swapTokens) (#1599)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1700-1706)
- swapAndSendDividends(sellTokens) (#1602)
- success = IERC20(BUSD).transfer(address(dividendTracker),dividends) (#1761)
- dividendTracker.distributeBUSDDividends(dividends) (#1764)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1720-1726)
External calls sending eth:
- swapAndLiquify(swapTokens) (#1599)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1104)
- swapAndSendDividends(sellTokens) (#1602)
- SendDividends(tokens,dividends) (#1765)
- swapAndSendDividends(sellTokens) (#1602)
- Transfer(sender,recipient,amount) (#1039)
- super._transfer(from,to,amount) (#1639)
- Transfer(sender,recipient,amount) (#1039)
- super._transfer(from,address(this),fees) (#1631)
- Transfer(sender,recipient,amount) (#1039)
- super._transfer(from,deadWallet,burnShare) (#1635)
Reentrancy in card._transfer(address,address,uint256) (#1569-1654):
External calls:
- swapAndSendToFee(marketingTokens) (#1596)
- IERC20(BUSD).transfer(_marketingWalletAddress,newBalance) (#1662)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1720-1726)
- swapAndLiquify(swapTokens) (#1599)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1700-1706)
- swapAndSendDividends(sellTokens) (#1602)
- success = IERC20(BUSD).transfer(address(dividendTracker),dividends) (#1761)
- dividendTracker.distributeBUSDDividends(dividends) (#1764)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1720-1726)
- dividendTracker.setBalance(address(from),balanceOf(from)) (#1641)
- dividendTracker.setBalance(address(to),balanceOf(to)) (#1642)
- dividendTracker.process(gas) (#1647-1652)
External calls sending eth:
- swapAndLiquify(swapTokens) (#1599)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1648)
Reentrancy in card.constructor() (#1383-1415):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1390-1391)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1396)
- dividendTracker.excludeFromDividends(pair) (#1483)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (#1486)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1396)
Reentrancy in card.constructor() (#1383-1415):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1390-1391)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1396)
- dividendTracker.excludeFromDividends(pair) (#1483)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (#1399)
- dividendTracker.excludeFromDividends(address(this)) (#1400)
- dividendTracker.excludeFromDividends(owner()) (#1401)
- dividendTracker.excludeFromDividends(deadWallet) (#1402)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (#1403)
Event emitted after the call(s):
- ExcludeFromFees(account,excluded) (#1451)
- excludeFromFees(owner(),true) (#1406)
- ExcludeFromFees(account,excluded) (#1451)
- excludeFromFees(address(this),true) (#1408)
- ExcludeFromFees(account,excluded) (#1451)
- excludeFromFees(_marketingWalletAddress,true) (#1407)
- Transfer(address(0),account,amount) (#1058)
- _mint(owner(),100000000 * (10 ** 18)) (#1414)
Reentrancy in cardDividendTracker.processAccount(address,bool) (#1966-1976):
External calls:
- amount = _withdrawDividendOfUser(account) (#1967)
- success = IERC20(BUSD).transfer(user,_withdrawableDividend) (#1209)
Event emitted after the call(s):
- Claim(account,amount,automatic) (#1971)
Reentrancy in card.processDividendTracker(uint256) (#1551-1554):
External calls:
- (iterations,claims,lastProcessedIndex) = dividendTracker.process(gas) (#1552)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,false,gas,tx.origin) (#1553)
Reentrancy in card.swapAndLiquify(uint256) (#1665-1686):
External calls:
- swapTokensForEth(half) (#1677)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1700-1706)
- addLiquidity(otherHalf,newBalance) (#1683)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1683)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (#1735-1742)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1104)
- addLiquidity(otherHalf,newBalance) (#1683)
- SwapAndLiquify(half,newBalance,otherHalf) (#1685)
Reentrancy in card.swapAndSendDividends(uint256) (#1758-1767):
External calls:
- swapTokensForBUSD(tokens) (#1759)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1720-1726)
- success = IERC20(BUSD).transfer(address(dividendTracker),dividends) (#1761)
- dividendTracker.distributeBUSDDividends(dividends) (#1764)
Event emitted after the call(s):
- SendDividends(tokens,dividends) (#1765)
Reentrancy in card.updateDividendTracker(address) (#1421-1436):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (#1428)
- newDividendTracker.excludeFromDividends(address(this)) (#1429)
- newDividendTracker.excludeFromDividends(owner()) (#1430)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (#1431)
Event emitted after the call(s):
- UpdateDividendTracker(newAddress,address(dividendTracker)) (#1433)
Apply the check-effects-interactions pattern.
Additional information: link
cardDividendTracker.getAccount(address) (#1830-1873) uses timestamp for comparisons
Dangerous comparisons:
- nextClaimTime > block.timestamp (#1870-1872)
cardDividendTracker.canAutoClaim(uint256) (#1894-1900) uses timestamp for comparisons
Dangerous comparisons:
- lastClaimTime > block.timestamp (#1895)
- block.timestamp.sub(lastClaimTime) >= claimWait (#1899)
Avoid relying on block.timestamp.
Additional information: link
Context._msgData() (#639-642) is never used and should be removed
DividendPayingToken._transfer(address,address,uint256) (#1260-1266) is never used and should be removed
SafeMath.mod(uint256,uint256) (#596-598) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#612-615) is never used and should be removed
SafeMathInt.abs(int256) (#439-442) is never used and should be removed
SafeMathInt.div(int256,int256) (#410-416) is never used and should be removed
SafeMathInt.mul(int256,int256) (#398-405) is never used and should be removed
Remove unused functions.
Additional information: link
card.totalBuyFees (#1328) is set pre-construction with a non-constant function or state variable:
- buyBUSDRewardsFee.add(buyLiquidityFee).add(buyMarketingFee).add(buyBurnFee)
card.totalSellFees (#1334) is set pre-construction with a non-constant function or state variable:
- sellBUSDRewardsFee.add(sellLiquidityFee).add(sellMarketingFee).add(sellBurnFee)
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.2 (#3) allows old versions
Pragma version^0.6.2 (#148) allows old versions
Pragma version^0.6.2 (#170) allows old versions
Pragma version^0.6.2 (#226) allows old versions
Pragma version^0.6.2 (#293) allows old versions
Pragma version^0.6.2 (#321) allows old versions
Pragma version^0.6.2 (#385) allows old versions
Pragma version^0.6.2 (#455) allows old versions
Pragma version^0.6.2 (#473) allows old versions
Pragma version^0.6.2 (#622) allows old versions
Pragma version^0.6.2 (#647) allows old versions
Pragma version^0.6.2 (#707) allows old versions
Pragma version^0.6.2 (#791) allows old versions
Pragma version^0.6.2 (#820) allows old versions
Pragma version^0.6.2 (#1132) allows old versions
Pragma version^0.6.2 (#1304) 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
Function IUniswapV2Router01.WETH() (#7) is not in mixedCase
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#187) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#188) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#205) is not in mixedCase
Parameter DividendPayingToken.dividendOf(address)._owner (#1226) is not in mixedCase
Parameter DividendPayingToken.withdrawableDividendOf(address)._owner (#1233) is not in mixedCase
Parameter DividendPayingToken.withdrawnDividendOf(address)._owner (#1240) is not in mixedCase
Parameter DividendPayingToken.accumulativeDividendOf(address)._owner (#1250) is not in mixedCase
Variable DividendPayingToken.BUSD (#1152) is not in mixedCase
Constant DividendPayingToken.magnitude (#1158) is not in UPPER_CASE_WITH_UNDERSCORES
Contract card (#1306-1768) is not in CapWords
Parameter card.removeStuckToken(address)._address (#1751) is not in mixedCase
Variable card.BUSD (#1318) is not in mixedCase
Variable card._isBlacklisted (#1322) is not in mixedCase
Variable card._marketingWalletAddress (#1336) is not in mixedCase
Contract cardDividendTracker (#1770-1978) is not in CapWords
Parameter cardDividendTracker.getAccount(address)._account (#1830) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#640)" inContext (#634-643)
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 (#12) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#13)
Variable DividendPayingToken._withdrawDividendOfUser(address)._withdrawableDividend (#1205) is too similar to cardDividendTracker.getAccount(address).withdrawableDividends (#1835)
Prevent variables from having similar names.
Additional information: link
card.constructor() (#1383-1415) uses literals with too many digits:
- _mint(owner(),100000000 * (10 ** 18)) (#1414)
card.updateGasForProcessing(uint256) (#1490-1495) uses literals with too many digits:
- require(bool,string)(newValue >= 200000 && newValue <= 500000,card: gasForProcessing must be between 200,000 and 500,000) (#1491)
card.slitherConstructorVariables() (#1306-1768) uses literals with too many digits:
- deadWallet = 0x000000000000000000000000000000000000dEaD (#1316)
card.slitherConstructorVariables() (#1306-1768) uses literals with too many digits:
- swapTokensAtAmount = 100000 * (10 ** 18) (#1320)
card.slitherConstructorVariables() (#1306-1768) uses literals with too many digits:
- gasForProcessing = 300000 (#1340)
cardDividendTracker.constructor() (#1790-1793) uses literals with too many digits:
- minimumTokenBalanceForDividends = 200000 * (10 ** 18) (#1792)
cardDividendTracker.getAccountAtIndex(uint256) (#1875-1892) uses literals with too many digits:
- (0x0000000000000000000000000000000000000000,- 1,- 1,0,0,0,0,0) (#1886)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
SafeMathInt.MAX_INT256 (#393) is never used in SafeMathInt (#391-449)
Remove unused state variables.
Additional information: link
card.buyBUSDRewardsFee (#1324) should be constant
card.buyBurnFee (#1327) should be constant
card.buyLiquidityFee (#1325) should be constant
card.buyMarketingFee (#1326) should be constant
card.deadWallet (#1316) should be constant
card.sellBUSDRewardsFee (#1330) should be constant
card.sellBurnFee (#1333) should be constant
card.sellLiquidityFee (#1331) should be constant
card.sellMarketingFee (#1332) should be constant
card.swapTokensAtAmount (#1320) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
get(IterableMapping.Map,address) should be declared external:
- IterableMapping.get(IterableMapping.Map,address) (#237-239)
getIndexOfKey(IterableMapping.Map,address) should be declared external:
- IterableMapping.getIndexOfKey(IterableMapping.Map,address) (#241-246)
getKeyAtIndex(IterableMapping.Map,uint256) should be declared external:
- IterableMapping.getKeyAtIndex(IterableMapping.Map,uint256) (#248-250)
size(IterableMapping.Map) should be declared external:
- IterableMapping.size(IterableMapping.Map) (#254-256)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#687-690)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#696-700)
name() should be declared external:
- ERC20.name() (#879-881)
symbol() should be declared external:
- ERC20.symbol() (#887-889)
decimals() should be declared external:
- ERC20.decimals() (#904-906)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (#930-933)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (#938-940)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (#949-952)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (#967-975)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (#989-992)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (#1008-1011)
distributeBUSDDividends(uint256) should be declared external:
- DividendPayingToken.distributeBUSDDividends(uint256) (#1183-1194)
withdrawDividend() should be declared external:
- DividendPayingToken.withdrawDividend() (#1198-1200)
- cardDividendTracker.withdrawDividend() (#1799-1801)
dividendOf(address) should be declared external:
- DividendPayingToken.dividendOf(address) (#1226-1228)
withdrawnDividendOf(address) should be declared external:
- DividendPayingToken.withdrawnDividendOf(address) (#1240-1242)
updateDividendTracker(address) should be declared external:
- card.updateDividendTracker(address) (#1421-1436)
updateUniswapV2Router(address) should be declared external:
- card.updateUniswapV2Router(address) (#1438-1445)
excludeMultipleAccountsFromFees(address[],bool) should be declared external:
- card.excludeMultipleAccountsFromFees(address[],bool) (#1454-1460)
setAutomatedMarketMakerPair(address,bool) should be declared external:
- card.setAutomatedMarketMakerPair(address,bool) (#1467-1471)
updateGasForProcessing(uint256) should be declared external:
- card.updateGasForProcessing(uint256) (#1490-1495)
isExcludedFromFees(address) should be declared external:
- card.isExcludedFromFees(address) (#1509-1511)
withdrawableDividendOf(address) should be declared external:
- card.withdrawableDividendOf(address) (#1513-1515)
dividendTokenBalanceOf(address) should be declared external:
- card.dividendTokenBalanceOf(address) (#1517-1519)
getAccountAtIndex(uint256) should be declared external:
- cardDividendTracker.getAccountAtIndex(uint256) (#1875-1892)
process(uint256) should be declared external:
- cardDividendTracker.process(uint256) (#1919-1964)
Use the external attribute for functions never called from the contract.
Additional information: link
Contract name (Crypney - Crypto Credit Card) contains non-alphanumeric characters.
Not a direct threat, but may indicate unreliable intentions of developer. Non-alphanumeric chars (,.;!#*&") are extremely rare among low risk tokens.
Not a direct threat, but may indicate unreliable intentions of developer. Widespread names (e.g. Elon, King, Moon, Doge) are common among meme-tokens and scams. The allow to gain free hype and attract unexperienced investors.
Unable to find website, listings and other project-related information
Young tokens have high risks of scam / price dump / death
Token has no active CoinGecko listing / rank
Token has no active CoinMarketCap listing / rank
Unable to find Telegram and Twitter accounts