Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
PirateKing.swapETHForTokens(uint256) (#782-797) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#789-794)
PirateKing.addLiquidity(uint256,uint256) (#799-812) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in PirateKing._transfer(address,address,uint256) (#659-735):
External calls:
- swapAndLiquify(contractTokenBalance) (#682)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#771-777)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#682)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#734)
- _rOwned[addressM] = _rOwned[addressM].add(rLiquidity.div(2)) (#919)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity.div(8)) (#920)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#834)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#843)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#854)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#864)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#835)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#845)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#855)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#866)
- _tokenTransfer(from,to,amount,takeFee) (#734)
- _rTotal = _rTotal.sub(rFee) (#873)
- _startTimeForSwap = block.timestamp (#683)
- _tokenTransfer(from,to,amount,takeFee) (#734)
- _tOwned[addressM] = _tOwned[addressM].add(tLiquidity.div(2)) (#922)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#863)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity.div(8)) (#923)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#853)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#844)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#865)
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.
PirateKing._rTotal (#432) is set pre-construction with a non-constant function or state variable:
- (MAX - (MAX % _tTotal))
PirateKing._previousTaxFee (#455) is set pre-construction with a non-constant function or state variable:
- _taxFee
PirateKing._previousLiquidityFee (#458) is set pre-construction with a non-constant function or state variable:
- _liquidityFee
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) (#98-104):
- (success) = recipient.call{value: amount}() (#102)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#124-141):
- (success,returndata) = target.call{value: weiValue}(data) (#127)
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() (#237) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#238) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#254) is not in mixedCase
Function IUniswapV2Router01.WETH() (#275) is not in mixedCase
Parameter PirateKing.calculateTaxFee(uint256)._amount (#926) is not in mixedCase
Parameter PirateKing.calculateLiquidityFee(uint256)._amount (#932) is not in mixedCase
Parameter PirateKing.setSwapAndLiquifyEnabled(bool)._enabled (#1006) is not in mixedCase
Parameter PirateKing.changeRouterVersion(address)._router (#1026) is not in mixedCase
Parameter PirateKing.transferForeignToken(address,address)._token (#1045) is not in mixedCase
Parameter PirateKing.transferForeignToken(address,address)._to (#1045) is not in mixedCase
Function PirateKing.Sweep() (#1051-1054) is not in mixedCase
Parameter PirateKing.setAddressFee(address,bool,uint256,uint256)._address (#1056) is not in mixedCase
Parameter PirateKing.setAddressFee(address,bool,uint256,uint256)._enable (#1056) is not in mixedCase
Parameter PirateKing.setAddressFee(address,bool,uint256,uint256)._addressTaxFee (#1056) is not in mixedCase
Parameter PirateKing.setAddressFee(address,bool,uint256,uint256)._addressLiquidityFee (#1056) is not in mixedCase
Parameter PirateKing.setBuyAddressFee(address,bool,uint256,uint256)._address (#1062) is not in mixedCase
Parameter PirateKing.setBuyAddressFee(address,bool,uint256,uint256)._enable (#1062) is not in mixedCase
Parameter PirateKing.setBuyAddressFee(address,bool,uint256,uint256)._addressTaxFee (#1062) is not in mixedCase
Parameter PirateKing.setBuyAddressFee(address,bool,uint256,uint256)._addressLiquidityFee (#1062) is not in mixedCase
Parameter PirateKing.setSellAddressFee(address,bool,uint256,uint256)._address (#1068) is not in mixedCase
Parameter PirateKing.setSellAddressFee(address,bool,uint256,uint256)._enable (#1068) is not in mixedCase
Parameter PirateKing.setSellAddressFee(address,bool,uint256,uint256)._addressTaxFee (#1068) is not in mixedCase
Parameter PirateKing.setSellAddressFee(address,bool,uint256,uint256)._addressLiquidityFee (#1068) is not in mixedCase
Variable PirateKing._taxFee (#454) is not in mixedCase
Variable PirateKing._liquidityFee (#457) is not in mixedCase
Variable PirateKing._buyTaxFee (#460) is not in mixedCase
Variable PirateKing._buyLiquidityFee (#461) is not in mixedCase
Variable PirateKing._sellTaxFee (#463) is not in mixedCase
Variable PirateKing._sellLiquidityFee (#464) is not in mixedCase
Variable PirateKing._startTimeForSwap (#468) is not in mixedCase
Variable PirateKing._intervalMinutesForSwap (#469) is not in mixedCase
Variable PirateKing._addressFees (#474) is not in mixedCase
Variable PirateKing._maxTxAmount (#476) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#11)" inContext (#5-14)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#280) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#281)
Variable PirateKing._transferStandard(address,address,uint256).rTransferAmount (#833) is too similar to PirateKing._getTValues(uint256).tTransferAmount (#886)
Variable PirateKing._transferFromExcluded(address,address,uint256).rTransferAmount (#852) is too similar to PirateKing._transferBothExcluded(address,address,uint256).tTransferAmount (#862)
Variable PirateKing._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#894) is too similar to PirateKing._transferBothExcluded(address,address,uint256).tTransferAmount (#862)
Variable PirateKing.reflectionFromToken(uint256,bool).rTransferAmount (#617) is too similar to PirateKing._transferFromExcluded(address,address,uint256).tTransferAmount (#852)
Variable PirateKing._getValues(uint256).rTransferAmount (#879) is too similar to PirateKing._transferFromExcluded(address,address,uint256).tTransferAmount (#852)
Variable PirateKing._transferBothExcluded(address,address,uint256).rTransferAmount (#862) is too similar to PirateKing._transferFromExcluded(address,address,uint256).tTransferAmount (#852)
Variable PirateKing._transferStandard(address,address,uint256).rTransferAmount (#833) is too similar to PirateKing._transferBothExcluded(address,address,uint256).tTransferAmount (#862)
Variable PirateKing._transferFromExcluded(address,address,uint256).rTransferAmount (#852) is too similar to PirateKing._transferFromExcluded(address,address,uint256).tTransferAmount (#852)
Variable PirateKing.reflectionFromToken(uint256,bool).rTransferAmount (#617) is too similar to PirateKing._transferBothExcluded(address,address,uint256).tTransferAmount (#862)
Variable PirateKing._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#894) is too similar to PirateKing._transferFromExcluded(address,address,uint256).tTransferAmount (#852)
Variable PirateKing._getValues(uint256).rTransferAmount (#879) is too similar to PirateKing._transferBothExcluded(address,address,uint256).tTransferAmount (#862)
Variable PirateKing._transferToExcluded(address,address,uint256).rTransferAmount (#842) is too similar to PirateKing._transferFromExcluded(address,address,uint256).tTransferAmount (#852)
Variable PirateKing._transferBothExcluded(address,address,uint256).rTransferAmount (#862) is too similar to PirateKing._transferBothExcluded(address,address,uint256).tTransferAmount (#862)
Variable PirateKing._transferStandard(address,address,uint256).rTransferAmount (#833) is too similar to PirateKing._getValues(uint256).tTransferAmount (#878)
Variable PirateKing._transferStandard(address,address,uint256).rTransferAmount (#833) is too similar to PirateKing._transferStandard(address,address,uint256).tTransferAmount (#833)
Variable PirateKing._transferStandard(address,address,uint256).rTransferAmount (#833) is too similar to PirateKing._transferFromExcluded(address,address,uint256).tTransferAmount (#852)
Variable PirateKing._transferBothExcluded(address,address,uint256).rTransferAmount (#862) is too similar to PirateKing._getTValues(uint256).tTransferAmount (#886)
Variable PirateKing._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#894) is too similar to PirateKing._getValues(uint256).tTransferAmount (#878)
Variable PirateKing._getValues(uint256).rTransferAmount (#879) is too similar to PirateKing._getValues(uint256).tTransferAmount (#878)
Variable PirateKing._transferBothExcluded(address,address,uint256).rTransferAmount (#862) is too similar to PirateKing._getValues(uint256).tTransferAmount (#878)
Variable PirateKing._transferToExcluded(address,address,uint256).rTransferAmount (#842) is too similar to PirateKing._transferBothExcluded(address,address,uint256).tTransferAmount (#862)
Variable PirateKing._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#894) is too similar to PirateKing._transferStandard(address,address,uint256).tTransferAmount (#833)
Variable PirateKing._transferToExcluded(address,address,uint256).rTransferAmount (#842) is too similar to PirateKing._getTValues(uint256).tTransferAmount (#886)
Variable PirateKing._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#894) is too similar to PirateKing._transferToExcluded(address,address,uint256).tTransferAmount (#842)
Variable PirateKing._transferBothExcluded(address,address,uint256).rTransferAmount (#862) is too similar to PirateKing._transferToExcluded(address,address,uint256).tTransferAmount (#842)
Variable PirateKing._transferBothExcluded(address,address,uint256).rTransferAmount (#862) is too similar to PirateKing._transferStandard(address,address,uint256).tTransferAmount (#833)
Variable PirateKing._transferToExcluded(address,address,uint256).rTransferAmount (#842) is too similar to PirateKing._getValues(uint256).tTransferAmount (#878)
Variable PirateKing._getValues(uint256).rTransferAmount (#879) is too similar to PirateKing._transferToExcluded(address,address,uint256).tTransferAmount (#842)
Variable PirateKing._getValues(uint256).rTransferAmount (#879) is too similar to PirateKing._transferStandard(address,address,uint256).tTransferAmount (#833)
Variable PirateKing.reflectionFromToken(uint256,bool).rTransferAmount (#617) is too similar to PirateKing._getTValues(uint256).tTransferAmount (#886)
Variable PirateKing.reflectionFromToken(uint256,bool).rTransferAmount (#617) is too similar to PirateKing._getValues(uint256).tTransferAmount (#878)
Variable PirateKing._transferFromExcluded(address,address,uint256).rTransferAmount (#852) is too similar to PirateKing._getTValues(uint256).tTransferAmount (#886)
Variable PirateKing._transferFromExcluded(address,address,uint256).rTransferAmount (#852) is too similar to PirateKing._getValues(uint256).tTransferAmount (#878)
Variable PirateKing._transferToExcluded(address,address,uint256).rTransferAmount (#842) is too similar to PirateKing._transferToExcluded(address,address,uint256).tTransferAmount (#842)
Variable PirateKing._transferToExcluded(address,address,uint256).rTransferAmount (#842) is too similar to PirateKing._transferStandard(address,address,uint256).tTransferAmount (#833)
Variable PirateKing.reflectionFromToken(uint256,bool).rTransferAmount (#617) is too similar to PirateKing._transferStandard(address,address,uint256).tTransferAmount (#833)
Variable PirateKing.reflectionFromToken(uint256,bool).rTransferAmount (#617) is too similar to PirateKing._transferToExcluded(address,address,uint256).tTransferAmount (#842)
Variable PirateKing._transferStandard(address,address,uint256).rTransferAmount (#833) is too similar to PirateKing._transferToExcluded(address,address,uint256).tTransferAmount (#842)
Variable PirateKing._transferFromExcluded(address,address,uint256).rTransferAmount (#852) is too similar to PirateKing._transferToExcluded(address,address,uint256).tTransferAmount (#842)
Variable PirateKing._transferFromExcluded(address,address,uint256).rTransferAmount (#852) is too similar to PirateKing._transferStandard(address,address,uint256).tTransferAmount (#833)
Variable PirateKing._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#894) is too similar to PirateKing._getTValues(uint256).tTransferAmount (#886)
Variable PirateKing._getValues(uint256).rTransferAmount (#879) is too similar to PirateKing._getTValues(uint256).tTransferAmount (#886)
Prevent variables from having similar names.
Additional information: link
PirateKing.slitherConstructorVariables() (#410-1075) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (#415)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
PirateKing._decimals (#437) should be constant
PirateKing._intervalMinutesForSwap (#469) should be constant
PirateKing._name (#435) should be constant
PirateKing._symbol (#436) should be constant
PirateKing._tTotal (#431) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
PirateKing.addLiquidity(uint256,uint256) (#799-812) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
Ensure that all the return values of the function calls are used.
Additional information: link
PirateKing.allowance(address,address).owner (#567) shadows:
- Ownable.owner() (#157-159) (function)
PirateKing._approve(address,address,uint256).owner (#651) shadows:
- Ownable.owner() (#157-159) (function)
Rename the local variables that shadow another component.
Additional information: link
PirateKing.setTaxFeePercent(uint256) (#972-974) should emit an event for:
- _taxFee = taxFee (#973)
PirateKing.setBuyFee(uint256,uint256) (#976-979) should emit an event for:
- _buyTaxFee = buyTaxFee (#977)
- _buyLiquidityFee = buyLiquidityFee (#978)
PirateKing.setSellFee(uint256,uint256) (#981-984) should emit an event for:
- _sellTaxFee = sellTaxFee (#982)
- _sellLiquidityFee = sellLiquidityFee (#983)
PirateKing.setLiquidityFeePercent(uint256) (#986-988) should emit an event for:
- _liquidityFee = liquidityFee (#987)
PirateKing.setMaxTxAmount(uint256) (#991-993) should emit an event for:
- _maxTxAmount = maxTxAmount (#992)
PirateKing.setLiquidityDivisor(uint256) (#1000-1002) should emit an event for:
- liquidityDivisor = divisor (#1001)
Emit an event for critical parameter changes.
Additional information: link
PirateKing.constructor(address)._addressM (#510) lacks a zero-check on :
- addressM = _addressM (#526)
Check that the address is not zero.
Additional information: link
Reentrancy in PirateKing._transfer(address,address,uint256) (#659-735):
External calls:
- swapAndLiquify(contractTokenBalance) (#682)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#771-777)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#682)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
State variables written after the call(s):
- removeAllFee() (#698)
- _liquidityFee = 0 (#945)
- _liquidityFee = _buyLiquidityFee (#700)
- removeAllFee() (#704)
- _liquidityFee = 0 (#945)
- _liquidityFee = _sellLiquidityFee (#706)
- removeAllFee() (#711)
- _liquidityFee = 0 (#945)
- _liquidityFee = _addressFees[from]._liquidityFee (#713)
- _liquidityFee = _addressFees[from]._sellLiquidityFee (#718)
- removeAllFee() (#725)
- _liquidityFee = 0 (#945)
- _liquidityFee = _addressFees[to]._buyLiquidityFee (#728)
- _tokenTransfer(from,to,amount,takeFee) (#734)
- _liquidityFee = _previousLiquidityFee (#950)
- _liquidityFee = 0 (#945)
- removeAllFee() (#698)
- _previousLiquidityFee = _liquidityFee (#942)
- removeAllFee() (#704)
- _previousLiquidityFee = _liquidityFee (#942)
- removeAllFee() (#711)
- _previousLiquidityFee = _liquidityFee (#942)
- removeAllFee() (#725)
- _previousLiquidityFee = _liquidityFee (#942)
- _tokenTransfer(from,to,amount,takeFee) (#734)
- _previousLiquidityFee = _liquidityFee (#942)
- removeAllFee() (#698)
- _previousTaxFee = _taxFee (#941)
- removeAllFee() (#704)
- _previousTaxFee = _taxFee (#941)
- removeAllFee() (#711)
- _previousTaxFee = _taxFee (#941)
- removeAllFee() (#725)
- _previousTaxFee = _taxFee (#941)
- _tokenTransfer(from,to,amount,takeFee) (#734)
- _previousTaxFee = _taxFee (#941)
- _tokenTransfer(from,to,amount,takeFee) (#734)
- _tFeeTotal = _tFeeTotal.add(tFee) (#874)
- removeAllFee() (#698)
- _taxFee = 0 (#944)
- _taxFee = _buyTaxFee (#699)
- removeAllFee() (#704)
- _taxFee = 0 (#944)
- _taxFee = _sellTaxFee (#705)
- removeAllFee() (#711)
- _taxFee = 0 (#944)
- _taxFee = _addressFees[from]._taxFee (#712)
- _taxFee = _addressFees[from]._sellTaxFee (#717)
- removeAllFee() (#725)
- _taxFee = 0 (#944)
- _taxFee = _addressFees[to]._buyTaxFee (#727)
- _tokenTransfer(from,to,amount,takeFee) (#734)
- _taxFee = _previousTaxFee (#949)
- _taxFee = 0 (#944)
- swapAndLiquifycount = swapAndLiquifycount.add(1) (#684)
Reentrancy in PirateKing.changeRouterVersion(address) (#1026-1039):
External calls:
- _pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1032-1033)
State variables written after the call(s):
- uniswapV2Pair = _pair (#1035)
- uniswapV2Router = _uniswapV2Router (#1038)
Reentrancy in PirateKing.constructor(address) (#510-539):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#521-522)
State variables written after the call(s):
- _isExcludedFromFee[addressM] = true (#527)
- _isExcludedFromFee[owner()] = true (#530)
- _isExcludedFromFee[address(this)] = true (#531)
- _startTimeForSwap = block.timestamp (#536)
- addressM = _addressM (#526)
- isTxLimitExempt[addressM] = true (#528)
- isTxLimitExempt[owner()] = true (#532)
- isTxLimitExempt[address(this)] = true (#533)
- uniswapV2Router = _uniswapV2Router (#524)
Reentrancy in PirateKing.swapAndLiquify(uint256) (#737-758):
External calls:
- swapTokensForEth(half) (#749)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#771-777)
- addLiquidity(otherHalf,newBalance) (#755)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#755)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (#755)
- _allowances[owner][spender] = amount (#655)
Reentrancy in PirateKing.transferFrom(address,address,uint256) (#576-580):
External calls:
- _transfer(sender,recipient,amount) (#577)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#771-777)
External calls sending eth:
- _transfer(sender,recipient,amount) (#577)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#578)
- _allowances[owner][spender] = amount (#655)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in PirateKing._transfer(address,address,uint256) (#659-735):
External calls:
- swapAndLiquify(contractTokenBalance) (#682)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#771-777)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#682)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (#838)
- _tokenTransfer(from,to,amount,takeFee) (#734)
- Transfer(sender,recipient,tTransferAmount) (#858)
- _tokenTransfer(from,to,amount,takeFee) (#734)
- Transfer(sender,recipient,tTransferAmount) (#848)
- _tokenTransfer(from,to,amount,takeFee) (#734)
- Transfer(sender,recipient,tTransferAmount) (#869)
- _tokenTransfer(from,to,amount,takeFee) (#734)
Reentrancy in PirateKing.constructor(address) (#510-539):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#521-522)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_tTotal) (#538)
Reentrancy in PirateKing.swapAndLiquify(uint256) (#737-758):
External calls:
- swapTokensForEth(half) (#749)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#771-777)
- addLiquidity(otherHalf,newBalance) (#755)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#755)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#656)
- addLiquidity(otherHalf,newBalance) (#755)
- SwapAndLiquify(half,newBalance,otherHalf) (#757)
Reentrancy in PirateKing.swapETHForTokens(uint256) (#782-797):
External calls:
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (#789-794)
Event emitted after the call(s):
- SwapETHForTokens(amount,path) (#796)
Reentrancy in PirateKing.swapTokensForEth(uint256) (#762-780):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#771-777)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#779)
Reentrancy in PirateKing.transferFrom(address,address,uint256) (#576-580):
External calls:
- _transfer(sender,recipient,amount) (#577)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#771-777)
External calls sending eth:
- _transfer(sender,recipient,amount) (#577)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#804-811)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#656)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#578)
Apply the check-effects-interactions pattern.
Additional information: link
Ownable.unlock() (#192-197) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp > _lockTime,Contract is locked until 7 days) (#194)
PirateKing._transfer(address,address,uint256) (#659-735) uses timestamp for comparisons
Dangerous comparisons:
- overMinTokenBalance && _startTimeForSwap + _intervalMinutesForSwap <= block.timestamp (#679)
Avoid relying on block.timestamp.
Additional information: link
Address.isContract(address) (#87-96) uses assembly
- INLINE ASM (#94)
Address._functionCallWithValue(address,bytes,uint256,string) (#124-141) uses assembly
- INLINE ASM (#133-136)
Do not use evm assembly.
Additional information: link
PirateKing.includeInReward(address) (#638-649) has costly operations inside a loop:
- _excluded.pop() (#645)
Use a local variable to hold the loop computation result.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#124-141) is never used and should be removed
Address.functionCall(address,bytes) (#107-109) is never used and should be removed
Address.functionCall(address,bytes,string) (#111-113) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#115-117) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#119-122) is never used and should be removed
Address.isContract(address) (#87-96) is never used and should be removed
Address.sendValue(address,uint256) (#98-104) is never used and should be removed
Context._msgData() (#10-13) is never used and should be removed
PirateKing.swapETHForTokens(uint256) (#782-797) is never used and should be removed
PirateKing.transferToAddressETH(address,uint256) (#1022-1024) is never used and should be removed
SafeMath.mod(uint256,uint256) (#75-77) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#79-82) is never used and should be removed
Remove unused functions.
Additional information: link
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#166-169)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#171-175)
getUnlockTime() should be declared external:
- Ownable.getUnlockTime() (#177-179)
getTime() should be declared external:
- Ownable.getTime() (#181-183)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#185-190)
unlock() should be declared external:
- Ownable.unlock() (#192-197)
name() should be declared external:
- PirateKing.name() (#541-543)
symbol() should be declared external:
- PirateKing.symbol() (#545-547)
decimals() should be declared external:
- PirateKing.decimals() (#549-551)
totalSupply() should be declared external:
- PirateKing.totalSupply() (#553-555)
transfer(address,uint256) should be declared external:
- PirateKing.transfer(address,uint256) (#562-565)
allowance(address,address) should be declared external:
- PirateKing.allowance(address,address) (#567-569)
approve(address,uint256) should be declared external:
- PirateKing.approve(address,uint256) (#571-574)
transferFrom(address,address,uint256) should be declared external:
- PirateKing.transferFrom(address,address,uint256) (#576-580)
increaseAllowance(address,uint256) should be declared external:
- PirateKing.increaseAllowance(address,uint256) (#582-585)
decreaseAllowance(address,uint256) should be declared external:
- PirateKing.decreaseAllowance(address,uint256) (#587-590)
isExcludedFromReward(address) should be declared external:
- PirateKing.isExcludedFromReward(address) (#592-594)
totalFees() should be declared external:
- PirateKing.totalFees() (#596-598)
deliver(uint256) should be declared external:
- PirateKing.deliver(uint256) (#601-608)
reflectionFromToken(uint256,bool) should be declared external:
- PirateKing.reflectionFromToken(uint256,bool) (#611-620)
excludeFromReward(address) should be declared external:
- PirateKing.excludeFromReward(address) (#628-636)
isExcludedFromFee(address) should be declared external:
- PirateKing.isExcludedFromFee(address) (#953-955)
excludeFromFee(address) should be declared external:
- PirateKing.excludeFromFee(address) (#957-959)
includeInFee(address) should be declared external:
- PirateKing.includeInFee(address) (#961-963)
changeRouterVersion(address) should be declared external:
- PirateKing.changeRouterVersion(address) (#1026-1039)
transferForeignToken(address,address) should be declared external:
- PirateKing.transferForeignToken(address,address) (#1045-1049)
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