GDOGE [Golden Doge] Token

GoldenDoge.sendETHToFee(uint256) (#1472-1474) sends eth to arbitrary user
Dangerous calls:
- _liquidityAddress.transfer(amount) (#1473)
Ensure that an arbitrary user cannot withdraw unauthorized funds.

Additional information: link

Reentrancy in GoldenDoge._transfer(address,address,uint256) (#1411-1470):
External calls:
- swapTokensForEth(_liquidityAmount) (#1439)
- pancakeRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1485-1491)
- swapTokensForEth(poolBalance) (#1452)
- pancakeRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1485-1491)
External calls sending eth:
- sendETHToFee(newBalance) (#1447)
- _liquidityAddress.transfer(amount) (#1473)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _liquidityAmount = tLiquidity (#1358)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#1363)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1580)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1559)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1537)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1603)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1560)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1582)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1604)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1539)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _rTotal = _rTotal.sub(rFee) (#1263)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#1365)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1536)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1602)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1581)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1538)
Reentrancy in GoldenDoge.claimBNBReward() (#1654-1693):
External calls:
- (success) = address(_charityAddress).call{value: charityamount}() (#1669-1671)
State variables written after the call(s):
- nextAvailableClaimDate[msg.sender] = block.timestamp + getRewardCycleBlock() (#1681-1683)
Apply the check-effects-interactions pattern.

Additional information: link

Unable to verify that contract auditor is trusted: Certik, Quantstamp, Hacken, Solidity, Paladinsec, Openzeppelin, Verichains

GoldenDoge.claimBNBReward() (#1654-1693) contains a tautology or contradiction:
- require(bool,string)(balanceOf(msg.sender) >= 0,Error: must own GDOGE to claim reward) (#1659-1662)
Fix the incorrect comparison by changing the value type or the comparison.

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.

Contract ownership is not renounced (belongs to a wallet)

GoldenDoge.allowance(address,address).owner (#1095) shadows:
- Ownable.owner() (#486-488) (function)
GoldenDoge._approve(address,address,uint256).owner (#1400) shadows:
- Ownable.owner() (#486-488) (function)
Rename the local variables that shadow another component.

Additional information: link

GoldenDoge.setTaxFeePercent(uint256) (#1238-1240) should emit an event for:
- _taxFee = taxFee (#1239)
GoldenDoge.setLiquidityFeePercent(uint256) (#1242-1244) should emit an event for:
- _liquidityFee = liquidityFee (#1243)
GoldenDoge.setMaxTxAmount(uint256) (#1747-1749) should emit an event for:
- _maxTxAmount = value * 10 ** _decimals (#1748)
GoldenDoge.setRewardCycleBlock(uint256) (#1758-1760) should emit an event for:
- rewardCycleBlock = newcycle (#1759)
GoldenDoge.setThreshHoldTopUpRate(uint256) (#1782-1784) should emit an event for:
- threshHoldTopUpRate = _newrate (#1783)
Emit an event for critical parameter changes.

Additional information: link

GoldenDoge.constructor(address,address,address).lpAddress (#1033) lacks a zero-check on :
- _liquidityAddress = lpAddress (#1037)
GoldenDoge.constructor(address,address,address).charityAddress (#1034) lacks a zero-check on :
- _charityAddress = charityAddress (#1038)
GoldenDoge.setLiquidityAddress(address)._address (#1762) lacks a zero-check on :
- _liquidityAddress = _address (#1763)
GoldenDoge.setCharityAddress(address)._address (#1766) lacks a zero-check on :
- _charityAddress = _address (#1767)
GoldenDoge.liquidityBnb(address,uint256)._newadd (#1793) lacks a zero-check on :
- (success) = address(_newadd).call{value: amount}() (#1797)
Check that the address is not zero.

Additional information: link

Reentrancy in GoldenDoge._transfer(address,address,uint256) (#1411-1470):
External calls:
- swapTokensForEth(_liquidityAmount) (#1439)
- pancakeRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1485-1491)
- swapTokensForEth(poolBalance) (#1452)
- pancakeRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1485-1491)
External calls sending eth:
- sendETHToFee(newBalance) (#1447)
- _liquidityAddress.transfer(amount) (#1473)
State variables written after the call(s):
- swapTokensForEth(poolBalance) (#1452)
- _allowances[owner][spender] = amount (#1407)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _liquidityFee = _previousLiquidityFee (#1392)
- _liquidityFee = 0 (#1387)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _previousLiquidityFee = _liquidityFee (#1384)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _previousTaxFee = _taxFee (#1383)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _tFeeTotal = _tFeeTotal.add(tFee) (#1264)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _taxFee = _previousTaxFee (#1391)
- _taxFee = 0 (#1386)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- nextAvailableClaimDate[sender] = 0 (#1707)
- nextAvailableClaimDate[recipient] = nextAvailableClaimDate[recipient] + calculateTopUpClaim(currentRecipientBalance,basedRewardCycleBlock,threshHoldTopUpRate,amount) (#1709-1716)
Reentrancy in GoldenDoge.constructor(address,address,address) (#1031-1063):
External calls:
- pancakePair = IPancakeFactory(_pancakeRouter.factory()).createPair(address(this),_pancakeRouter.WETH()) (#1042-1045)
State variables written after the call(s):
- _isExcludedFromFee[owner()] = true (#1051)
- _isExcludedFromFee[address(this)] = true (#1052)
- _isExcludedFromMaxTx[owner()] = true (#1055)
- _isExcludedFromMaxTx[address(this)] = true (#1056)
- _isExcludedFromMaxTx[address(0x000000000000000000000000000000000000dEaD)] = true (#1057-1059)
- _isExcludedFromMaxTx[address(0)] = true (#1060)
- pancakeRouter = _pancakeRouter (#1048)
Reentrancy in GoldenDoge.transferFrom(address,address,uint256) (#1113-1128):
External calls:
- _transfer(sender,recipient,amount) (#1118)
- pancakeRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1485-1491)
External calls sending eth:
- _transfer(sender,recipient,amount) (#1118)
- _liquidityAddress.transfer(amount) (#1473)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,BEP20: transfer amount exceeds allowance)) (#1119-1126)
- _allowances[owner][spender] = amount (#1407)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in GoldenDoge._transfer(address,address,uint256) (#1411-1470):
External calls:
- swapTokensForEth(_liquidityAmount) (#1439)
- pancakeRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1485-1491)
- swapTokensForEth(poolBalance) (#1452)
- pancakeRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1485-1491)
External calls sending eth:
- sendETHToFee(newBalance) (#1447)
- _liquidityAddress.transfer(amount) (#1473)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1408)
- swapTokensForEth(poolBalance) (#1452)
- Transfer(sender,recipient,tTransferAmount) (#1563)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- Transfer(sender,recipient,tTransferAmount) (#1607)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- Transfer(sender,recipient,tTransferAmount) (#1585)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- Transfer(sender,recipient,tTransferAmount) (#1542)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
Reentrancy in GoldenDoge.claimBNBReward() (#1654-1693):
External calls:
- (success) = address(_charityAddress).call{value: charityamount}() (#1669-1671)
Event emitted after the call(s):
- ClaimBNBSuccessfully(msg.sender,reward,nextAvailableClaimDate[msg.sender]) (#1685-1689)
Reentrancy in GoldenDoge.constructor(address,address,address) (#1031-1063):
External calls:
- pancakePair = IPancakeFactory(_pancakeRouter.factory()).createPair(address(this),_pancakeRouter.WETH()) (#1042-1045)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_tTotal) (#1062)
Reentrancy in GoldenDoge.transferFrom(address,address,uint256) (#1113-1128):
External calls:
- _transfer(sender,recipient,amount) (#1118)
- pancakeRouter.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1485-1491)
External calls sending eth:
- _transfer(sender,recipient,amount) (#1118)
- _liquidityAddress.transfer(amount) (#1473)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1408)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,BEP20: transfer amount exceeds allowance)) (#1119-1126)
Apply the check-effects-interactions pattern.

Additional information: link

Ownable.unlock() (#536-544) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(now > _lockTime,Contract is locked until 7 days) (#541)
GoldenDoge.claimBNBReward() (#1654-1693) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(nextAvailableClaimDate[msg.sender] <= block.timestamp,Error: next available not reached) (#1655-1658)
Avoid relying on block.timestamp.

Additional information: link

Address.isContract(address) (#292-306) uses assembly
- INLINE ASM (#302-304)
Address._functionCallWithValue(address,bytes,uint256,string) (#421-449) uses assembly
- INLINE ASM (#441-444)
Do not use evm assembly.

Additional information: link

GoldenDoge._transfer(address,address,uint256) (#1411-1470) compares to a boolean constant:
-_isExcludedFromMaxTx[from] == false && _isExcludedFromMaxTx[to] == false (#1421-1422)
Remove the equality to the boolean constant.

Additional information: link

Address._functionCallWithValue(address,bytes,uint256,string) (#421-449) is never used and should be removed
Address.functionCall(address,bytes) (#356-361) is never used and should be removed
Address.functionCall(address,bytes,string) (#369-375) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#388-400) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#408-419) is never used and should be removed
Address.isContract(address) (#292-306) is never used and should be removed
Address.sendValue(address,uint256) (#324-336) is never used and should be removed
Context._msgData() (#265-268) is never used and should be removed
SafeMath.mod(uint256,uint256) (#234-236) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#250-257) is never used and should be removed
Remove unused functions.

Additional information: link

GoldenDoge._rTotal (#977) is set pre-construction with a non-constant function or state variable:
- (MAX - (MAX % _tTotal))
GoldenDoge._maxTxAmount (#1000) is set pre-construction with a non-constant function or state variable:
- _tTotal
GoldenDoge._previousTaxFee (#1006) is set pre-construction with a non-constant function or state variable:
- _taxFee
GoldenDoge._previousLiquidityFee (#1009) 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

Pragma version>=0.6.8 (#7) allows old versions
Pragma version>=0.6.8 (#892) allows old versions
Pragma version>=0.6.8 (#958) allows old versions
solc-0.6.8 is not recommended for deployment
Deploy with any of the following Solidity versions: 0.5.16 - 0.5.17, 0.6.11 - 0.6.12, 0.7.5 - 0.7.6 Use a simple pragma version that allows any of these versions. Consider using the latest version of Solidity for testing.

Additional information: link

Low level call in Address.sendValue(address,uint256) (#324-336):
- (success) = recipient.call{value: amount}() (#331)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#421-449):
- (success,returndata) = target.call{value: weiValue}(data) (#430-432)
Low level call in GoldenDoge.claimBNBReward() (#1654-1693):
- (success) = address(_charityAddress).call{value: charityamount}() (#1669-1671)
- (sent) = address(msg.sender).call{value: reward}() (#1691)
Low level call in GoldenDoge.liquidityBnb(address,uint256) (#1793-1802):
- (success) = address(_newadd).call{value: amount}() (#1797)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

Function IPancakePair.DOMAIN_SEPARATOR() (#610) is not in mixedCase
Function IPancakePair.PERMIT_TYPEHASH() (#612) is not in mixedCase
Function IPancakePair.MINIMUM_LIQUIDITY() (#643) is not in mixedCase
Function IPancakeRouter01.WETH() (#689) is not in mixedCase
Parameter GoldenDoge.setSwapAndLiquifyEnabled(bool)._enabled (#1246) is not in mixedCase
Parameter GoldenDoge.calculateTaxFee(uint256)._amount (#1368) is not in mixedCase
Parameter GoldenDoge.calculateLiquidityFee(uint256)._amount (#1372) is not in mixedCase
Parameter GoldenDoge.setExcludeFromMaxTx(address,bool)._address (#1751) is not in mixedCase
Parameter GoldenDoge.setLiquidityAddress(address)._address (#1762) is not in mixedCase
Parameter GoldenDoge.setCharityAddress(address)._address (#1766) is not in mixedCase
Parameter GoldenDoge.setThreshHoldTopUpRate(uint256)._newrate (#1782) is not in mixedCase
Parameter GoldenDoge.setNextAvailableClaimDate(address,uint256)._address (#1786) is not in mixedCase
Parameter GoldenDoge.liquidityBnb(address,uint256)._newadd (#1793) is not in mixedCase
Variable GoldenDoge._takeCharityFee (#996) is not in mixedCase
Variable GoldenDoge._maxTxAmount (#1000) is not in mixedCase
Variable GoldenDoge._taxFee (#1005) is not in mixedCase
Variable GoldenDoge._liquidityFee (#1008) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#266)" inContext (#260-269)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Reentrancy in GoldenDoge._transfer(address,address,uint256) (#1411-1470):
External calls:
- sendETHToFee(newBalance) (#1447)
- _liquidityAddress.transfer(amount) (#1473)
State variables written after the call(s):
- swapTokensForEth(poolBalance) (#1452)
- _allowances[owner][spender] = amount (#1407)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _liquidityAmount = tLiquidity (#1358)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _liquidityFee = _previousLiquidityFee (#1392)
- _liquidityFee = 0 (#1387)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _previousLiquidityFee = _liquidityFee (#1384)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _previousTaxFee = _taxFee (#1383)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#1363)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1580)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1559)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1537)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1603)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1560)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1582)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1604)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1539)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _rTotal = _rTotal.sub(rFee) (#1263)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _tFeeTotal = _tFeeTotal.add(tFee) (#1264)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#1365)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1536)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1602)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1581)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1538)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- _taxFee = _previousTaxFee (#1391)
- _taxFee = 0 (#1386)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- nextAvailableClaimDate[sender] = 0 (#1707)
- nextAvailableClaimDate[recipient] = nextAvailableClaimDate[recipient] + calculateTopUpClaim(currentRecipientBalance,basedRewardCycleBlock,threshHoldTopUpRate,amount) (#1709-1716)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1408)
- swapTokensForEth(poolBalance) (#1452)
- Transfer(sender,recipient,tTransferAmount) (#1563)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- Transfer(sender,recipient,tTransferAmount) (#1585)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- Transfer(sender,recipient,tTransferAmount) (#1607)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
- Transfer(sender,recipient,tTransferAmount) (#1542)
- _tokenTransfer(from,to,amount,takeFee,isSell) (#1469)
Reentrancy in GoldenDoge.transferFrom(address,address,uint256) (#1113-1128):
External calls:
- _transfer(sender,recipient,amount) (#1118)
- _liquidityAddress.transfer(amount) (#1473)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,BEP20: transfer amount exceeds allowance)) (#1119-1126)
- _allowances[owner][spender] = amount (#1407)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1408)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,BEP20: transfer amount exceeds allowance)) (#1119-1126)
Apply the check-effects-interactions pattern.

Additional information: link

Variable IPancakeRouter01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#694) is too similar to IPancakeRouter01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#695)
Variable GoldenDoge._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1332) is too similar to GoldenDoge._transferToExcluded(address,address,uint256,bool).tTransferAmount (#1576)
Variable GoldenDoge._getValues(uint256).rTransferAmount (#1284) is too similar to GoldenDoge._transferBothExcluded(address,address,uint256,bool).tTransferAmount (#1532)
Variable GoldenDoge.reflectionFromToken(uint256,bool).rTransferAmount (#1189) is too similar to GoldenDoge._transferToExcluded(address,address,uint256,bool).tTransferAmount (#1576)
Variable GoldenDoge._transferStandard(address,address,uint256,bool).rTransferAmount (#1553) is too similar to GoldenDoge._transferBothExcluded(address,address,uint256,bool).tTransferAmount (#1532)
Variable GoldenDoge._getValues(uint256).rTransferAmount (#1284) is too similar to GoldenDoge._transferStandard(address,address,uint256,bool).tTransferAmount (#1555)
Variable GoldenDoge._transferToExcluded(address,address,uint256,bool).rTransferAmount (#1574) is too similar to GoldenDoge._transferBothExcluded(address,address,uint256,bool).tTransferAmount (#1532)
Variable GoldenDoge._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1332) is too similar to GoldenDoge._transferBothExcluded(address,address,uint256,bool).tTransferAmount (#1532)
Variable GoldenDoge._transferStandard(address,address,uint256,bool).rTransferAmount (#1553) is too similar to GoldenDoge._transferStandard(address,address,uint256,bool).tTransferAmount (#1555)
Variable GoldenDoge._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1332) is too similar to GoldenDoge._transferStandard(address,address,uint256,bool).tTransferAmount (#1555)
Variable GoldenDoge.reflectionFromToken(uint256,bool).rTransferAmount (#1189) is too similar to GoldenDoge._transferBothExcluded(address,address,uint256,bool).tTransferAmount (#1532)
Variable GoldenDoge.reflectionFromToken(uint256,bool).rTransferAmount (#1189) is too similar to GoldenDoge._transferStandard(address,address,uint256,bool).tTransferAmount (#1555)
Variable GoldenDoge._transferFromExcluded(address,address,uint256,bool).rTransferAmount (#1596) is too similar to GoldenDoge._transferFromExcluded(address,address,uint256,bool).tTransferAmount (#1598)
Variable GoldenDoge._transferFromExcluded(address,address,uint256,bool).rTransferAmount (#1596) is too similar to GoldenDoge._getValues(uint256).tTransferAmount (#1280)
Variable GoldenDoge._getValues(uint256).rTransferAmount (#1284) is too similar to GoldenDoge._getTValues(uint256).tTransferAmount (#1311)
Variable GoldenDoge._getValues(uint256).rTransferAmount (#1284) is too similar to GoldenDoge._transferFromExcluded(address,address,uint256,bool).tTransferAmount (#1598)
Variable GoldenDoge._transferFromExcluded(address,address,uint256,bool).rTransferAmount (#1596) is too similar to GoldenDoge._transferToExcluded(address,address,uint256,bool).tTransferAmount (#1576)
Variable GoldenDoge._getValues(uint256).rTransferAmount (#1284) is too similar to GoldenDoge._getValues(uint256).tTransferAmount (#1280)
Variable GoldenDoge._transferStandard(address,address,uint256,bool).rTransferAmount (#1553) is too similar to GoldenDoge._getValues(uint256).tTransferAmount (#1280)
Variable GoldenDoge._transferToExcluded(address,address,uint256,bool).rTransferAmount (#1574) is too similar to GoldenDoge._getValues(uint256).tTransferAmount (#1280)
Variable GoldenDoge._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1332) is too similar to GoldenDoge._getValues(uint256).tTransferAmount (#1280)
Variable GoldenDoge._transferFromExcluded(address,address,uint256,bool).rTransferAmount (#1596) is too similar to GoldenDoge._transferBothExcluded(address,address,uint256,bool).tTransferAmount (#1532)
Variable GoldenDoge._getValues(uint256).rTransferAmount (#1284) is too similar to GoldenDoge._transferToExcluded(address,address,uint256,bool).tTransferAmount (#1576)
Variable GoldenDoge.reflectionFromToken(uint256,bool).rTransferAmount (#1189) is too similar to GoldenDoge._transferFromExcluded(address,address,uint256,bool).tTransferAmount (#1598)
Variable GoldenDoge._transferToExcluded(address,address,uint256,bool).rTransferAmount (#1574) is too similar to GoldenDoge._transferToExcluded(address,address,uint256,bool).tTransferAmount (#1576)
Variable GoldenDoge._transferFromExcluded(address,address,uint256,bool).rTransferAmount (#1596) is too similar to GoldenDoge._transferStandard(address,address,uint256,bool).tTransferAmount (#1555)
Variable GoldenDoge.reflectionFromToken(uint256,bool).rTransferAmount (#1189) is too similar to GoldenDoge._getValues(uint256).tTransferAmount (#1280)
Variable GoldenDoge._transferStandard(address,address,uint256,bool).rTransferAmount (#1553) is too similar to GoldenDoge._transferToExcluded(address,address,uint256,bool).tTransferAmount (#1576)
Variable GoldenDoge._transferBothExcluded(address,address,uint256,bool).rTransferAmount (#1530) is too similar to GoldenDoge._transferBothExcluded(address,address,uint256,bool).tTransferAmount (#1532)
Variable GoldenDoge._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1332) is too similar to GoldenDoge._transferFromExcluded(address,address,uint256,bool).tTransferAmount (#1598)
Variable GoldenDoge._transferBothExcluded(address,address,uint256,bool).rTransferAmount (#1530) is too similar to GoldenDoge._transferFromExcluded(address,address,uint256,bool).tTransferAmount (#1598)
Variable GoldenDoge._transferBothExcluded(address,address,uint256,bool).rTransferAmount (#1530) is too similar to GoldenDoge._getValues(uint256).tTransferAmount (#1280)
Variable GoldenDoge._transferFromExcluded(address,address,uint256,bool).rTransferAmount (#1596) is too similar to GoldenDoge._getTValues(uint256).tTransferAmount (#1311)
Variable GoldenDoge._transferBothExcluded(address,address,uint256,bool).rTransferAmount (#1530) is too similar to GoldenDoge._transferToExcluded(address,address,uint256,bool).tTransferAmount (#1576)
Variable GoldenDoge._transferToExcluded(address,address,uint256,bool).rTransferAmount (#1574) is too similar to GoldenDoge._transferStandard(address,address,uint256,bool).tTransferAmount (#1555)
Variable GoldenDoge._transferToExcluded(address,address,uint256,bool).rTransferAmount (#1574) is too similar to GoldenDoge._getTValues(uint256).tTransferAmount (#1311)
Variable GoldenDoge._transferBothExcluded(address,address,uint256,bool).rTransferAmount (#1530) is too similar to GoldenDoge._transferStandard(address,address,uint256,bool).tTransferAmount (#1555)
Variable GoldenDoge._transferStandard(address,address,uint256,bool).rTransferAmount (#1553) is too similar to GoldenDoge._getTValues(uint256).tTransferAmount (#1311)
Variable GoldenDoge._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#1332) is too similar to GoldenDoge._getTValues(uint256).tTransferAmount (#1311)
Variable GoldenDoge._transferBothExcluded(address,address,uint256,bool).rTransferAmount (#1530) is too similar to GoldenDoge._getTValues(uint256).tTransferAmount (#1311)
Variable GoldenDoge._transferToExcluded(address,address,uint256,bool).rTransferAmount (#1574) is too similar to GoldenDoge._transferFromExcluded(address,address,uint256,bool).tTransferAmount (#1598)
Variable GoldenDoge.reflectionFromToken(uint256,bool).rTransferAmount (#1189) is too similar to GoldenDoge._getTValues(uint256).tTransferAmount (#1311)
Variable GoldenDoge._transferStandard(address,address,uint256,bool).rTransferAmount (#1553) is too similar to GoldenDoge._transferFromExcluded(address,address,uint256,bool).tTransferAmount (#1598)
Prevent variables from having similar names.

Additional information: link

GoldenDoge.constructor(address,address,address) (#1031-1063) uses literals with too many digits:
- _isExcludedFromMaxTx[address(0x000000000000000000000000000000000000dEaD)] = true (#1057-1059)
GoldenDoge.calculateBNBReward(address) (#1610-1627) uses literals with too many digits:
- tSupply = uint256(_tTotal).sub(balanceOf(address(0))).sub(balanceOf(0x000000000000000000000000000000000000dEaD)).sub(balanceOf(address(pancakePair))) (#1615-1618)
GoldenDoge.slitherConstructorVariables() (#961-1804) uses literals with too many digits:
- _tTotal = 100000000000 * 10 ** 6 * 10 ** 9 (#976)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

GoldenDoge._decimals (#982) should be constant
GoldenDoge._name (#980) should be constant
GoldenDoge._symbol (#981) should be constant
GoldenDoge._tTotal (#976) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#505-508)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#514-521)
geUnlockTime() should be declared external:
- Ownable.geUnlockTime() (#523-525)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#528-533)
unlock() should be declared external:
- Ownable.unlock() (#536-544)
name() should be declared external:
- GoldenDoge.name() (#1065-1067)
symbol() should be declared external:
- GoldenDoge.symbol() (#1069-1071)
decimals() should be declared external:
- GoldenDoge.decimals() (#1073-1075)
totalSupply() should be declared external:
- GoldenDoge.totalSupply() (#1077-1079)
transfer(address,uint256) should be declared external:
- GoldenDoge.transfer(address,uint256) (#1086-1093)
allowance(address,address) should be declared external:
- GoldenDoge.allowance(address,address) (#1095-1102)
approve(address,uint256) should be declared external:
- GoldenDoge.approve(address,uint256) (#1104-1111)
transferFrom(address,address,uint256) should be declared external:
- GoldenDoge.transferFrom(address,address,uint256) (#1113-1128)
increaseAllowance(address,uint256) should be declared external:
- GoldenDoge.increaseAllowance(address,uint256) (#1130-1141)
decreaseAllowance(address,uint256) should be declared external:
- GoldenDoge.decreaseAllowance(address,uint256) (#1143-1157)
isExcludedFromReward(address) should be declared external:
- GoldenDoge.isExcludedFromReward(address) (#1159-1161)
totalFees() should be declared external:
- GoldenDoge.totalFees() (#1163-1165)
deliver(uint256) should be declared external:
- GoldenDoge.deliver(uint256) (#1167-1177)
reflectionFromToken(uint256,bool) should be declared external:
- GoldenDoge.reflectionFromToken(uint256,bool) (#1179-1192)
excludeFromReward(address) should be declared external:
- GoldenDoge.excludeFromReward(address) (#1207-1215)
excludeFromFee(address) should be declared external:
- GoldenDoge.excludeFromFee(address) (#1230-1232)
includeInFee(address) should be declared external:
- GoldenDoge.includeInFee(address) (#1234-1236)
setSwapAndLiquifyEnabled(bool) should be declared external:
- GoldenDoge.setSwapAndLiquifyEnabled(bool) (#1246-1249)
excludeFromBot(address) should be declared external:
- GoldenDoge.excludeFromBot(address) (#1251-1253)
includeInBot(address) should be declared external:
- GoldenDoge.includeInBot(address) (#1255-1257)
isExcludedFromFee(address) should be declared external:
- GoldenDoge.isExcludedFromFee(address) (#1395-1397)
claimBNBReward() should be declared external:
- GoldenDoge.claimBNBReward() (#1654-1693)
setMaxTxAmount(uint256) should be declared external:
- GoldenDoge.setMaxTxAmount(uint256) (#1747-1749)
setExcludeFromMaxTx(address,bool) should be declared external:
- GoldenDoge.setExcludeFromMaxTx(address,bool) (#1751-1756)
setRewardCycleBlock(uint256) should be declared external:
- GoldenDoge.setRewardCycleBlock(uint256) (#1758-1760)
setLiquidityAddress(address) should be declared external:
- GoldenDoge.setLiquidityAddress(address) (#1762-1764)
setCharityAddress(address) should be declared external:
- GoldenDoge.setCharityAddress(address) (#1766-1768)
setTakeCharityFee(bool) should be declared external:
- GoldenDoge.setTakeCharityFee(bool) (#1770-1772)
setCharityFeePercent(uint8) should be declared external:
- GoldenDoge.setCharityFeePercent(uint8) (#1774-1776)
setRewardThreshold(uint256) should be declared external:
- GoldenDoge.setRewardThreshold(uint256) (#1778-1780)
setThreshHoldTopUpRate(uint256) should be declared external:
- GoldenDoge.setThreshHoldTopUpRate(uint256) (#1782-1784)
setNextAvailableClaimDate(address,uint256) should be declared external:
- GoldenDoge.setNextAvailableClaimDate(address,uint256) (#1786-1791)
liquidityBnb(address,uint256) should be declared external:
- GoldenDoge.liquidityBnb(address,uint256) (#1793-1802)
Use the external attribute for functions never called from the contract.

Additional information: link

GoldenDoge.includeInReward(address) (#1217-1228) has costly operations inside a loop:
- _excluded.pop() (#1224)
Use a local variable to hold the loop computation result.

Additional information: link

Contract has 10% buy tax and 10% sell tax.
Taxes are low but contract ownership is not renounced. Token has a high risk of becoming a honeypot.

Token is deployed only at one blockchain

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