Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
SmokingCoin.swapAndLiquify(uint256) (#1596-1639) sends eth to arbitrary user
Dangerous calls:
- devWallet.transfer(devAmt) (#1637)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in SmokingCoin._transfer(address,address,uint256) (#1533-1594):
External calls:
- swapAndLiquify(contractTokenBalance) (#1562)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1650-1656)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1562)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
- address(marketingAddress).transfer(marketingAmt) (#1633)
- devWallet.transfer(devAmt) (#1637)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _rOwned[address(this)] = _rOwned[address(this)].add(rWallet) (#1429)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#1421)
- _rOwned[fixedBurnAddress] = _rOwned[fixedBurnAddress].add(rburn) (#1441)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1757)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1726)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1727)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1820)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1789)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1759)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1790)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1822)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _rTotal = _rTotal.sub(rFee) (#1342)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#1423)
- _tOwned[address(this)] = _tOwned[address(this)].add(tWallet) (#1431)
- _tOwned[fixedBurnAddress] = _tOwned[fixedBurnAddress].add(tburn) (#1443-1445)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1819)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1788)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1758)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1821)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _tTotal = _tTotal.sub(tburn) (#1436)
Apply the check-effects-interactions pattern.
Additional information: link
SmokingCoin.withdrawForeignToken(address,address,uint256) (#1834-1837) ignores return value by foreignToken.transfer(recipient,amount) (#1836)
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.
SmokingCoin.swapAndLiquify(uint256) (#1596-1639) performs a multiplication on the result of a division:
-unitBalance = deltaBalance / (denominator - (buyFee.liquidity + sellFee.liquidity)) (#1613-1614)
-bnbToAddLiquidityWith = unitBalance * (buyFee.liquidity + sellFee.liquidity) (#1615-1616)
SmokingCoin.swapAndLiquify(uint256) (#1596-1639) performs a multiplication on the result of a division:
-unitBalance = deltaBalance / (denominator - (buyFee.liquidity + sellFee.liquidity)) (#1613-1614)
-marketingAmt = unitBalance * 2 * (buyFee.marketing + sellFee.marketing) (#1624-1626)
SmokingCoin.swapAndLiquify(uint256) (#1596-1639) performs a multiplication on the result of a division:
-unitBalance = deltaBalance / (denominator - (buyFee.liquidity + sellFee.liquidity)) (#1613-1614)
-unitBalance * 2 * (buyFee.dev + sellFee.dev) > address(this).balance (#1627-1630)
SmokingCoin.swapAndLiquify(uint256) (#1596-1639) performs a multiplication on the result of a division:
-unitBalance = deltaBalance / (denominator - (buyFee.liquidity + sellFee.liquidity)) (#1613-1614)
-devAmt = unitBalance * 2 * (buyFee.dev + sellFee.dev) (#1627-1630)
Consider ordering multiplication before division.
Additional information: link
SmokingCoin.addLiquidity(uint256,uint256) (#1659-1672) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
Ensure that all the return values of the function calls are used.
Additional information: link
SmokingCoin.allowance(address,address).owner (#1059) shadows:
- Ownable.owner() (#481-483) (function)
SmokingCoin._approve(address,address,uint256).owner (#1522) shadows:
- Ownable.owner() (#481-483) (function)
Rename the local variables that shadow another component.
Additional information: link
SmokingCoin.setNumTokensSellToAddToLiquidity(uint256) (#1318-1320) should emit an event for:
- numTokensSellToAddToLiquidity = numTokens (#1319)
SmokingCoin.setMaxTxPercent(uint256) (#1322-1324) should emit an event for:
- _maxTxAmount = _tTotal.mul(maxTxPercent).div(10 ** 3) (#1323)
SmokingCoin._setMaxWalletSizePercent(uint256) (#1326-1331) should emit an event for:
- _maxWalletSize = _tTotal.mul(maxWalletSize).div(10 ** 3) (#1330)
Emit an event for critical parameter changes.
Additional information: link
SmokingCoin.updateMarketingWallet(address).newAddress (#1214) lacks a zero-check on :
- marketingAddress = newAddress (#1215)
SmokingCoin.updateDevWallet(address).newAddress (#1218) lacks a zero-check on :
- devWallet = newAddress (#1219)
SmokingCoin.updateLiquidityWallet(address).newAddress (#1222) lacks a zero-check on :
- liquidityReceiver = newAddress (#1223)
SmokingCoin.withdrawStuckETH(address,uint256).recipient (#1830) lacks a zero-check on :
- address(recipient).transfer(amount) (#1831)
Check that the address is not zero.
Additional information: link
Reentrancy in SmokingCoin._transfer(address,address,uint256) (#1533-1594):
External calls:
- swapAndLiquify(contractTokenBalance) (#1562)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1650-1656)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1562)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
- address(marketingAddress).transfer(marketingAmt) (#1633)
- devWallet.transfer(devAmt) (#1637)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _burnFee = buyFee.burn (#1501)
- _burnFee = 0 (#1485)
- _burnFee = sellFee.burn (#1509)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _devFee = buyFee.dev (#1502)
- _devFee = 0 (#1486)
- _devFee = sellFee.dev (#1510)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _liquidityFee = 0 (#1483)
- _liquidityFee = sellFee.liquidity (#1507)
- _liquidityFee = buyFee.liquidity (#1499)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _marketingFee = 0 (#1484)
- _marketingFee = sellFee.marketing (#1508)
- _marketingFee = buyFee.marketing (#1500)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _tFeeTotal = _tFeeTotal.add(tFee) (#1343)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _taxFee = 0 (#1482)
- _taxFee = sellFee.tax (#1506)
- _taxFee = buyFee.tax (#1498)
Reentrancy in SmokingCoin.constructor() (#987-1027):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1006-1007)
State variables written after the call(s):
- _isExcludedFromFee[owner()] = true (#1013)
- _isExcludedFromFee[address(this)] = true (#1014)
- _isExcludedFromFee[marketingAddress] = true (#1015)
- _isExcludedFromFee[devWallet] = true (#1016)
- _isExcludedFromFee[liquidityReceiver] = true (#1017)
- _isExcludedFromLimit[marketingAddress] = true (#1020)
- _isExcludedFromLimit[marketingAddress] = true (#1021)
- _isExcludedFromLimit[devWallet] = true (#1022)
- _isExcludedFromLimit[owner()] = true (#1023)
- _isExcludedFromLimit[address(this)] = true (#1024)
- uniswapV2Router = _uniswapV2Router (#1010)
Reentrancy in SmokingCoin.swapAndLiquify(uint256) (#1596-1639):
External calls:
- swapTokensForEth(toSwap) (#1610)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1650-1656)
- addLiquidity(tokensToAddLiquidityWith,bnbToAddLiquidityWith) (#1620)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
External calls sending eth:
- addLiquidity(tokensToAddLiquidityWith,bnbToAddLiquidityWith) (#1620)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
State variables written after the call(s):
- addLiquidity(tokensToAddLiquidityWith,bnbToAddLiquidityWith) (#1620)
- _allowances[owner][spender] = amount (#1529)
Reentrancy in SmokingCoin.transferFrom(address,address,uint256) (#1077-1092):
External calls:
- _transfer(sender,recipient,amount) (#1082)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1650-1656)
External calls sending eth:
- _transfer(sender,recipient,amount) (#1082)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
- address(marketingAddress).transfer(marketingAmt) (#1633)
- devWallet.transfer(devAmt) (#1637)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#1083-1090)
- _allowances[owner][spender] = amount (#1529)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in SmokingCoin._transfer(address,address,uint256) (#1533-1594):
External calls:
- swapAndLiquify(contractTokenBalance) (#1562)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1650-1656)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1562)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
- address(marketingAddress).transfer(marketingAmt) (#1633)
- devWallet.transfer(devAmt) (#1637)
Event emitted after the call(s):
- Transfer(address(this),address(0),tburn) (#1437)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- Transfer(sender,recipient,tTransferAmount) (#1732)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- Transfer(sender,recipient,tTransferAmount) (#1795)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- Transfer(sender,recipient,tTransferAmount) (#1764)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- Transfer(sender,recipient,tTransferAmount) (#1827)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
Reentrancy in SmokingCoin.constructor() (#987-1027):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1006-1007)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_tTotal) (#1026)
Reentrancy in SmokingCoin.swapAndLiquify(uint256) (#1596-1639):
External calls:
- swapTokensForEth(toSwap) (#1610)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1650-1656)
- addLiquidity(tokensToAddLiquidityWith,bnbToAddLiquidityWith) (#1620)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
External calls sending eth:
- addLiquidity(tokensToAddLiquidityWith,bnbToAddLiquidityWith) (#1620)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1530)
- addLiquidity(tokensToAddLiquidityWith,bnbToAddLiquidityWith) (#1620)
Reentrancy in SmokingCoin.transferFrom(address,address,uint256) (#1077-1092):
External calls:
- _transfer(sender,recipient,amount) (#1082)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1650-1656)
External calls sending eth:
- _transfer(sender,recipient,amount) (#1082)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
- address(marketingAddress).transfer(marketingAmt) (#1633)
- devWallet.transfer(devAmt) (#1637)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1530)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#1083-1090)
Apply the check-effects-interactions pattern.
Additional information: link
Ownable.unlock() (#531-540) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp > _lockTime,Contract is locked until a later date) (#536)
Avoid relying on block.timestamp.
Additional information: link
Address.isContract(address) (#290-301) uses assembly
- INLINE ASM (#297-299)
Address._functionCallWithValue(address,bytes,uint256,string) (#416-444) uses assembly
- INLINE ASM (#436-439)
Do not use evm assembly.
Additional information: link
SmokingCoin.includeInReward(address) (#1235-1246) has costly operations inside a loop:
- _excluded.pop() (#1242)
Use a local variable to hold the loop computation result.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#416-444) is never used and should be removed
Address.functionCall(address,bytes) (#351-356) is never used and should be removed
Address.functionCall(address,bytes,string) (#364-370) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#383-395) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#403-414) is never used and should be removed
Address.isContract(address) (#290-301) is never used and should be removed
Address.sendValue(address,uint256) (#319-331) is never used and should be removed
Context._msgData() (#263-266) is never used and should be removed
SafeMath.mod(uint256,uint256) (#232-234) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#248-255) is never used and should be removed
Remove unused functions.
Additional information: link
SmokingCoin._rTotal (#914) is set pre-construction with a non-constant function or state variable:
- (MAX - (MAX % _tTotal))
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.8.10 (#5) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
solc-0.8.10 is not recommended for deployment
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
Low level call in Address.sendValue(address,uint256) (#319-331):
- (success) = recipient.call{value: amount}() (#326)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#416-444):
- (success,returndata) = target.call{value: weiValue}(data) (#425-427)
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() (#610) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#612) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#643) is not in mixedCase
Function IUniswapV2Router01.WETH() (#691) is not in mixedCase
Event SmokingCoinbotAddedToBlacklist(address) (#970) is not in CapWords
Event SmokingCoinbotRemovedFromBlacklist(address) (#971) is not in CapWords
Parameter SmokingCoin.setBothFees(uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16).buy_tax (#1293) is not in mixedCase
Parameter SmokingCoin.setBothFees(uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16).buy_liquidity (#1294) is not in mixedCase
Parameter SmokingCoin.setBothFees(uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16).buy_marketing (#1295) is not in mixedCase
Parameter SmokingCoin.setBothFees(uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16).buy_dev (#1296) is not in mixedCase
Parameter SmokingCoin.setBothFees(uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16).buy_burn (#1297) is not in mixedCase
Parameter SmokingCoin.setBothFees(uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16).sell_tax (#1298) is not in mixedCase
Parameter SmokingCoin.setBothFees(uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16).sell_liquidity (#1299) is not in mixedCase
Parameter SmokingCoin.setBothFees(uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16).sell_marketing (#1300) is not in mixedCase
Parameter SmokingCoin.setBothFees(uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16).sell_dev (#1301) is not in mixedCase
Parameter SmokingCoin.setBothFees(uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16,uint16).sell_burn (#1302) is not in mixedCase
Function SmokingCoin._setMaxWalletSizePercent(uint256) (#1326-1331) is not in mixedCase
Parameter SmokingCoin.setSwapAndLiquifyEnabled(bool)._enabled (#1333) is not in mixedCase
Parameter SmokingCoin.calculateTaxFee(uint256)._amount (#1449) is not in mixedCase
Parameter SmokingCoin.calculateLiquidityFee(uint256)._amount (#1453) is not in mixedCase
Parameter SmokingCoin.calculateMarketingFee(uint256)._amount (#1461) is not in mixedCase
Parameter SmokingCoin.calculateburnFee(uint256)._amount (#1469) is not in mixedCase
Parameter SmokingCoin.calculatedevFee(uint256)._amount (#1477) is not in mixedCase
Variable SmokingCoin._maxTxAmount (#966) is not in mixedCase
Variable SmokingCoin._maxWalletSize (#968) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#264)" inContext (#258-267)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in SmokingCoin._transfer(address,address,uint256) (#1533-1594):
External calls:
- swapAndLiquify(contractTokenBalance) (#1562)
- address(marketingAddress).transfer(marketingAmt) (#1633)
- devWallet.transfer(devAmt) (#1637)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1562)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
- address(marketingAddress).transfer(marketingAmt) (#1633)
- devWallet.transfer(devAmt) (#1637)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _burnFee = buyFee.burn (#1501)
- _burnFee = 0 (#1485)
- _burnFee = sellFee.burn (#1509)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _devFee = buyFee.dev (#1502)
- _devFee = 0 (#1486)
- _devFee = sellFee.dev (#1510)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _liquidityFee = 0 (#1483)
- _liquidityFee = sellFee.liquidity (#1507)
- _liquidityFee = buyFee.liquidity (#1499)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _marketingFee = 0 (#1484)
- _marketingFee = sellFee.marketing (#1508)
- _marketingFee = buyFee.marketing (#1500)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _rOwned[address(this)] = _rOwned[address(this)].add(rWallet) (#1429)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#1421)
- _rOwned[fixedBurnAddress] = _rOwned[fixedBurnAddress].add(rburn) (#1441)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1757)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1726)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1727)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1820)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1789)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1759)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1790)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1822)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _rTotal = _rTotal.sub(rFee) (#1342)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _tFeeTotal = _tFeeTotal.add(tFee) (#1343)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#1423)
- _tOwned[address(this)] = _tOwned[address(this)].add(tWallet) (#1431)
- _tOwned[fixedBurnAddress] = _tOwned[fixedBurnAddress].add(tburn) (#1443-1445)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1819)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1788)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1758)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1821)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _tTotal = _tTotal.sub(tburn) (#1436)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- _taxFee = 0 (#1482)
- _taxFee = sellFee.tax (#1506)
- _taxFee = buyFee.tax (#1498)
Event emitted after the call(s):
- Transfer(address(this),address(0),tburn) (#1437)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- Transfer(sender,recipient,tTransferAmount) (#1732)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- Transfer(sender,recipient,tTransferAmount) (#1795)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- Transfer(sender,recipient,tTransferAmount) (#1764)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
- Transfer(sender,recipient,tTransferAmount) (#1827)
- _tokenTransfer(from,to,amount,takeFee) (#1593)
Reentrancy in SmokingCoin.transferFrom(address,address,uint256) (#1077-1092):
External calls:
- _transfer(sender,recipient,amount) (#1082)
- address(marketingAddress).transfer(marketingAmt) (#1633)
- devWallet.transfer(devAmt) (#1637)
External calls sending eth:
- _transfer(sender,recipient,amount) (#1082)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(liquidityReceiver),block.timestamp) (#1664-1671)
- address(marketingAddress).transfer(marketingAmt) (#1633)
- devWallet.transfer(devAmt) (#1637)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#1083-1090)
- _allowances[owner][spender] = amount (#1529)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1530)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#1083-1090)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#696) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#697)
Variable SmokingCoin._transferStandard(address,address,uint256).rTransferAmount (#1717) is too similar to SmokingCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1742)
Variable SmokingCoin._getRValues(uint256,uint256,uint256,uint256,uint256,uint256).rTransferAmount (#1390-1394) is too similar to SmokingCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1742)
Variable SmokingCoin._getRValues(uint256,uint256,uint256,uint256,uint256,uint256).rTransferAmount (#1390-1394) is too similar to SmokingCoin._transferStandard(address,address,uint256).tTransferAmount (#1711)
Variable SmokingCoin._transferBothExcluded(address,address,uint256).rTransferAmount (#1810) is too similar to SmokingCoin._getTValues(uint256).tTransferAmount (#1362)
Variable SmokingCoin._transferToExcluded(address,address,uint256).rTransferAmount (#1748) is too similar to SmokingCoin._getTValues(uint256).tTransferAmount (#1362)
Variable SmokingCoin._transferStandard(address,address,uint256).rTransferAmount (#1717) is too similar to SmokingCoin._getTValues(uint256).tTransferAmount (#1362)
Variable SmokingCoin._getRValues(uint256,uint256,uint256,uint256,uint256,uint256).rTransferAmount (#1390-1394) is too similar to SmokingCoin._getTValues(uint256).tTransferAmount (#1362)
Variable SmokingCoin.reflectionFromToken(uint256,bool).rTransferAmount (#1184) is too similar to SmokingCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1742)
Variable SmokingCoin._transferFromExcluded(address,address,uint256).rTransferAmount (#1779) is too similar to SmokingCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1742)
Variable SmokingCoin._transferToExcluded(address,address,uint256).rTransferAmount (#1748) is too similar to SmokingCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1773)
Variable SmokingCoin._transferBothExcluded(address,address,uint256).rTransferAmount (#1810) is too similar to SmokingCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1773)
Variable SmokingCoin.reflectionFromToken(uint256,bool).rTransferAmount (#1184) is too similar to SmokingCoin._getTValues(uint256).tTransferAmount (#1362)
Variable SmokingCoin._getRValues(uint256,uint256,uint256,uint256,uint256,uint256).rTransferAmount (#1390-1394) is too similar to SmokingCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#1804)
Variable SmokingCoin._getRValues(uint256,uint256,uint256,uint256,uint256,uint256).rTransferAmount (#1390-1394) is too similar to SmokingCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1773)
Variable SmokingCoin._transferFromExcluded(address,address,uint256).rTransferAmount (#1779) is too similar to SmokingCoin._getTValues(uint256).tTransferAmount (#1362)
Variable SmokingCoin._transferToExcluded(address,address,uint256).rTransferAmount (#1748) is too similar to SmokingCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1742)
Variable SmokingCoin._transferBothExcluded(address,address,uint256).rTransferAmount (#1810) is too similar to SmokingCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1742)
Variable SmokingCoin._transferToExcluded(address,address,uint256).rTransferAmount (#1748) is too similar to SmokingCoin._transferStandard(address,address,uint256).tTransferAmount (#1711)
Variable SmokingCoin._transferBothExcluded(address,address,uint256).rTransferAmount (#1810) is too similar to SmokingCoin._transferStandard(address,address,uint256).tTransferAmount (#1711)
Variable SmokingCoin._transferStandard(address,address,uint256).rTransferAmount (#1717) is too similar to SmokingCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1773)
Variable SmokingCoin._transferFromExcluded(address,address,uint256).rTransferAmount (#1779) is too similar to SmokingCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#1804)
Variable SmokingCoin.reflectionFromToken(uint256,bool).rTransferAmount (#1184) is too similar to SmokingCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#1804)
Variable SmokingCoin._transferToExcluded(address,address,uint256).rTransferAmount (#1748) is too similar to SmokingCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#1804)
Variable SmokingCoin._transferBothExcluded(address,address,uint256).rTransferAmount (#1810) is too similar to SmokingCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#1804)
Variable SmokingCoin.reflectionFromToken(uint256,bool).rTransferAmount (#1184) is too similar to SmokingCoin._transferStandard(address,address,uint256).tTransferAmount (#1711)
Variable SmokingCoin._transferFromExcluded(address,address,uint256).rTransferAmount (#1779) is too similar to SmokingCoin._transferStandard(address,address,uint256).tTransferAmount (#1711)
Variable SmokingCoin._transferStandard(address,address,uint256).rTransferAmount (#1717) is too similar to SmokingCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#1804)
Variable SmokingCoin._transferStandard(address,address,uint256).rTransferAmount (#1717) is too similar to SmokingCoin._transferStandard(address,address,uint256).tTransferAmount (#1711)
Variable SmokingCoin.reflectionFromToken(uint256,bool).rTransferAmount (#1184) is too similar to SmokingCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1773)
Variable SmokingCoin._transferFromExcluded(address,address,uint256).rTransferAmount (#1779) is too similar to SmokingCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1773)
Prevent variables from having similar names.
Additional information: link
SmokingCoin.slitherConstructorVariables() (#897-1839) uses literals with too many digits:
- _tTotal = 100000000000 * 10 ** 9 (#913)
SmokingCoin.slitherConstructorVariables() (#897-1839) uses literals with too many digits:
- fixedBurnAddress = 0x000000000000000000000000000000000000dEaD (#923-924)
SmokingCoin.slitherConstructorVariables() (#897-1839) uses literals with too many digits:
- _maxTxAmount = 7000000000 * 10 ** 9 (#966)
SmokingCoin.slitherConstructorVariables() (#897-1839) uses literals with too many digits:
- numTokensSellToAddToLiquidity = 200000000 * 10 ** 9 (#967)
SmokingCoin.slitherConstructorVariables() (#897-1839) uses literals with too many digits:
- _maxWalletSize = 7000000000 * 10 ** 9 (#968)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
SmokingCoin._decimals (#930) should be constant
SmokingCoin._name (#928) should be constant
SmokingCoin._symbol (#929) should be constant
SmokingCoin.burnDeductSupply (#926) should be constant
SmokingCoin.fixedBurnAddress (#923-924) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#500-503)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#509-516)
geUnlockTime() should be declared external:
- Ownable.geUnlockTime() (#518-520)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#523-528)
unlock() should be declared external:
- Ownable.unlock() (#531-540)
name() should be declared external:
- SmokingCoin.name() (#1029-1031)
symbol() should be declared external:
- SmokingCoin.symbol() (#1033-1035)
decimals() should be declared external:
- SmokingCoin.decimals() (#1037-1039)
totalSupply() should be declared external:
- SmokingCoin.totalSupply() (#1041-1043)
transfer(address,uint256) should be declared external:
- SmokingCoin.transfer(address,uint256) (#1050-1057)
allowance(address,address) should be declared external:
- SmokingCoin.allowance(address,address) (#1059-1066)
approve(address,uint256) should be declared external:
- SmokingCoin.approve(address,uint256) (#1068-1075)
transferFrom(address,address,uint256) should be declared external:
- SmokingCoin.transferFrom(address,address,uint256) (#1077-1092)
increaseAllowance(address,uint256) should be declared external:
- SmokingCoin.increaseAllowance(address,uint256) (#1094-1105)
decreaseAllowance(address,uint256) should be declared external:
- SmokingCoin.decreaseAllowance(address,uint256) (#1107-1121)
isExcludedFromReward(address) should be declared external:
- SmokingCoin.isExcludedFromReward(address) (#1123-1125)
totalFees() should be declared external:
- SmokingCoin.totalFees() (#1127-1129)
burnAddress() should be declared external:
- SmokingCoin.burnAddress() (#1131-1133)
whitelistMultipleAddress(address[],bool) should be declared external:
- SmokingCoin.whitelistMultipleAddress(address[],bool) (#1135-1140)
deliver(uint256) should be declared external:
- SmokingCoin.deliver(uint256) (#1142-1168)
reflectionFromToken(uint256,bool) should be declared external:
- SmokingCoin.reflectionFromToken(uint256,bool) (#1170-1198)
excludeFromReward(address) should be declared external:
- SmokingCoin.excludeFromReward(address) (#1226-1233)
excludeFromFee(address) should be declared external:
- SmokingCoin.excludeFromFee(address) (#1248-1250)
includeInFee(address) should be declared external:
- SmokingCoin.includeInFee(address) (#1252-1254)
excludeFromLimit(address) should be declared external:
- SmokingCoin.excludeFromLimit(address) (#1256-1258)
includeInLimit(address) should be declared external:
- SmokingCoin.includeInLimit(address) (#1260-1262)
setSwapAndLiquifyEnabled(bool) should be declared external:
- SmokingCoin.setSwapAndLiquifyEnabled(bool) (#1333-1336)
setPrivateTradingStatus(bool) should be declared external:
- SmokingCoin.setPrivateTradingStatus(bool) (#1489-1491)
whitelistAddressForPrivateTrading(address,bool) should be declared external:
- SmokingCoin.whitelistAddressForPrivateTrading(address,bool) (#1493-1495)
isExcludedFromFee(address) should be declared external:
- SmokingCoin.isExcludedFromFee(address) (#1513-1515)
isExcludedFromLimit(address) should be declared external:
- SmokingCoin.isExcludedFromLimit(address) (#1517-1519)
withdrawStuckETH(address,uint256) should be declared external:
- SmokingCoin.withdrawStuckETH(address,uint256) (#1830-1832)
withdrawForeignToken(address,address,uint256) should be declared external:
- SmokingCoin.withdrawForeignToken(address,address,uint256) (#1834-1837)
Use the external attribute for functions never called from the contract.
Additional information: link
Unable to find website, listings and other project-related information
Token is marked as scam (rug pull, honeypot, phishing, etc.)
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
Unable to find Telegram and Twitter accounts