POOR [PoorQUACK.com] Token

PoorQuackOT.marketingFallBack() (#1309-1318) ignores return value by this.transfer(_wMarketing,balanceOf(address(this))) (#1312)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.

Additional information: link

PoorQuackOT._doMegaPump(uint256) (#1229-1240) sends eth to arbitrary user
Dangerous calls:
- _wMegaPump.transfer(bnbNewBalance) (#1237)
PoorQuackOT._doBuyBack(uint256) (#1242-1253) sends eth to arbitrary user
Dangerous calls:
- _wBuyBack.transfer(bnbNewBalance) (#1250)
PoorQuackOT._doMarketing(uint256) (#1255-1266) sends eth to arbitrary user
Dangerous calls:
- _wMarketing.transfer(bnbNewBalance) (#1263)
PoorQuackOT.addLiquidity(uint256,uint256) (#1293-1307) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
Ensure that an arbitrary user cannot withdraw unauthorized funds.

Additional information: link

Reentrancy in PoorQuackOT._transfer(address,address,uint256) (#970-1029):
External calls:
- _doTokenomics() (#977)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _doTokenomics() (#977)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _wMarketing.transfer(bnbNewBalance) (#1263)
State variables written after the call(s):
- _transferFromExcluded(sender,recipient,amount) (#1015)
- _rOwned[address(this)] = _rOwned[address(this)].add(rContractFee) (#1080)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1054)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1055)
- _transferToExcluded(sender,recipient,amount) (#1017)
- _rOwned[address(this)] = _rOwned[address(this)].add(rContractFee) (#1080)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1043)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1045)
- _transferStandard(sender,recipient,amount) (#1019)
- _rOwned[address(this)] = _rOwned[address(this)].add(rContractFee) (#1080)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1034)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1035)
- _transferBothExcluded(sender,recipient,amount) (#1021)
- _rOwned[address(this)] = _rOwned[address(this)].add(rContractFee) (#1080)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1064)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1066)
- _transferStandard(sender,recipient,amount) (#1023)
- _rOwned[address(this)] = _rOwned[address(this)].add(rContractFee) (#1080)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1034)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1035)
- _transferFromExcluded(sender,recipient,amount) (#1015)
- _rTotal = _rTotal.sub(newRFee) (#1082)
- _transferToExcluded(sender,recipient,amount) (#1017)
- _rTotal = _rTotal.sub(newRFee) (#1082)
- _transferStandard(sender,recipient,amount) (#1019)
- _rTotal = _rTotal.sub(newRFee) (#1082)
- _transferBothExcluded(sender,recipient,amount) (#1021)
- _rTotal = _rTotal.sub(newRFee) (#1082)
- _transferStandard(sender,recipient,amount) (#1023)
- _rTotal = _rTotal.sub(newRFee) (#1082)
- _transferFromExcluded(sender,recipient,amount) (#1015)
- _tOwned[address(this)] = _tOwned[address(this)].add(tContractFee) (#1079)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1053)
- _transferToExcluded(sender,recipient,amount) (#1017)
- _tOwned[address(this)] = _tOwned[address(this)].add(tContractFee) (#1079)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1044)
- _transferStandard(sender,recipient,amount) (#1019)
- _tOwned[address(this)] = _tOwned[address(this)].add(tContractFee) (#1079)
- _transferBothExcluded(sender,recipient,amount) (#1021)
- _tOwned[address(this)] = _tOwned[address(this)].add(tContractFee) (#1079)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1063)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1065)
- _transferStandard(sender,recipient,amount) (#1023)
- _tOwned[address(this)] = _tOwned[address(this)].add(tContractFee) (#1079)
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.

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)

Contract name (PoorQUACK.com) 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.

PoorQuackOT.addLiquidity(uint256,uint256) (#1293-1307) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
Ensure that all the return values of the function calls are used.

Additional information: link

PoorQuackOT._decimals (#754) should be constant
PoorQuackOT._maxHoldAmount (#783) should be constant
PoorQuackOT._name (#752) should be constant
PoorQuackOT._symbol (#753) should be constant
PoorQuackOT._tTotal (#748) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

PoorQuackOT._reflectFee(uint256,uint256) (#1071-1084) performs a multiplication on the result of a division:
-newTFee = tFee.div(100).mul(_refPer) (#1073)
PoorQuackOT._reflectFee(uint256,uint256) (#1071-1084) performs a multiplication on the result of a division:
-newRFee = rFee.div(100).mul(_refPer) (#1074)
Consider ordering multiplication before division.

Additional information: link

PoorQuackOT.allowance(address,address).owner (#850) shadows:
- Ownable.owner() (#688-690) (function)
PoorQuackOT._approve(address,address,uint256).owner (#957) shadows:
- Ownable.owner() (#688-690) (function)
Rename the local variables that shadow another component.

Additional information: link

PoorQuackOT.setMaxTrans(uint256) (#1130-1133) should emit an event for:
- _maxTransAmount = maxTransAmount (#1132)
PoorQuackOT._setTotalTax(uint256) (#1135-1139) should emit an event for:
- _currentTax = totalTax (#1137)
- _totalTax = totalTax (#1138)
PoorQuackOT._set24Tax(uint256) (#1141-1144) should emit an event for:
- _tax24HR = totalTax (#1143)
PoorQuackOT._set72Tax(uint256) (#1146-1149) should emit an event for:
- _tax72HR = totalTax (#1148)
PoorQuackOT._setRefPer(uint256) (#1151-1154) should emit an event for:
- _refPer = refPer (#1153)
PoorQuackOT._setMegaPumpPer(uint256) (#1156-1159) should emit an event for:
- _megaPumpPer = megaPumpPer (#1158)
PoorQuackOT._setAutoLiqPer(uint256) (#1161-1164) should emit an event for:
- _autoLiqPer = autoLiqPer (#1163)
PoorQuackOT._setBuyBacksPer(uint256) (#1166-1169) should emit an event for:
- _buyBacksPer = buyBacksPer (#1168)
PoorQuackOT._setMinTokensForLiquidity(uint256) (#1176-1179) should emit an event for:
- _minTokensForLiquidity = minTokensForLiquidity (#1178)
Emit an event for critical parameter changes.

Additional information: link

PoorQuackOT.setMegaPumpWallet(address).wallet (#1327) lacks a zero-check on :
- _wMegaPump = wallet (#1329)
PoorQuackOT.setBuyBackWallet(address).wallet (#1332) lacks a zero-check on :
- _wBuyBack = wallet (#1334)
PoorQuackOT.setMarketingWallet(address).wallet (#1337) lacks a zero-check on :
- _wMarketing = wallet (#1339)
Check that the address is not zero.

Additional information: link

Reentrancy in PoorQuackOT._doBuyBack(uint256) (#1242-1253):
External calls:
- swapTokensForEth(amount,address(this)) (#1246)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _wBuyBack.transfer(bnbNewBalance) (#1250)
State variables written after the call(s):
- _buyBacksAllTime += bnbNewBalance (#1252)
Reentrancy in PoorQuackOT._doLiquidity(uint256) (#1213-1227):
External calls:
- swapTokensForEth(bnbHalf,address(this)) (#1222)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- addLiquidity(tokenHalf,bnbNewBalance) (#1226)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
External calls sending eth:
- addLiquidity(tokenHalf,bnbNewBalance) (#1226)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
State variables written after the call(s):
- addLiquidity(tokenHalf,bnbNewBalance) (#1226)
- _allowances[owner][spender] = amount (#962)
Reentrancy in PoorQuackOT._doMarketing(uint256) (#1255-1266):
External calls:
- swapTokensForEth(amount,address(this)) (#1259)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _wMarketing.transfer(bnbNewBalance) (#1263)
State variables written after the call(s):
- _marketingAllTime += bnbNewBalance (#1265)
Reentrancy in PoorQuackOT._doMegaPump(uint256) (#1229-1240):
External calls:
- swapTokensForEth(amount,address(this)) (#1233)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _wMegaPump.transfer(bnbNewBalance) (#1237)
State variables written after the call(s):
- _megaPumpAllTime += bnbNewBalance (#1239)
Reentrancy in PoorQuackOT._doTokenomics() (#1191-1211):
External calls:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- _doMegaPump(megaPumpAmount) (#1207)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _doMegaPump(megaPumpAmount) (#1207)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
State variables written after the call(s):
- _doMegaPump(megaPumpAmount) (#1207)
- _allowances[owner][spender] = amount (#962)
Reentrancy in PoorQuackOT._doTokenomics() (#1191-1211):
External calls:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- _doMegaPump(megaPumpAmount) (#1207)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- _doBuyBack(buybacksAmount) (#1208)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _doMegaPump(megaPumpAmount) (#1207)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _doBuyBack(buybacksAmount) (#1208)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
State variables written after the call(s):
- _doBuyBack(buybacksAmount) (#1208)
- _allowances[owner][spender] = amount (#962)
Reentrancy in PoorQuackOT._doTokenomics() (#1191-1211):
External calls:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- _doMegaPump(megaPumpAmount) (#1207)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- _doBuyBack(buybacksAmount) (#1208)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- _doMarketing(marketingAmount) (#1209)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _doMegaPump(megaPumpAmount) (#1207)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _doBuyBack(buybacksAmount) (#1208)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _doMarketing(marketingAmount) (#1209)
- _wMarketing.transfer(bnbNewBalance) (#1263)
State variables written after the call(s):
- _doMarketing(marketingAmount) (#1209)
- _allowances[owner][spender] = amount (#962)
Reentrancy in PoorQuackOT._transfer(address,address,uint256) (#970-1029):
External calls:
- _doTokenomics() (#977)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _doTokenomics() (#977)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _wMarketing.transfer(bnbNewBalance) (#1263)
State variables written after the call(s):
- _buyMap[recipient] = block.timestamp (#1008)
- removeTax() (#981)
- _previousRefTax = _totalTax (#894)
- removeTax() (#981)
- _totalTax = 0 (#896)
- _totalTax = _tax24HR (#990)
- _totalTax = _tax72HR (#995)
- _totalTax = _currentTax (#999)
Reentrancy in PoorQuackOT._transfer(address,address,uint256) (#970-1029):
External calls:
- _doTokenomics() (#977)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _doTokenomics() (#977)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _wMarketing.transfer(bnbNewBalance) (#1263)
State variables written after the call(s):
- _transferFromExcluded(sender,recipient,amount) (#1015)
- _tFeeTotal = _tFeeTotal.add(newTFee) (#1083)
- _transferToExcluded(sender,recipient,amount) (#1017)
- _tFeeTotal = _tFeeTotal.add(newTFee) (#1083)
- _transferStandard(sender,recipient,amount) (#1019)
- _tFeeTotal = _tFeeTotal.add(newTFee) (#1083)
- _transferBothExcluded(sender,recipient,amount) (#1021)
- _tFeeTotal = _tFeeTotal.add(newTFee) (#1083)
- _transferStandard(sender,recipient,amount) (#1023)
- _tFeeTotal = _tFeeTotal.add(newTFee) (#1083)
- _totalTax = _currentTax (#1010)
- restoreTax() (#1027)
- _totalTax = _previousRefTax (#901)
Reentrancy in PoorQuackOT.constructor() (#800-813):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#805-806)
State variables written after the call(s):
- _rOwned[_msgSender()] = _rTotal (#811)
- uniswapV2Router = _uniswapV2Router (#809)
Reentrancy in PoorQuackOT.setRouterAddress(address) (#1320-1325):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_newPancakeRouter.factory()).createPair(address(this),_newPancakeRouter.WETH()) (#1323)
State variables written after the call(s):
- uniswapV2Router = _newPancakeRouter (#1324)
Reentrancy in PoorQuackOT.transferFrom(address,address,uint256) (#861-866):
External calls:
- _transfer(sender,recipient,amount) (#863)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _transfer(sender,recipient,amount) (#863)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _wMarketing.transfer(bnbNewBalance) (#1263)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#864)
- _allowances[owner][spender] = amount (#962)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in PoorQuackOT._doLiquidity(uint256) (#1213-1227):
External calls:
- swapTokensForEth(bnbHalf,address(this)) (#1222)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- addLiquidity(tokenHalf,bnbNewBalance) (#1226)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
External calls sending eth:
- addLiquidity(tokenHalf,bnbNewBalance) (#1226)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#963)
- addLiquidity(tokenHalf,bnbNewBalance) (#1226)
Reentrancy in PoorQuackOT._doTokenomics() (#1191-1211):
External calls:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- _doMegaPump(megaPumpAmount) (#1207)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _doMegaPump(megaPumpAmount) (#1207)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#963)
- _doMegaPump(megaPumpAmount) (#1207)
Reentrancy in PoorQuackOT._doTokenomics() (#1191-1211):
External calls:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- _doMegaPump(megaPumpAmount) (#1207)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- _doBuyBack(buybacksAmount) (#1208)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _doMegaPump(megaPumpAmount) (#1207)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _doBuyBack(buybacksAmount) (#1208)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#963)
- _doBuyBack(buybacksAmount) (#1208)
Reentrancy in PoorQuackOT._doTokenomics() (#1191-1211):
External calls:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- _doMegaPump(megaPumpAmount) (#1207)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- _doBuyBack(buybacksAmount) (#1208)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
- _doMarketing(marketingAmount) (#1209)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _doMegaPump(megaPumpAmount) (#1207)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _doBuyBack(buybacksAmount) (#1208)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _doMarketing(marketingAmount) (#1209)
- _wMarketing.transfer(bnbNewBalance) (#1263)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#963)
- _doMarketing(marketingAmount) (#1209)
Reentrancy in PoorQuackOT._transfer(address,address,uint256) (#970-1029):
External calls:
- _doTokenomics() (#977)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _doTokenomics() (#977)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _wMarketing.transfer(bnbNewBalance) (#1263)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (#1037)
- _transferStandard(sender,recipient,amount) (#1019)
- Transfer(sender,recipient,tTransferAmount) (#1037)
- _transferStandard(sender,recipient,amount) (#1023)
- Transfer(sender,recipient,tTransferAmount) (#1047)
- _transferToExcluded(sender,recipient,amount) (#1017)
- Transfer(sender,recipient,tTransferAmount) (#1057)
- _transferFromExcluded(sender,recipient,amount) (#1015)
- Transfer(sender,recipient,tTransferAmount) (#1068)
- _transferBothExcluded(sender,recipient,amount) (#1021)
Reentrancy in PoorQuackOT.constructor() (#800-813):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#805-806)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_tTotal) (#812)
Reentrancy in PoorQuackOT.transferFrom(address,address,uint256) (#861-866):
External calls:
- _transfer(sender,recipient,amount) (#863)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,tokenContract,block.timestamp) (#1284-1290)
External calls sending eth:
- _transfer(sender,recipient,amount) (#863)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _wMarketing.transfer(bnbNewBalance) (#1263)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#963)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#864)
Apply the check-effects-interactions pattern.

Additional information: link

PoorQuackOT._transfer(address,address,uint256) (#970-1029) uses timestamp for comparisons
Dangerous comparisons:
- _buyMap[sender] != 0 && (_buyMap[sender] + (86400) >= block.timestamp) (#989)
- _buyMap[sender] != 0 && (_buyMap[sender] + (259200) >= block.timestamp) && _buyMap[sender] != 0 && (_buyMap[sender] + (86400) < block.timestamp) (#993-994)
Avoid relying on block.timestamp.

Additional information: link

Address.isContract(address) (#243-252) uses assembly
- INLINE ASM (#250)
Address._verifyCallResult(bool,bytes,string) (#364-381) uses assembly
- INLINE ASM (#373-376)
Do not use evm assembly.

Additional information: link

PoorQuackOT._doTokenomics() (#1191-1211) compares to a boolean constant:
-amount >= _minTokensForLiquidity && _lockLiquiditiesEnabled == true && amount > 0 (#1195)
Remove the equality to the boolean constant.

Additional information: link

Different versions of Solidity is used:
- Version used: ['>=0.5.0', '>=0.6.0<0.8.0', '>=0.6.2', '>=0.6.2<0.8.0', '^0.6.2']
- >=0.5.0 (#4)
- >=0.5.0 (#23)
- >=0.6.2 (#76)
- >=0.6.2 (#174)
- >=0.6.2<0.8.0 (#220)
- >=0.6.0<0.8.0 (#388)
- >=0.6.0<0.8.0 (#550)
- >=0.6.0<0.8.0 (#630)
- >=0.6.0<0.8.0 (#657)
- ^0.6.2 (#727)
Use one Solidity version.

Additional information: link

Address._verifyCallResult(bool,bytes,string) (#364-381) is never used and should be removed
Address.functionCall(address,bytes) (#296-298) is never used and should be removed
Address.functionCall(address,bytes,string) (#306-308) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#321-323) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#331-338) is never used and should be removed
Address.functionStaticCall(address,bytes) (#346-348) is never used and should be removed
Address.functionStaticCall(address,bytes,string) (#356-362) is never used and should be removed
Address.isContract(address) (#243-252) is never used and should be removed
Address.sendValue(address,uint256) (#270-276) is never used and should be removed
Context._msgData() (#647-650) is never used and should be removed
SafeMath.mod(uint256,uint256) (#524-526) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#540-543) is never used and should be removed
Remove unused functions.

Additional information: link

PoorQuackOT._rTotal (#749) is set pre-construction with a non-constant function or state variable:
- (MAX - (MAX % _tTotal))
PoorQuackOT._previousRefTax (#762) is set pre-construction with a non-constant function or state variable:
- _totalTax
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.5.0 (#4) allows old versions
Pragma version>=0.5.0 (#23) allows old versions
Pragma version>=0.6.2 (#76) allows old versions
Pragma version>=0.6.2 (#174) allows old versions
Pragma version>=0.6.2<0.8.0 (#220) is too complex
Pragma version>=0.6.0<0.8.0 (#388) is too complex
Pragma version>=0.6.0<0.8.0 (#550) is too complex
Pragma version>=0.6.0<0.8.0 (#630) is too complex
Pragma version>=0.6.0<0.8.0 (#657) is too complex
Pragma version^0.6.2 (#727) 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

Low level call in Address.sendValue(address,uint256) (#270-276):
- (success) = recipient.call{value: amount}() (#274)
Low level call in Address.functionCallWithValue(address,bytes,uint256,string) (#331-338):
- (success,returndata) = target.call{value: value}(data) (#336)
Low level call in Address.functionStaticCall(address,bytes,string) (#356-362):
- (success,returndata) = target.staticcall(data) (#360)
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() (#40) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#41) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#58) is not in mixedCase
Function IUniswapV2Router01.WETH() (#80) is not in mixedCase
Function PoorQuackOT._setTotalTax(uint256) (#1135-1139) is not in mixedCase
Function PoorQuackOT._set24Tax(uint256) (#1141-1144) is not in mixedCase
Function PoorQuackOT._set72Tax(uint256) (#1146-1149) is not in mixedCase
Function PoorQuackOT._setRefPer(uint256) (#1151-1154) is not in mixedCase
Function PoorQuackOT._setMegaPumpPer(uint256) (#1156-1159) is not in mixedCase
Function PoorQuackOT._setAutoLiqPer(uint256) (#1161-1164) is not in mixedCase
Function PoorQuackOT._setBuyBacksPer(uint256) (#1166-1169) is not in mixedCase
Function PoorQuackOT._setMarketingPer(uint256) (#1171-1174) is not in mixedCase
Function PoorQuackOT._setMinTokensForLiquidity(uint256) (#1176-1179) is not in mixedCase
Function PoorQuackOT._setLockLiquiditiesEnabled(bool) (#1181-1184) is not in mixedCase
Function PoorQuackOT._setAutoTaxEnabled(bool) (#1186-1189) is not in mixedCase
Parameter PoorQuackOT._setAutoTaxEnabled(bool).AutoTaxEnabled (#1186) is not in mixedCase
Function PoorQuackOT._doTokenomics() (#1191-1211) is not in mixedCase
Variable PoorQuackOT._rOwned (#739) is not in mixedCase
Variable PoorQuackOT._tOwned (#740) is not in mixedCase
Variable PoorQuackOT._wMegaPump (#756) is not in mixedCase
Variable PoorQuackOT._wBuyBack (#757) is not in mixedCase
Variable PoorQuackOT._wMarketing (#758) is not in mixedCase
Variable PoorQuackOT._currentTax (#760) is not in mixedCase
Variable PoorQuackOT._totalTax (#761) is not in mixedCase
Variable PoorQuackOT._tax24HR (#765) is not in mixedCase
Variable PoorQuackOT._tax72HR (#766) is not in mixedCase
Variable PoorQuackOT._refPer (#769) is not in mixedCase
Variable PoorQuackOT._megaPumpPer (#773) is not in mixedCase
Variable PoorQuackOT._autoLiqPer (#774) is not in mixedCase
Variable PoorQuackOT._buyBacksPer (#775) is not in mixedCase
Variable PoorQuackOT._marketingPer (#776) is not in mixedCase
Variable PoorQuackOT._liqAllTime (#778) is not in mixedCase
Variable PoorQuackOT._megaPumpAllTime (#779) is not in mixedCase
Variable PoorQuackOT._buyBacksAllTime (#780) is not in mixedCase
Variable PoorQuackOT._marketingAllTime (#781) is not in mixedCase
Variable PoorQuackOT._maxHoldAmount (#783) is not in mixedCase
Variable PoorQuackOT._maxTransAmount (#784) is not in mixedCase
Variable PoorQuackOT._minTokensForLiquidity (#785) is not in mixedCase
Variable PoorQuackOT._AutoTaxEnabled (#787) is not in mixedCase
Variable PoorQuackOT._lockLiquiditiesEnabled (#788) is not in mixedCase
Variable PoorQuackOT._inLockLiquidities (#789) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#648)" inContext (#642-651)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Reentrancy in PoorQuackOT._doBuyBack(uint256) (#1242-1253):
External calls:
- _wBuyBack.transfer(bnbNewBalance) (#1250)
State variables written after the call(s):
- _buyBacksAllTime += bnbNewBalance (#1252)
Reentrancy in PoorQuackOT._doMarketing(uint256) (#1255-1266):
External calls:
- _wMarketing.transfer(bnbNewBalance) (#1263)
State variables written after the call(s):
- _marketingAllTime += bnbNewBalance (#1265)
Reentrancy in PoorQuackOT._doMegaPump(uint256) (#1229-1240):
External calls:
- _wMegaPump.transfer(bnbNewBalance) (#1237)
State variables written after the call(s):
- _megaPumpAllTime += bnbNewBalance (#1239)
Reentrancy in PoorQuackOT._doTokenomics() (#1191-1211):
External calls:
- _doMegaPump(megaPumpAmount) (#1207)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _doBuyBack(buybacksAmount) (#1208)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
External calls sending eth:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _doMegaPump(megaPumpAmount) (#1207)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _doBuyBack(buybacksAmount) (#1208)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
State variables written after the call(s):
- _doBuyBack(buybacksAmount) (#1208)
- _allowances[owner][spender] = amount (#962)
- _doBuyBack(buybacksAmount) (#1208)
- _buyBacksAllTime += bnbNewBalance (#1252)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#963)
- _doBuyBack(buybacksAmount) (#1208)
Reentrancy in PoorQuackOT._doTokenomics() (#1191-1211):
External calls:
- _doMegaPump(megaPumpAmount) (#1207)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _doBuyBack(buybacksAmount) (#1208)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _doMarketing(marketingAmount) (#1209)
- _wMarketing.transfer(bnbNewBalance) (#1263)
External calls sending eth:
- _doLiquidity(liqAmount) (#1206)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _doMegaPump(megaPumpAmount) (#1207)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _doBuyBack(buybacksAmount) (#1208)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _doMarketing(marketingAmount) (#1209)
- _wMarketing.transfer(bnbNewBalance) (#1263)
State variables written after the call(s):
- _doMarketing(marketingAmount) (#1209)
- _allowances[owner][spender] = amount (#962)
- _doMarketing(marketingAmount) (#1209)
- _marketingAllTime += bnbNewBalance (#1265)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#963)
- _doMarketing(marketingAmount) (#1209)
Reentrancy in PoorQuackOT._transfer(address,address,uint256) (#970-1029):
External calls:
- _doTokenomics() (#977)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _wMarketing.transfer(bnbNewBalance) (#1263)
External calls sending eth:
- _doTokenomics() (#977)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _wMarketing.transfer(bnbNewBalance) (#1263)
State variables written after the call(s):
- _buyMap[recipient] = block.timestamp (#1008)
- removeTax() (#981)
- _previousRefTax = _totalTax (#894)
- removeTax() (#981)
- _totalTax = 0 (#896)
- _totalTax = _tax24HR (#990)
- _totalTax = _tax72HR (#995)
- _totalTax = _currentTax (#999)
Reentrancy in PoorQuackOT._transfer(address,address,uint256) (#970-1029):
External calls:
- _doTokenomics() (#977)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _wMarketing.transfer(bnbNewBalance) (#1263)
External calls sending eth:
- _doTokenomics() (#977)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _wMarketing.transfer(bnbNewBalance) (#1263)
State variables written after the call(s):
- _transferFromExcluded(sender,recipient,amount) (#1015)
- _rOwned[address(this)] = _rOwned[address(this)].add(rContractFee) (#1080)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1054)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1055)
- _transferToExcluded(sender,recipient,amount) (#1017)
- _rOwned[address(this)] = _rOwned[address(this)].add(rContractFee) (#1080)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1043)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1045)
- _transferStandard(sender,recipient,amount) (#1019)
- _rOwned[address(this)] = _rOwned[address(this)].add(rContractFee) (#1080)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1034)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1035)
- _transferBothExcluded(sender,recipient,amount) (#1021)
- _rOwned[address(this)] = _rOwned[address(this)].add(rContractFee) (#1080)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1064)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1066)
- _transferStandard(sender,recipient,amount) (#1023)
- _rOwned[address(this)] = _rOwned[address(this)].add(rContractFee) (#1080)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1034)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1035)
- _transferFromExcluded(sender,recipient,amount) (#1015)
- _rTotal = _rTotal.sub(newRFee) (#1082)
- _transferToExcluded(sender,recipient,amount) (#1017)
- _rTotal = _rTotal.sub(newRFee) (#1082)
- _transferStandard(sender,recipient,amount) (#1019)
- _rTotal = _rTotal.sub(newRFee) (#1082)
- _transferBothExcluded(sender,recipient,amount) (#1021)
- _rTotal = _rTotal.sub(newRFee) (#1082)
- _transferStandard(sender,recipient,amount) (#1023)
- _rTotal = _rTotal.sub(newRFee) (#1082)
- _transferFromExcluded(sender,recipient,amount) (#1015)
- _tFeeTotal = _tFeeTotal.add(newTFee) (#1083)
- _transferToExcluded(sender,recipient,amount) (#1017)
- _tFeeTotal = _tFeeTotal.add(newTFee) (#1083)
- _transferStandard(sender,recipient,amount) (#1019)
- _tFeeTotal = _tFeeTotal.add(newTFee) (#1083)
- _transferBothExcluded(sender,recipient,amount) (#1021)
- _tFeeTotal = _tFeeTotal.add(newTFee) (#1083)
- _transferStandard(sender,recipient,amount) (#1023)
- _tFeeTotal = _tFeeTotal.add(newTFee) (#1083)
- _transferFromExcluded(sender,recipient,amount) (#1015)
- _tOwned[address(this)] = _tOwned[address(this)].add(tContractFee) (#1079)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1053)
- _transferToExcluded(sender,recipient,amount) (#1017)
- _tOwned[address(this)] = _tOwned[address(this)].add(tContractFee) (#1079)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1044)
- _transferStandard(sender,recipient,amount) (#1019)
- _tOwned[address(this)] = _tOwned[address(this)].add(tContractFee) (#1079)
- _transferBothExcluded(sender,recipient,amount) (#1021)
- _tOwned[address(this)] = _tOwned[address(this)].add(tContractFee) (#1079)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1063)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1065)
- _transferStandard(sender,recipient,amount) (#1023)
- _tOwned[address(this)] = _tOwned[address(this)].add(tContractFee) (#1079)
- _totalTax = _currentTax (#1010)
- restoreTax() (#1027)
- _totalTax = _previousRefTax (#901)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (#1037)
- _transferStandard(sender,recipient,amount) (#1023)
- Transfer(sender,recipient,tTransferAmount) (#1037)
- _transferStandard(sender,recipient,amount) (#1019)
- Transfer(sender,recipient,tTransferAmount) (#1057)
- _transferFromExcluded(sender,recipient,amount) (#1015)
- Transfer(sender,recipient,tTransferAmount) (#1047)
- _transferToExcluded(sender,recipient,amount) (#1017)
- Transfer(sender,recipient,tTransferAmount) (#1068)
- _transferBothExcluded(sender,recipient,amount) (#1021)
Reentrancy in PoorQuackOT.transferFrom(address,address,uint256) (#861-866):
External calls:
- _transfer(sender,recipient,amount) (#863)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _wMarketing.transfer(bnbNewBalance) (#1263)
External calls sending eth:
- _transfer(sender,recipient,amount) (#863)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1299-1306)
- _wMegaPump.transfer(bnbNewBalance) (#1237)
- _wBuyBack.transfer(bnbNewBalance) (#1250)
- _wMarketing.transfer(bnbNewBalance) (#1263)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#864)
- _allowances[owner][spender] = amount (#962)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#963)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#864)
Apply the check-effects-interactions pattern.

Additional information: link

Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#85) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#86)
Variable PoorQuackOT._transferStandard(address,address,uint256).rTransferAmount (#1033) is too similar to PoorQuackOT._transferBothExcluded(address,address,uint256).tTransferAmount (#1062)
Variable PoorQuackOT._transferStandard(address,address,uint256).rTransferAmount (#1033) is too similar to PoorQuackOT._getTValues(uint256).tTransferAmount (#1097)
Variable PoorQuackOT.reflectionFromToken(uint256,bool).rTransferAmount (#921) is too similar to PoorQuackOT._getValues(uint256).tTransferAmount (#1088)
Variable PoorQuackOT._transferFromExcluded(address,address,uint256).rTransferAmount (#1052) is too similar to PoorQuackOT._transferToExcluded(address,address,uint256).tTransferAmount (#1042)
Variable PoorQuackOT._transferToExcluded(address,address,uint256).rTransferAmount (#1042) is too similar to PoorQuackOT._transferToExcluded(address,address,uint256).tTransferAmount (#1042)
Variable PoorQuackOT._transferFromExcluded(address,address,uint256).rTransferAmount (#1052) is too similar to PoorQuackOT._transferBothExcluded(address,address,uint256).tTransferAmount (#1062)
Variable PoorQuackOT._transferToExcluded(address,address,uint256).rTransferAmount (#1042) is too similar to PoorQuackOT._transferBothExcluded(address,address,uint256).tTransferAmount (#1062)
Variable PoorQuackOT._transferStandard(address,address,uint256).rTransferAmount (#1033) is too similar to PoorQuackOT._transferFromExcluded(address,address,uint256).tTransferAmount (#1052)
Variable PoorQuackOT._getRValues(uint256,uint256,uint256).rTransferAmount (#1105) is too similar to PoorQuackOT._transferBothExcluded(address,address,uint256).tTransferAmount (#1062)
Variable PoorQuackOT._transferStandard(address,address,uint256).rTransferAmount (#1033) is too similar to PoorQuackOT._getValues(uint256).tTransferAmount (#1088)
Variable PoorQuackOT._transferFromExcluded(address,address,uint256).rTransferAmount (#1052) is too similar to PoorQuackOT._transferFromExcluded(address,address,uint256).tTransferAmount (#1052)
Variable PoorQuackOT.reflectionFromToken(uint256,bool).rTransferAmount (#921) is too similar to PoorQuackOT._transferBothExcluded(address,address,uint256).tTransferAmount (#1062)
Variable PoorQuackOT._transferStandard(address,address,uint256).rTransferAmount (#1033) is too similar to PoorQuackOT._transferStandard(address,address,uint256).tTransferAmount (#1033)
Variable PoorQuackOT._transferFromExcluded(address,address,uint256).rTransferAmount (#1052) is too similar to PoorQuackOT._getValues(uint256).tTransferAmount (#1088)
Variable PoorQuackOT._transferToExcluded(address,address,uint256).rTransferAmount (#1042) is too similar to PoorQuackOT._getValues(uint256).tTransferAmount (#1088)
Variable PoorQuackOT._getValues(uint256).rTransferAmount (#1090) is too similar to PoorQuackOT._getValues(uint256).tTransferAmount (#1088)
Variable PoorQuackOT._getRValues(uint256,uint256,uint256).rTransferAmount (#1105) is too similar to PoorQuackOT._getValues(uint256).tTransferAmount (#1088)
Variable PoorQuackOT._transferStandard(address,address,uint256).rTransferAmount (#1033) is too similar to PoorQuackOT._transferToExcluded(address,address,uint256).tTransferAmount (#1042)
Variable PoorQuackOT._reflectFee(uint256,uint256).rContractFee (#1077) is too similar to PoorQuackOT._reflectFee(uint256,uint256).tContractFee (#1076)
Variable PoorQuackOT._transferBothExcluded(address,address,uint256).rTransferAmount (#1062) is too similar to PoorQuackOT._getValues(uint256).tTransferAmount (#1088)
Variable PoorQuackOT._transferFromExcluded(address,address,uint256).rTransferAmount (#1052) is too similar to PoorQuackOT._getTValues(uint256).tTransferAmount (#1097)
Variable PoorQuackOT._getValues(uint256).rTransferAmount (#1090) is too similar to PoorQuackOT._transferFromExcluded(address,address,uint256).tTransferAmount (#1052)
Variable PoorQuackOT._getValues(uint256).rTransferAmount (#1090) is too similar to PoorQuackOT._transferToExcluded(address,address,uint256).tTransferAmount (#1042)
Variable PoorQuackOT.reflectionFromToken(uint256,bool).rTransferAmount (#921) is too similar to PoorQuackOT._transferFromExcluded(address,address,uint256).tTransferAmount (#1052)
Variable PoorQuackOT.reflectionFromToken(uint256,bool).rTransferAmount (#921) is too similar to PoorQuackOT._transferToExcluded(address,address,uint256).tTransferAmount (#1042)
Variable PoorQuackOT._transferToExcluded(address,address,uint256).rTransferAmount (#1042) is too similar to PoorQuackOT._transferStandard(address,address,uint256).tTransferAmount (#1033)
Variable PoorQuackOT._getValues(uint256).rTransferAmount (#1090) is too similar to PoorQuackOT._transferBothExcluded(address,address,uint256).tTransferAmount (#1062)
Variable PoorQuackOT._transferBothExcluded(address,address,uint256).rTransferAmount (#1062) is too similar to PoorQuackOT._transferStandard(address,address,uint256).tTransferAmount (#1033)
Variable PoorQuackOT._getRValues(uint256,uint256,uint256).rTransferAmount (#1105) is too similar to PoorQuackOT._transferStandard(address,address,uint256).tTransferAmount (#1033)
Variable PoorQuackOT._getValues(uint256).rTransferAmount (#1090) is too similar to PoorQuackOT._getTValues(uint256).tTransferAmount (#1097)
Variable PoorQuackOT._transferToExcluded(address,address,uint256).rTransferAmount (#1042) is too similar to PoorQuackOT._transferFromExcluded(address,address,uint256).tTransferAmount (#1052)
Variable PoorQuackOT.reflectionFromToken(uint256,bool).rTransferAmount (#921) is too similar to PoorQuackOT._getTValues(uint256).tTransferAmount (#1097)
Variable PoorQuackOT._transferBothExcluded(address,address,uint256).rTransferAmount (#1062) is too similar to PoorQuackOT._transferFromExcluded(address,address,uint256).tTransferAmount (#1052)
Variable PoorQuackOT._transferBothExcluded(address,address,uint256).rTransferAmount (#1062) is too similar to PoorQuackOT._transferToExcluded(address,address,uint256).tTransferAmount (#1042)
Variable PoorQuackOT._transferFromExcluded(address,address,uint256).rTransferAmount (#1052) is too similar to PoorQuackOT._transferStandard(address,address,uint256).tTransferAmount (#1033)
Variable PoorQuackOT._getRValues(uint256,uint256,uint256).rTransferAmount (#1105) is too similar to PoorQuackOT._transferFromExcluded(address,address,uint256).tTransferAmount (#1052)
Variable PoorQuackOT._getRValues(uint256,uint256,uint256).rTransferAmount (#1105) is too similar to PoorQuackOT._transferToExcluded(address,address,uint256).tTransferAmount (#1042)
Variable PoorQuackOT._transferBothExcluded(address,address,uint256).rTransferAmount (#1062) is too similar to PoorQuackOT._transferBothExcluded(address,address,uint256).tTransferAmount (#1062)
Variable PoorQuackOT._transferToExcluded(address,address,uint256).rTransferAmount (#1042) is too similar to PoorQuackOT._getTValues(uint256).tTransferAmount (#1097)
Variable PoorQuackOT._transferBothExcluded(address,address,uint256).rTransferAmount (#1062) is too similar to PoorQuackOT._getTValues(uint256).tTransferAmount (#1097)
Variable PoorQuackOT._getValues(uint256).rTransferAmount (#1090) is too similar to PoorQuackOT._transferStandard(address,address,uint256).tTransferAmount (#1033)
Variable PoorQuackOT._getRValues(uint256,uint256,uint256).rTransferAmount (#1105) is too similar to PoorQuackOT._getTValues(uint256).tTransferAmount (#1097)
Variable PoorQuackOT.reflectionFromToken(uint256,bool).rTransferAmount (#921) is too similar to PoorQuackOT._transferStandard(address,address,uint256).tTransferAmount (#1033)
Prevent variables from having similar names.

Additional information: link

PoorQuackOT.slitherConstructorVariables() (#735-1342) uses literals with too many digits:
- _tTotal = 100000000000000000000000000000000 (#748)
PoorQuackOT.slitherConstructorVariables() (#735-1342) uses literals with too many digits:
- _maxHoldAmount = 1000000000000000000000000000000 (#783)
PoorQuackOT.slitherConstructorVariables() (#735-1342) uses literals with too many digits:
- _maxTransAmount = 1000000000000000000000000000000 (#784)
PoorQuackOT.slitherConstructorVariables() (#735-1342) uses literals with too many digits:
- _minTokensForLiquidity = 100000000000000000000000000000 (#785)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#707-710)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#716-720)
name() should be declared external:
- PoorQuackOT.name() (#818-821)
symbol() should be declared external:
- PoorQuackOT.symbol() (#823-826)
decimals() should be declared external:
- PoorQuackOT.decimals() (#828-831)
totalSupply() should be declared external:
- PoorQuackOT.totalSupply() (#833-836)
transfer(address,uint256) should be declared external:
- PoorQuackOT.transfer(address,uint256) (#844-848)
allowance(address,address) should be declared external:
- PoorQuackOT.allowance(address,address) (#850-853)
approve(address,uint256) should be declared external:
- PoorQuackOT.approve(address,uint256) (#855-859)
transferFrom(address,address,uint256) should be declared external:
- PoorQuackOT.transferFrom(address,address,uint256) (#861-866)
increaseAllowance(address,uint256) should be declared external:
- PoorQuackOT.increaseAllowance(address,uint256) (#868-872)
decreaseAllowance(address,uint256) should be declared external:
- PoorQuackOT.decreaseAllowance(address,uint256) (#874-878)
isExcluded(address) should be declared external:
- PoorQuackOT.isExcluded(address) (#880-883)
totalFees() should be declared external:
- PoorQuackOT.totalFees() (#885-888)
reflect(uint256) should be declared external:
- PoorQuackOT.reflect(uint256) (#904-912)
reflectionFromToken(uint256,bool) should be declared external:
- PoorQuackOT.reflectionFromToken(uint256,bool) (#914-924)
marketingFallBack() should be declared external:
- PoorQuackOT.marketingFallBack() (#1309-1318)
setRouterAddress(address) should be declared external:
- PoorQuackOT.setRouterAddress(address) (#1320-1325)
setMegaPumpWallet(address) should be declared external:
- PoorQuackOT.setMegaPumpWallet(address) (#1327-1330)
setBuyBackWallet(address) should be declared external:
- PoorQuackOT.setBuyBackWallet(address) (#1332-1335)
setMarketingWallet(address) should be declared external:
- PoorQuackOT.setMarketingWallet(address) (#1337-1340)
Use the external attribute for functions never called from the contract.

Additional information: link

PoorQuackOT.includeAccount(address) (#943-955) has costly operations inside a loop:
- _excluded.pop() (#951)
Use a local variable to hold the loop computation result.

Additional information: link

Contract has 12% buy tax and 12% sell tax.
Taxes are suspiciously high (over 10%) and contract ownership is not renounced. Token has a high risk of becoming a honeypot.

Swap operations require suspiciously high gas. Contract logic is complex and may disguise some form of scam.

Unable to find token on CoinHunt

Additional information: link

Token is not listed at Mobula.Finance

Additional information: link

Young tokens have high risks of price dump / death

Young tokens have high risks of price dump / death

Young tokens have high risks of price dump / death

Young tokens have high risks of price dump / death

Young tokens have high risks of price dump / death

Token has relatively low CoinGecko rank