QT.addLiquidity(uint256,uint256) (#808-821) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in QT._transfer(address,address,uint256) (#701-755):
External calls:
- swapAndLiquify(contractTokenBalance) (#729)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#797-803)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#729)
- recipient.transfer(amount) (#649)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#732)
- finalAmount = takeFee(sender,recipient,amount) (#743)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#835)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#749)
- addBot(recipient) (#740)
- _isbclisted[recipient] = true (#690)
Apply the check-effects-interactions pattern.
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.
Contract ownership is not renounced (belongs to a wallet)
QT._transfer(address,address,uint256) (#701-755) performs a multiplication on the result of a division:
-_sellReserveFeeAmount = amount.div(100).mul(_sellReserveFee) (#714)
Consider ordering multiplication before division.
Additional information: link
QT.addLiquidity(uint256,uint256) (#808-821) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
Ensure that all the return values of the function calls are used.
Additional information: link
QT.allowance(address,address).owner (#531) shadows:
- Ownable.owner() (#158-160) (function)
QT._approve(address,address,uint256).owner (#554) shadows:
- Ownable.owner() (#158-160) (function)
Rename the local variables that shadow another component.
Additional information: link
QT.setBuyTaxes(uint256,uint256,uint256) (#579-585) should emit an event for:
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#584)
QT.setSellTaxes(uint256,uint256,uint256) (#587-593) should emit an event for:
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#592)
QT.setDistributionSettings(uint256,uint256,uint256) (#595-601) should emit an event for:
- _liquidityShare = newLiquidityShare (#596)
- _teamShare = newTeamShare (#598)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#600)
QT.setMaxTxAmount(uint256) (#603-605) should emit an event for:
- _maxTxAmount = maxTxAmount (#604)
QT.setWalletLimit(uint256) (#615-617) should emit an event for:
- _walletMax = newLimit (#616)
QT.setNumTokensBeforeSwap(uint256) (#619-621) should emit an event for:
- minimumTokensBeforeSwap = newLimit (#620)
QT.setKillBlock(uint256) (#693-695) should emit an event for:
- killblock = num (#694)
Emit an event for critical parameter changes.
Additional information: link
QT.setMarketingWalletAddress(address).newAddress (#623) lacks a zero-check on :
- marketingWalletAddress = address(newAddress) (#624)
QT.setTeamWalletAddress(address).newAddress (#627) lacks a zero-check on :
- teamWalletAddress = address(newAddress) (#628)
QT.setLiquidityWalletAddress(address).newAddress (#631) lacks a zero-check on :
- liquidityAddress = address(newAddress) (#632)
Check that the address is not zero.
Additional information: link
Reentrancy in QT.changeRouterVersion(address) (#652-669):
External calls:
- newPairAddress = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#660-661)
State variables written after the call(s):
- isMarketPair[address(uniswapPair)] = true (#668)
- isWalletLimitExempt[address(uniswapPair)] = true (#667)
- uniswapPair = newPairAddress (#664)
- uniswapV2Router = _uniswapV2Router (#665)
Reentrancy in QT.constructor() (#481-509):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#485-486)
State variables written after the call(s):
- _allowances[address(this)][address(uniswapV2Router)] = _totalSupply (#489)
- _balances[_msgSender()] = _totalSupply (#507)
- _totalDistributionShares = _liquidityShare.add(_marketingShare).add(_teamShare) (#496)
- _totalTaxIfBuying = _buyLiquidityFee.add(_buyMarketingFee).add(_buyTeamFee) (#494)
- _totalTaxIfSelling = _sellLiquidityFee.add(_sellMarketingFee).add(_sellTeamFee) (#495)
- isExcludedFromFee[owner()] = true (#491)
- isExcludedFromFee[address(this)] = true (#492)
- isMarketPair[address(uniswapPair)] = true (#505)
- isTxLimitExempt[owner()] = true (#502)
- isTxLimitExempt[address(this)] = true (#503)
- isWalletLimitExempt[owner()] = true (#498)
- isWalletLimitExempt[address(uniswapPair)] = true (#499)
- isWalletLimitExempt[address(this)] = true (#500)
- uniswapV2Router = _uniswapV2Router (#488)
Reentrancy in QT.swapAndLiquify(uint256) (#764-786):
External calls:
- swapTokensForEth(tokensForSwap) (#769)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#797-803)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#785)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#779)
- recipient.transfer(amount) (#649)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#782)
- recipient.transfer(amount) (#649)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#785)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#785)
- _allowances[owner][spender] = amount (#558)
Reentrancy in QT.transferFrom(address,address,uint256) (#679-683):
External calls:
- _transfer(sender,recipient,amount) (#680)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#797-803)
External calls sending eth:
- _transfer(sender,recipient,amount) (#680)
- recipient.transfer(amount) (#649)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#681)
- _allowances[owner][spender] = amount (#558)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in QT._transfer(address,address,uint256) (#701-755):
External calls:
- swapAndLiquify(contractTokenBalance) (#729)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#797-803)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#729)
- recipient.transfer(amount) (#649)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#836)
- finalAmount = takeFee(sender,recipient,amount) (#743)
- Transfer(sender,recipient,finalAmount) (#751)
Reentrancy in QT.constructor() (#481-509):
External calls:
- uniswapPair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#485-486)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_totalSupply) (#508)
Reentrancy in QT.swapAndLiquify(uint256) (#764-786):
External calls:
- swapTokensForEth(tokensForSwap) (#769)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#797-803)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#785)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#779)
- recipient.transfer(amount) (#649)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#782)
- recipient.transfer(amount) (#649)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#785)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#559)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#785)
Reentrancy in QT.swapTokensForEth(uint256) (#788-806):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#797-803)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#805)
Reentrancy in QT.transferFrom(address,address,uint256) (#679-683):
External calls:
- _transfer(sender,recipient,amount) (#680)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#797-803)
External calls sending eth:
- _transfer(sender,recipient,amount) (#680)
- recipient.transfer(amount) (#649)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#559)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#681)
Apply the check-effects-interactions pattern.
Additional information: link
Ownable.unlock() (#193-198) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp > _lockTime,Contract is locked until 7 days) (#195)
Avoid relying on block.timestamp.
Additional information: link
Address.isContract(address) (#89-98) uses assembly
- INLINE ASM (#96)
Address._functionCallWithValue(address,bytes,uint256,string) (#125-142) uses assembly
- INLINE ASM (#134-137)
Do not use evm assembly.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#125-142) is never used and should be removed
Address.functionCall(address,bytes) (#108-110) is never used and should be removed
Address.functionCall(address,bytes,string) (#112-114) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#116-118) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#120-123) is never used and should be removed
Address.isContract(address) (#89-98) is never used and should be removed
Address.sendValue(address,uint256) (#100-106) is never used and should be removed
Context._msgData() (#16-19) is never used and should be removed
SafeMath.mod(uint256,uint256) (#77-79) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#81-84) is never used and should be removed
Remove unused functions.
Additional information: link
QT._totalSupply (#445) is set pre-construction with a non-constant function or state variable:
- 10000000000 * 10 ** _decimals
QT._maxTxAmount (#446) is set pre-construction with a non-constant function or state variable:
- 10000000000 * 10 ** _decimals
QT._walletMax (#447) is set pre-construction with a non-constant function or state variable:
- 10000000000 * 10 ** _decimals
QT.minimumTokensBeforeSwap (#448) is set pre-construction with a non-constant function or state variable:
- 10000000000 * 5 ** _decimals
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
Low level call in Address.sendValue(address,uint256) (#100-106):
- (success) = recipient.call{value: amount}() (#104)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#125-142):
- (success,returndata) = target.call{value: weiValue}(data) (#128)
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() (#232) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#233) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#249) is not in mixedCase
Function IUniswapV2Router01.WETH() (#268) is not in mixedCase
Function QT.Launch() (#562-565) is not in mixedCase
Parameter QT.setSwapAndLiquifyEnabled(bool)._enabled (#635) is not in mixedCase
Variable QT._balances (#418) is not in mixedCase
Variable QT._buyLiquidityFee (#427) is not in mixedCase
Variable QT._buyMarketingFee (#428) is not in mixedCase
Variable QT._buyTeamFee (#429) is not in mixedCase
Variable QT._sellLiquidityFee (#431) is not in mixedCase
Variable QT._sellMarketingFee (#432) is not in mixedCase
Variable QT._sellTeamFee (#433) is not in mixedCase
Variable QT._sellReserveFee (#435) is not in mixedCase
Variable QT._liquidityShare (#437) is not in mixedCase
Variable QT._marketingShare (#438) is not in mixedCase
Variable QT._teamShare (#439) is not in mixedCase
Variable QT._totalTaxIfBuying (#441) is not in mixedCase
Variable QT._totalTaxIfSelling (#442) is not in mixedCase
Variable QT._totalDistributionShares (#443) is not in mixedCase
Variable QT._maxTxAmount (#446) is not in mixedCase
Variable QT._walletMax (#447) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#17)" inContext (#10-20)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in QT._transfer(address,address,uint256) (#701-755):
External calls:
- swapAndLiquify(contractTokenBalance) (#729)
- recipient.transfer(amount) (#649)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#729)
- recipient.transfer(amount) (#649)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#732)
- finalAmount = takeFee(sender,recipient,amount) (#743)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#835)
- _balances[recipient] = _balances[recipient].add(finalAmount) (#749)
- addBot(recipient) (#740)
- _isbclisted[recipient] = true (#690)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#836)
- finalAmount = takeFee(sender,recipient,amount) (#743)
- Transfer(sender,recipient,finalAmount) (#751)
Reentrancy in QT.swapAndLiquify(uint256) (#764-786):
External calls:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#779)
- recipient.transfer(amount) (#649)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#782)
- recipient.transfer(amount) (#649)
External calls sending eth:
- transferToAddressETH(marketingWalletAddress,amountBNBMarketing) (#779)
- recipient.transfer(amount) (#649)
- transferToAddressETH(teamWalletAddress,amountBNBTeam) (#782)
- recipient.transfer(amount) (#649)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#785)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
State variables written after the call(s):
- addLiquidity(tokensForLP,amountBNBLiquidity) (#785)
- _allowances[owner][spender] = amount (#558)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#559)
- addLiquidity(tokensForLP,amountBNBLiquidity) (#785)
Reentrancy in QT.transferFrom(address,address,uint256) (#679-683):
External calls:
- _transfer(sender,recipient,amount) (#680)
- recipient.transfer(amount) (#649)
External calls sending eth:
- _transfer(sender,recipient,amount) (#680)
- recipient.transfer(amount) (#649)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,liquidityAddress,block.timestamp) (#813-820)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#681)
- _allowances[owner][spender] = amount (#558)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#559)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#681)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#273) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#274)
Prevent variables from having similar names.
Additional information: link
QT.slitherConstructorVariables() (#401-843) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#416)
QT.slitherConstructorVariables() (#401-843) uses literals with too many digits:
- _totalSupply = 10000000000 * 10 ** _decimals (#445)
QT.slitherConstructorVariables() (#401-843) uses literals with too many digits:
- _maxTxAmount = 10000000000 * 10 ** _decimals (#446)
QT.slitherConstructorVariables() (#401-843) uses literals with too many digits:
- _walletMax = 10000000000 * 10 ** _decimals (#447)
QT.slitherConstructorVariables() (#401-843) uses literals with too many digits:
- minimumTokensBeforeSwap = 10000000000 * 5 ** _decimals (#448)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
QT._decimals (#408) should be constant
QT._name (#406) should be constant
QT._sellReserveFee (#435) should be constant
QT._symbol (#407) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
waiveOwnership() should be declared external:
- Ownable.waiveOwnership() (#167-170)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#172-176)
getUnlockTime() should be declared external:
- Ownable.getUnlockTime() (#178-180)
getTime() should be declared external:
- Ownable.getTime() (#182-184)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#186-191)
unlock() should be declared external:
- Ownable.unlock() (#193-198)
name() should be declared external:
- QT.name() (#511-513)
symbol() should be declared external:
- QT.symbol() (#515-517)
decimals() should be declared external:
- QT.decimals() (#519-521)
totalSupply() should be declared external:
- QT.totalSupply() (#523-525)
allowance(address,address) should be declared external:
- QT.allowance(address,address) (#531-533)
increaseAllowance(address,uint256) should be declared external:
- QT.increaseAllowance(address,uint256) (#535-538)
decreaseAllowance(address,uint256) should be declared external:
- QT.decreaseAllowance(address,uint256) (#540-543)
minimumTokensBeforeSwapAmount() should be declared external:
- QT.minimumTokensBeforeSwapAmount() (#545-547)
approve(address,uint256) should be declared external:
- QT.approve(address,uint256) (#549-552)
Launch() should be declared external:
- QT.Launch() (#562-565)
setMarketPairStatus(address,bool) should be declared external:
- QT.setMarketPairStatus(address,bool) (#567-569)
setIsExcludedFromFee(address,bool) should be declared external:
- QT.setIsExcludedFromFee(address,bool) (#575-577)
setSwapAndLiquifyEnabled(bool) should be declared external:
- QT.setSwapAndLiquifyEnabled(bool) (#635-638)
setSwapAndLiquifyByLimitOnly(bool) should be declared external:
- QT.setSwapAndLiquifyByLimitOnly(bool) (#640-642)
getCirculatingSupply() should be declared external:
- QT.getCirculatingSupply() (#644-646)
changeRouterVersion(address) should be declared external:
- QT.changeRouterVersion(address) (#652-669)
transfer(address,uint256) should be declared external:
- QT.transfer(address,uint256) (#674-677)
transferFrom(address,address,uint256) should be declared external:
- QT.transferFrom(address,address,uint256) (#679-683)
isbcList(address) should be declared external:
- QT.isbcList(address) (#685-687)
setKillBlock(uint256) should be declared external:
- QT.setKillBlock(uint256) (#693-695)
writebcList(address,bool) should be declared external:
- QT.writebcList(address,bool) (#697-699)
Use the external attribute for functions never called from the contract.
Additional information: link
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 Twitter account
Unable to find Blog account (Reddit or Medium)
Unable to find Youtube account
Unable to find Discord account