Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
KingJames.swapBack() (#744-787) sends eth to arbitrary user
Dangerous calls:
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#771)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in KingJames._transferFrom(address,address,uint256) (#546-597):
External calls:
- swapBack() (#576)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#755-761)
- distributor.deposit{value: amountBNBReflection}() (#770)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#771)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#772)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#777-784)
External calls sending eth:
- swapBack() (#576)
- distributor.deposit{value: amountBNBReflection}() (#770)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#771)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#772)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#777-784)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,Insufficient Balance) (#579)
- _balances[recipient] = _balances[recipient].add(amountReceived) (#582)
- amountReceived = takeFee(sender,amount,isSell,GREEDMode) (#581)
- _balances[address(this)] = _balances[address(this)].add(feeAmount) (#701)
Apply the check-effects-interactions pattern.
Additional information: link
DividendDistributor.distributeDividend(address) (#323-334) ignores return value by SHIB.transfer(shareholder,amount) (#329)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.
Additional information: link
Combination 1: Reentrancy vulnerabilities + Functions that send Ether to arbitraty destination. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.
Combination 2: Unchecked transfer + Functions that send Ether to arbitraty destination. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.
Combination 3: Reentrancy vulnerabilities + Unchecked transfer vulnerability. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.
Reentrancy in DividendDistributor.distributeDividend(address) (#323-334):
External calls:
- SHIB.transfer(shareholder,amount) (#329)
State variables written after the call(s):
- shares[shareholder].totalRealised = shares[shareholder].totalRealised.add(amount) (#331)
- shares[shareholder].totalExcluded = getCumulativeDividends(shares[shareholder].amount) (#332)
Reentrancy in DividendDistributor.process(uint256) (#292-316):
External calls:
- distributeDividend(shareholders[currentIndex]) (#308)
- SHIB.transfer(shareholder,amount) (#329)
State variables written after the call(s):
- currentIndex ++ (#313)
Reentrancy in DividendDistributor.setShare(address,uint256) (#256-270):
External calls:
- distributeDividend(shareholder) (#258)
- SHIB.transfer(shareholder,amount) (#329)
State variables written after the call(s):
- shares[shareholder].amount = amount (#268)
- shares[shareholder].totalExcluded = getCumulativeDividends(shares[shareholder].amount) (#269)
Apply the check-effects-interactions pattern.
Additional information: link
KingJames.swapBack() (#744-787) ignores return value by router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#777-784)
Ensure that all the return values of the function calls are used.
Additional information: link
DividendDistributor.setDistributionCriteria(uint256,uint256) (#251-254) should emit an event for:
- minPeriod = _minPeriod (#252)
- minDistribution = _minDistribution (#253)
KingJames.setMaxWalletPercent(uint256) (#540-543) should emit an event for:
- _maxWalletToken = _totalSupply.mul(maxWallPercent).div(10000) (#541)
KingJames.enableGREED(uint256) (#722-725) should emit an event for:
- GREEDDuration = _seconds (#724)
KingJames.cooldownEnabled(bool,uint8) (#733-736) should emit an event for:
- cooldownTimerInterval = _interval (#735)
KingJames.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool) (#841-849) should emit an event for:
- autoBuybackCap = _cap (#843)
- autoBuybackAmount = _amount (#845)
KingJames.setBuybackMultiplierSettings(uint256,uint256,uint256) (#852-857) should emit an event for:
- buybackMultiplierNumerator = numerator (#854)
- buybackMultiplierDenominator = denominator (#855)
- buybackMultiplierLength = length (#856)
KingJames.setBuyTxLimitInPercent(uint256) (#870-872) should emit an event for:
- _maxBuyTxAmount = _totalSupply.mul(maxBuyTxPercent).div(10000) (#871)
KingJames.setSellTxLimitInPercent(uint256) (#875-877) should emit an event for:
- _maxSellTxAmount = _totalSupply.mul(maxSellTxPercent).div(10000) (#876)
KingJames.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256) (#906-914) should emit an event for:
- liquidityFeeBuy = _liquidityFeeBuy (#907)
- buybackFeeBuy = _buybackFeeBuy (#908)
- reflectionFeeBuy = _reflectionFeeBuy (#909)
- marketingFeeBuy = _marketingFeeBuy (#910)
- devFeeBuy = _devFeeBuy (#911)
- totalFeeBuy = _liquidityFeeBuy.add(_buybackFeeBuy).add(_reflectionFeeBuy).add(_marketingFeeBuy).add(_devFeeBuy) (#912)
- feeDenominator = _feeDenominator (#913)
KingJames.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256) (#917-925) should emit an event for:
- liquidityFeeSell = _liquidityFeeSell (#918)
- buybackFeeSell = _buybackFeeSell (#919)
- reflectionFeeSell = _reflectionFeeSell (#920)
- marketingFeeSell = _marketingFeeSell (#921)
- devFeeSell = _devFeeSell (#922)
- totalFeeSell = _liquidityFeeSell.add(_buybackFeeSell).add(_reflectionFeeSell).add(_marketingFeeSell).add(_devFeeSell) (#923)
- feeDenominator = _feeDenominator (#924)
KingJames.setSwapBackSettings(bool,uint256) (#934-937) should emit an event for:
- swapThreshold = _totalSupply * _amount / 10000 (#936)
KingJames.setTargetLiquidity(uint256,uint256) (#940-943) should emit an event for:
- targetLiquidity = _target (#941)
- targetLiquidityDenominator = _denominator (#942)
Emit an event for critical parameter changes.
Additional information: link
Auth.transferOwnership(address).account (#128) lacks a zero-check on :
- owner = account (#129)
KingJames.setFeeReceivers(address,address)._autoLiquidityReceiver (#928) lacks a zero-check on :
- autoLiquidityReceiver = _autoLiquidityReceiver (#929)
KingJames.setFeeReceivers(address,address)._marketingFeeReceiver (#928) lacks a zero-check on :
- marketingFeeReceiver = _marketingFeeReceiver (#930)
Check that the address is not zero.
Additional information: link
DividendDistributor.distributeDividend(address) (#323-334) has external calls inside a loop: SHIB.transfer(shareholder,amount) (#329)
Favor pull over push strategy for external calls.
Additional information: link
Reentrancy in KingJames.constructor() (#475-505):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#478)
State variables written after the call(s):
- _allowances[address(this)][address(router)] = type()(uint256).max (#479)
- _balances[_presaler] = _totalSupply (#503)
- autoLiquidityReceiver = msg.sender (#500)
- distributor = new DividendDistributor(address(router)) (#481)
- isDividendExempt[pair] = true (#495)
- isDividendExempt[address(this)] = true (#496)
- isDividendExempt[DEAD] = true (#497)
- isFeeExempt[_presaler] = true (#485)
- isTimelockExempt[msg.sender] = true (#489)
- isTimelockExempt[DEAD] = true (#490)
- isTimelockExempt[address(this)] = true (#491)
- isTimelockExempt[DEV] = true (#492)
- isTxLimitExempt[_presaler] = true (#486)
- marketingFeeReceiver = 0x1702910F5501ce4231806C493aCD5D14BebA93c4 (#501)
Reentrancy in DividendDistributor.deposit() (#272-290):
External calls:
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: msg.value}(0,path,address(this),block.timestamp) (#279-284)
State variables written after the call(s):
- dividendsPerShare = dividendsPerShare.add(dividendsPerShareAccuracyFactor.mul(amount).div(totalShares)) (#289)
- totalDividends = totalDividends.add(amount) (#288)
Reentrancy in DividendDistributor.distributeDividend(address) (#323-334):
External calls:
- SHIB.transfer(shareholder,amount) (#329)
State variables written after the call(s):
- shareholderClaims[shareholder] = block.timestamp (#330)
Reentrancy in DividendDistributor.setShare(address,uint256) (#256-270):
External calls:
- distributeDividend(shareholder) (#258)
- SHIB.transfer(shareholder,amount) (#329)
State variables written after the call(s):
- addShareholder(shareholder) (#262)
- shareholderIndexes[shareholder] = shareholders.length (#356)
- removeShareholder(shareholder) (#264)
- shareholderIndexes[shareholders[shareholders.length - 1]] = shareholderIndexes[shareholder] (#362)
- addShareholder(shareholder) (#262)
- shareholders.push(shareholder) (#357)
- removeShareholder(shareholder) (#264)
- shareholders[shareholderIndexes[shareholder]] = shareholders[shareholders.length - 1] (#361)
- shareholders.pop() (#363)
- totalShares = totalShares.sub(shares[shareholder].amount).add(amount) (#267)
Reentrancy in KingJames.triggerAutoBuyback() (#815-824):
External calls:
- buyTokens(autoBuybackAmount,DEAD) (#816)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,to,block.timestamp) (#832-837)
State variables written after the call(s):
- autoBuybackAccumulator = autoBuybackAccumulator.add(autoBuybackAmount) (#822)
- autoBuybackBlockLast = block.number (#821)
- autoBuybackEnabled = false (#823)
- buybackMultiplierTriggeredAt = block.timestamp (#818)
Reentrancy in KingJames.triggerManualBuyback(uint256,bool) (#799-807):
External calls:
- buyTokens(amountToBuy,DEAD) (#802)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,to,block.timestamp) (#832-837)
State variables written after the call(s):
- buybackMultiplierTriggeredAt = block.timestamp (#804)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in KingJames._transferFrom(address,address,uint256) (#546-597):
External calls:
- swapBack() (#576)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#755-761)
- distributor.deposit{value: amountBNBReflection}() (#770)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#771)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#772)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#777-784)
- distributor.setShare(sender,_balances[sender]) (#586)
- distributor.setShare(recipient,_balances[recipient]) (#590)
- distributor.process(distributorGas) (#593)
External calls sending eth:
- swapBack() (#576)
- distributor.deposit{value: amountBNBReflection}() (#770)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#771)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#772)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#777-784)
Event emitted after the call(s):
- Transfer(sender,recipient,amountReceived) (#595)
Reentrancy in KingJames._transferFrom(address,address,uint256) (#546-597):
External calls:
- swapBack() (#576)
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#755-761)
- distributor.deposit{value: amountBNBReflection}() (#770)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#771)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#772)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#777-784)
External calls sending eth:
- swapBack() (#576)
- distributor.deposit{value: amountBNBReflection}() (#770)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#771)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#772)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#777-784)
Event emitted after the call(s):
- Transfer(sender,address(this),feeAmount) (#702)
- amountReceived = takeFee(sender,amount,isSell,GREEDMode) (#581)
Reentrancy in KingJames.constructor() (#475-505):
External calls:
- pair = IDEXFactory(router.factory()).createPair(WBNB,address(this)) (#478)
Event emitted after the call(s):
- Transfer(address(0),_presaler,_totalSupply) (#504)
Reentrancy in KingJames.swapBack() (#744-787):
External calls:
- router.swapExactTokensForETHSupportingFeeOnTransferTokens(amountToSwap,0,path,address(this),block.timestamp) (#755-761)
- distributor.deposit{value: amountBNBReflection}() (#770)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#771)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#772)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#777-784)
External calls sending eth:
- distributor.deposit{value: amountBNBReflection}() (#770)
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#771)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#772)
- router.addLiquidityETH{value: amountBNBLiquidity}(address(this),amountToLiquify,0,0,autoLiquidityReceiver,block.timestamp) (#777-784)
Event emitted after the call(s):
- AutoLiquify(amountBNBLiquidity,amountToLiquify) (#785)
Reentrancy in KingJames.triggerAutoBuyback() (#815-824):
External calls:
- buyTokens(autoBuybackAmount,DEAD) (#816)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,to,block.timestamp) (#832-837)
Event emitted after the call(s):
- BuybackMultiplierActive(buybackMultiplierLength) (#819)
Reentrancy in KingJames.triggerManualBuyback(uint256,bool) (#799-807):
External calls:
- buyTokens(amountToBuy,DEAD) (#802)
- router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,to,block.timestamp) (#832-837)
Event emitted after the call(s):
- BuybackMultiplierActive(buybackMultiplierLength) (#805)
Apply the check-effects-interactions pattern.
Additional information: link
DividendDistributor.shouldDistribute(address) (#318-321) uses timestamp for comparisons
Dangerous comparisons:
- shareholderClaims[shareholder] + minPeriod < block.timestamp && getUnpaidEarnings(shareholder) > minDistribution (#319-320)
KingJames.inGREEDTime() (#627-633) uses timestamp for comparisons
Dangerous comparisons:
- GREEDTriggeredAt.add(GREEDDuration) > block.timestamp (#628)
KingJames.checkBuyCooldown(address,address) (#648-655) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(cooldownTimer[recipient] < block.timestamp,Please wait between two buys) (#652)
KingJames.getTotalFee(bool) (#671-675) uses timestamp for comparisons
Dangerous comparisons:
- selling && buybackMultiplierTriggeredAt.add(buybackMultiplierLength) > block.timestamp (#673)
Avoid relying on block.timestamp.
Additional information: link
DividendDistributor.process(uint256) (#292-316) has costly operations inside a loop:
- currentIndex = 0 (#304)
DividendDistributor.distributeDividend(address) (#323-334) has costly operations inside a loop:
- totalDistributed = totalDistributed.add(amount) (#328)
DividendDistributor.process(uint256) (#292-316) has costly operations inside a loop:
- currentIndex ++ (#313)
Use a local variable to hold the loop computation result.
Additional information: link
KingJames.launched() (#860-862) is never used and should be removed
KingJames.shouldAutoBuyback() (#790-796) is never used and should be removed
KingJames.triggerAutoBuyback() (#815-824) is never used and should be removed
Remove unused functions.
Additional information: link
KingJames._maxBuyTxAmount (#391) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 25 / 10000
KingJames._maxSellTxAmount (#392) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 28 / 10000
KingJames._maxWalletToken (#393) is set pre-construction with a non-constant function or state variable:
- (_totalSupply * 100) / 10000
KingJames.swapThreshold (#437) is set pre-construction with a non-constant function or state variable:
- _totalSupply * 100 / 10000
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 KingJames.swapBack() (#744-787):
- (successMarketing) = address(marketingFeeReceiver).call{gas: 30000,value: amountBNBMarketing}() (#771)
- (successDev) = address(DEV).call{gas: 30000,value: amountBNBDev}() (#772)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Function IDEXRouter.WETH() (#145) is not in mixedCase
Parameter DividendDistributor.setDistributionCriteria(uint256,uint256)._minPeriod (#251) is not in mixedCase
Parameter DividendDistributor.setDistributionCriteria(uint256,uint256)._minDistribution (#251) is not in mixedCase
Variable DividendDistributor._token (#203) is not in mixedCase
Variable DividendDistributor.SHIB (#212) is not in mixedCase
Variable DividendDistributor.WBNB (#213) is not in mixedCase
Parameter KingJames.takeFee(address,uint256,bool,bool).GREEDMode (#685) is not in mixedCase
Parameter KingJames.tradingStatus(bool)._status (#716) is not in mixedCase
Parameter KingJames.enableGREED(uint256)._seconds (#722) is not in mixedCase
Parameter KingJames.cooldownEnabled(bool,uint8)._status (#733) is not in mixedCase
Parameter KingJames.cooldownEnabled(bool,uint8)._interval (#733) is not in mixedCase
Parameter KingJames.blacklistAddress(address,bool)._address (#739) is not in mixedCase
Parameter KingJames.blacklistAddress(address,bool)._value (#739) is not in mixedCase
Parameter KingJames.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._enabled (#841) is not in mixedCase
Parameter KingJames.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._cap (#841) is not in mixedCase
Parameter KingJames.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._amount (#841) is not in mixedCase
Parameter KingJames.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._period (#841) is not in mixedCase
Parameter KingJames.setAutoBuybackSettings(bool,uint256,uint256,uint256,bool)._autoBuybackMultiplier (#841) is not in mixedCase
Parameter KingJames.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._liquidityFeeBuy (#906) is not in mixedCase
Parameter KingJames.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._buybackFeeBuy (#906) is not in mixedCase
Parameter KingJames.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._reflectionFeeBuy (#906) is not in mixedCase
Parameter KingJames.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._marketingFeeBuy (#906) is not in mixedCase
Parameter KingJames.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._devFeeBuy (#906) is not in mixedCase
Parameter KingJames.setBuyFees(uint256,uint256,uint256,uint256,uint256,uint256)._feeDenominator (#906) is not in mixedCase
Parameter KingJames.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._liquidityFeeSell (#917) is not in mixedCase
Parameter KingJames.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._buybackFeeSell (#917) is not in mixedCase
Parameter KingJames.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._reflectionFeeSell (#917) is not in mixedCase
Parameter KingJames.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._marketingFeeSell (#917) is not in mixedCase
Parameter KingJames.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._devFeeSell (#917) is not in mixedCase
Parameter KingJames.setSellFees(uint256,uint256,uint256,uint256,uint256,uint256)._feeDenominator (#917) is not in mixedCase
Parameter KingJames.setFeeReceivers(address,address)._autoLiquidityReceiver (#928) is not in mixedCase
Parameter KingJames.setFeeReceivers(address,address)._marketingFeeReceiver (#928) is not in mixedCase
Parameter KingJames.setSwapBackSettings(bool,uint256)._enabled (#934) is not in mixedCase
Parameter KingJames.setSwapBackSettings(bool,uint256)._amount (#934) is not in mixedCase
Parameter KingJames.setTargetLiquidity(uint256,uint256)._target (#940) is not in mixedCase
Parameter KingJames.setTargetLiquidity(uint256,uint256)._denominator (#940) is not in mixedCase
Parameter KingJames.setDistributionCriteria(uint256,uint256)._minPeriod (#952) is not in mixedCase
Parameter KingJames.setDistributionCriteria(uint256,uint256)._minDistribution (#952) is not in mixedCase
Variable KingJames.SHIB (#372) is not in mixedCase
Variable KingJames.WBNB (#373) is not in mixedCase
Variable KingJames.DEAD (#374) is not in mixedCase
Variable KingJames.ZERO (#375) is not in mixedCase
Variable KingJames.DEV (#376) is not in mixedCase
Constant KingJames._name (#383) is not in UPPER_CASE_WITH_UNDERSCORES
Constant KingJames._symbol (#384) is not in UPPER_CASE_WITH_UNDERSCORES
Constant KingJames._decimals (#385) is not in UPPER_CASE_WITH_UNDERSCORES
Variable KingJames._totalSupply (#388) is not in mixedCase
Variable KingJames._maxBuyTxAmount (#391) is not in mixedCase
Variable KingJames._maxSellTxAmount (#392) is not in mixedCase
Variable KingJames._maxWalletToken (#393) is not in mixedCase
Variable KingJames._balances (#396) is not in mixedCase
Variable KingJames._allowances (#397) is not in mixedCase
Variable KingJames.GREEDTriggeredAt (#430) is not in mixedCase
Variable KingJames.GREEDDuration (#431) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Variable IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#150) is too similar to IDEXRouter.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#151)
Prevent variables from having similar names.
Additional information: link
KingJames.slitherConstructorVariables() (#368-990) uses literals with too many digits:
- DEAD = 0x000000000000000000000000000000000000dEaD (#374)
KingJames.slitherConstructorVariables() (#368-990) uses literals with too many digits:
- ZERO = 0x0000000000000000000000000000000000000000 (#375)
KingJames.slitherConstructorVariables() (#368-990) uses literals with too many digits:
- _totalSupply = 1000000 * 10 ** 0 * (10 ** _decimals) (#388)
KingJames.slitherConstructorVariables() (#368-990) uses literals with too many digits:
- distributorGas = 500000 (#458)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
KingJames.SHIB (#372) is never used in KingJames (#368-990)
Remove unused state variables.
Additional information: link
DividendDistributor.WBNB (#213) should be constant
DividendDistributor.dividendsPerShareAccuracyFactor (#226) should be constant
KingJames.DEAD (#374) should be constant
KingJames.DEV (#376) should be constant
KingJames.SHIB (#372) should be constant
KingJames.WBNB (#373) should be constant
KingJames.ZERO (#375) should be constant
KingJames._totalSupply (#388) should be constant
KingJames.deadBlocks (#434) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
authorize(address) should be declared external:
- Auth.authorize(address) (#100-102)
unauthorize(address) should be declared external:
- Auth.unauthorize(address) (#107-109)
transferOwnership(address) should be declared external:
- Auth.transferOwnership(address) (#128-132)
getTotalFee(bool) should be declared external:
- KingJames.getTotalFee(bool) (#671-675)
tradingStatus(bool) should be declared external:
- KingJames.tradingStatus(bool) (#716-719)
enableGREED(uint256) should be declared external:
- KingJames.enableGREED(uint256) (#722-725)
cooldownEnabled(bool,uint8) should be declared external:
- KingJames.cooldownEnabled(bool,uint8) (#733-736)
blacklistAddress(address,bool) should be declared external:
- KingJames.blacklistAddress(address,bool) (#739-741)
getUnpaidEarnings(address) should be declared external:
- KingJames.getUnpaidEarnings(address) (#962-964)
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 Telegram and Twitter accounts