EGT [ElonGoat] Token

Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)

Reentrancy in ElonGoat._transfer(address,address,uint256) (#641-678):
External calls:
- swapTokens(contractTokenBalance) (#666)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
External calls sending eth:
- swapTokens(contractTokenBalance) (#666)
- recipient.transfer(amount) (#939)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _liquidityFee = _previousLiquidityFee (#861)
- _liquidityFee = 0 (#856)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#832)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#757)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#748)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#778)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#749)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#768)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#769)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#759)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#780)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _rTotal = _rTotal.sub(rFee) (#787)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#834)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#767)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#777)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#758)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#779)
Apply the check-effects-interactions pattern.

Additional information: link

Contract ownership is not renounced (belongs to a wallet)

Address._functionCallWithValue(address,bytes,uint256,string) (#149-166) is never used and should be removed
Address.functionCall(address,bytes) (#132-134) is never used and should be removed
Address.functionCall(address,bytes,string) (#136-138) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#140-142) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#144-147) is never used and should be removed
Address.isContract(address) (#112-121) is never used and should be removed
Address.sendValue(address,uint256) (#123-129) is never used and should be removed
Context._msgData() (#35-38) is never used and should be removed
ElonGoat.addLiquidity(uint256,uint256) (#713-726) is never used and should be removed
SafeMath.mod(uint256,uint256) (#100-102) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#104-107) is never used and should be removed
Remove unused functions.

Additional information: link

ElonGoat.swapTokens(uint256) (#680-691) performs a multiplication on the result of a division:
-transferToAddressETH(marketingAddress,transferredBalance.div(_liquidityFee).mul(marketingDivisor)) (#687)
ElonGoat.swapTokens(uint256) (#680-691) performs a multiplication on the result of a division:
-transferToAddressETH(devAddress,transferredBalance.div(_liquidityFee).mul(devDivisor)) (#688)
ElonGoat.swapTokens(uint256) (#680-691) performs a multiplication on the result of a division:
-transferToAddressETH(burnAddress,transferredBalance.div(_liquidityFee).mul(burnDivisor)) (#689)
Consider ordering multiplication before division.

Additional information: link

ElonGoat.addLiquidity(uint256,uint256) (#713-726) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#718-725)
Ensure that all the return values of the function calls are used.

Additional information: link

Ownable.unlock() (#217-222) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp > _lockTime,Contract is locked until 7 days) (#219)
Avoid relying on block.timestamp.

Additional information: link

Address.isContract(address) (#112-121) uses assembly
- INLINE ASM (#119)
Address._functionCallWithValue(address,bytes,uint256,string) (#149-166) uses assembly
- INLINE ASM (#158-161)
Do not use evm assembly.

Additional information: link

Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#304) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#305)
Variable ElonGoat.reflectionFromToken(uint256,bool).rTransferAmount (#599) is too similar to ElonGoat._transferFromExcluded(address,address,uint256).tTransferAmount (#766)
Variable ElonGoat.reflectionFromToken(uint256,bool).rTransferAmount (#599) is too similar to ElonGoat._transferBothExcluded(address,address,uint256).tTransferAmount (#776)
Variable ElonGoat._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#808) is too similar to ElonGoat._getTValues(uint256).tTransferAmount (#800)
Variable ElonGoat.reflectionFromToken(uint256,bool).rTransferAmount (#599) is too similar to ElonGoat._transferToExcluded(address,address,uint256).tTransferAmount (#756)
Variable ElonGoat._transferToExcluded(address,address,uint256).rTransferAmount (#756) is too similar to ElonGoat._transferFromExcluded(address,address,uint256).tTransferAmount (#766)
Variable ElonGoat.reflectionFromToken(uint256,bool).rTransferAmount (#599) is too similar to ElonGoat._getValues(uint256).tTransferAmount (#792)
Variable ElonGoat._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#808) is too similar to ElonGoat._transferFromExcluded(address,address,uint256).tTransferAmount (#766)
Variable ElonGoat._transferToExcluded(address,address,uint256).rTransferAmount (#756) is too similar to ElonGoat._transferToExcluded(address,address,uint256).tTransferAmount (#756)
Variable ElonGoat.reflectionFromToken(uint256,bool).rTransferAmount (#599) is too similar to ElonGoat._transferStandard(address,address,uint256).tTransferAmount (#747)
Variable ElonGoat._transferFromExcluded(address,address,uint256).rTransferAmount (#766) is too similar to ElonGoat._transferFromExcluded(address,address,uint256).tTransferAmount (#766)
Variable ElonGoat._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#808) is too similar to ElonGoat._transferBothExcluded(address,address,uint256).tTransferAmount (#776)
Variable ElonGoat._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#808) is too similar to ElonGoat._transferToExcluded(address,address,uint256).tTransferAmount (#756)
Variable ElonGoat._transferFromExcluded(address,address,uint256).rTransferAmount (#766) is too similar to ElonGoat._transferBothExcluded(address,address,uint256).tTransferAmount (#776)
Variable ElonGoat._transferToExcluded(address,address,uint256).rTransferAmount (#756) is too similar to ElonGoat._getValues(uint256).tTransferAmount (#792)
Variable ElonGoat._transferFromExcluded(address,address,uint256).rTransferAmount (#766) is too similar to ElonGoat._transferToExcluded(address,address,uint256).tTransferAmount (#756)
Variable ElonGoat._transferToExcluded(address,address,uint256).rTransferAmount (#756) is too similar to ElonGoat._transferStandard(address,address,uint256).tTransferAmount (#747)
Variable ElonGoat.reflectionFromToken(uint256,bool).rTransferAmount (#599) is too similar to ElonGoat._getTValues(uint256).tTransferAmount (#800)
Variable ElonGoat._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#808) is too similar to ElonGoat._getValues(uint256).tTransferAmount (#792)
Variable ElonGoat._transferFromExcluded(address,address,uint256).rTransferAmount (#766) is too similar to ElonGoat._getValues(uint256).tTransferAmount (#792)
Variable ElonGoat._transferStandard(address,address,uint256).rTransferAmount (#747) is too similar to ElonGoat._transferStandard(address,address,uint256).tTransferAmount (#747)
Variable ElonGoat._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#808) is too similar to ElonGoat._transferStandard(address,address,uint256).tTransferAmount (#747)
Variable ElonGoat._transferFromExcluded(address,address,uint256).rTransferAmount (#766) is too similar to ElonGoat._transferStandard(address,address,uint256).tTransferAmount (#747)
Variable ElonGoat._transferBothExcluded(address,address,uint256).rTransferAmount (#776) is too similar to ElonGoat._getTValues(uint256).tTransferAmount (#800)
Variable ElonGoat._transferStandard(address,address,uint256).rTransferAmount (#747) is too similar to ElonGoat._transferToExcluded(address,address,uint256).tTransferAmount (#756)
Variable ElonGoat._transferToExcluded(address,address,uint256).rTransferAmount (#756) is too similar to ElonGoat._getTValues(uint256).tTransferAmount (#800)
Variable ElonGoat._transferStandard(address,address,uint256).rTransferAmount (#747) is too similar to ElonGoat._transferFromExcluded(address,address,uint256).tTransferAmount (#766)
Variable ElonGoat._getValues(uint256).rTransferAmount (#793) is too similar to ElonGoat._transferBothExcluded(address,address,uint256).tTransferAmount (#776)
Variable ElonGoat._getValues(uint256).rTransferAmount (#793) is too similar to ElonGoat._getValues(uint256).tTransferAmount (#792)
Variable ElonGoat._transferStandard(address,address,uint256).rTransferAmount (#747) is too similar to ElonGoat._getTValues(uint256).tTransferAmount (#800)
Variable ElonGoat._transferBothExcluded(address,address,uint256).rTransferAmount (#776) is too similar to ElonGoat._transferBothExcluded(address,address,uint256).tTransferAmount (#776)
Variable ElonGoat._getValues(uint256).rTransferAmount (#793) is too similar to ElonGoat._transferToExcluded(address,address,uint256).tTransferAmount (#756)
Variable ElonGoat._transferBothExcluded(address,address,uint256).rTransferAmount (#776) is too similar to ElonGoat._getValues(uint256).tTransferAmount (#792)
Variable ElonGoat._transferToExcluded(address,address,uint256).rTransferAmount (#756) is too similar to ElonGoat._transferBothExcluded(address,address,uint256).tTransferAmount (#776)
Variable ElonGoat._getValues(uint256).rTransferAmount (#793) is too similar to ElonGoat._transferStandard(address,address,uint256).tTransferAmount (#747)
Variable ElonGoat._transferBothExcluded(address,address,uint256).rTransferAmount (#776) is too similar to ElonGoat._transferToExcluded(address,address,uint256).tTransferAmount (#756)
Variable ElonGoat._getValues(uint256).rTransferAmount (#793) is too similar to ElonGoat._transferFromExcluded(address,address,uint256).tTransferAmount (#766)
Variable ElonGoat._transferBothExcluded(address,address,uint256).rTransferAmount (#776) is too similar to ElonGoat._transferStandard(address,address,uint256).tTransferAmount (#747)
Variable ElonGoat._transferStandard(address,address,uint256).rTransferAmount (#747) is too similar to ElonGoat._transferBothExcluded(address,address,uint256).tTransferAmount (#776)
Variable ElonGoat._getValues(uint256).rTransferAmount (#793) is too similar to ElonGoat._getTValues(uint256).tTransferAmount (#800)
Variable ElonGoat._transferBothExcluded(address,address,uint256).rTransferAmount (#776) is too similar to ElonGoat._transferFromExcluded(address,address,uint256).tTransferAmount (#766)
Variable ElonGoat._transferStandard(address,address,uint256).rTransferAmount (#747) is too similar to ElonGoat._getValues(uint256).tTransferAmount (#792)
Variable ElonGoat._transferFromExcluded(address,address,uint256).rTransferAmount (#766) is too similar to ElonGoat._getTValues(uint256).tTransferAmount (#800)
Prevent variables from having similar names.

Additional information: link

ElonGoat.allowance(address,address).owner (#546) shadows:
- Ownable.owner() (#182-184) (function)
ElonGoat._approve(address,address,uint256).owner (#633) shadows:
- Ownable.owner() (#182-184) (function)
Rename the local variables that shadow another component.

Additional information: link

ElonGoat.setTaxFeePercent(uint256) (#888-890) should emit an event for:
- _taxFee = taxFee (#889)
ElonGoat.setLiquidityFeePercent(uint256) (#892-894) should emit an event for:
- _liquidityFee = liquidityFee (#893)
ElonGoat.setMaxTxAmount(uint256) (#896-898) should emit an event for:
- _maxTxAmount = maxTxAmount (#897)
ElonGoat.setMarketingDivisor(uint256) (#900-902) should emit an event for:
- marketingDivisor = divisor (#901)
ElonGoat.setDevDivisor(uint256) (#904-906) should emit an event for:
- devDivisor = divisor (#905)
ElonGoat.setBurnDivisor(uint256) (#908-910) should emit an event for:
- burnDivisor = divisor (#909)
ElonGoat.setNumTokensSellToAddToLiquidity(uint256) (#912-914) should emit an event for:
- minimumTokensBeforeSwap = _minimumTokensBeforeSwap (#913)
Emit an event for critical parameter changes.

Additional information: link

ElonGoat.includeInReward(address) (#620-631) has costly operations inside a loop:
- _excluded.pop() (#627)
Use a local variable to hold the loop computation result.

Additional information: link

ElonGoat.setMarketingAddress(address)._marketingAddress (#916) lacks a zero-check on :
- marketingAddress = address(_marketingAddress) (#917)
ElonGoat.setDevAddress(address)._devAddress (#920) lacks a zero-check on :
- devAddress = address(_devAddress) (#921)
ElonGoat.setBurnAddress(address)._burnAddress (#924) lacks a zero-check on :
- burnAddress = address(_burnAddress) (#925)
Check that the address is not zero.

Additional information: link

Reentrancy in ElonGoat._transfer(address,address,uint256) (#641-678):
External calls:
- swapTokens(contractTokenBalance) (#666)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
External calls sending eth:
- swapTokens(contractTokenBalance) (#666)
- recipient.transfer(amount) (#939)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _previousLiquidityFee = _liquidityFee (#853)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _previousTaxFee = _taxFee (#852)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _tFeeTotal = _tFeeTotal.add(tFee) (#788)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _taxFee = _previousTaxFee (#860)
- _taxFee = 0 (#855)
Reentrancy in ElonGoat.constructor() (#504-518):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#508-509)
State variables written after the call(s):
- _isExcludedFromFee[owner()] = true (#514)
- _isExcludedFromFee[address(this)] = true (#515)
- uniswapV2Router = _uniswapV2Router (#511)
Reentrancy in ElonGoat.transferFrom(address,address,uint256) (#555-559):
External calls:
- _transfer(sender,recipient,amount) (#556)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
External calls sending eth:
- _transfer(sender,recipient,amount) (#556)
- recipient.transfer(amount) (#939)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#557)
- _allowances[owner][spender] = amount (#637)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in ElonGoat._transfer(address,address,uint256) (#641-678):
External calls:
- swapTokens(contractTokenBalance) (#666)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
External calls sending eth:
- swapTokens(contractTokenBalance) (#666)
- recipient.transfer(amount) (#939)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (#752)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- Transfer(sender,recipient,tTransferAmount) (#772)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- Transfer(sender,recipient,tTransferAmount) (#762)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- Transfer(sender,recipient,tTransferAmount) (#783)
- _tokenTransfer(from,to,amount,takeFee) (#677)
Reentrancy in ElonGoat.constructor() (#504-518):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#508-509)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_tTotal) (#517)
Reentrancy in ElonGoat.swapTokensForEth(uint256) (#693-711):
External calls:
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
Event emitted after the call(s):
- SwapTokensForETH(tokenAmount,path) (#710)
Reentrancy in ElonGoat.transferFrom(address,address,uint256) (#555-559):
External calls:
- _transfer(sender,recipient,amount) (#556)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#702-708)
External calls sending eth:
- _transfer(sender,recipient,amount) (#556)
- recipient.transfer(amount) (#939)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#638)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#557)
Apply the check-effects-interactions pattern.

Additional information: link

ElonGoat._rTotal (#456) is set pre-construction with a non-constant function or state variable:
- (MAX - (MAX % _tTotal))
ElonGoat._previousTaxFee (#465) is set pre-construction with a non-constant function or state variable:
- _taxFee
ElonGoat._previousLiquidityFee (#468) 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) (#123-129):
- (success) = recipient.call{value: amount}() (#127)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#149-166):
- (success,returndata) = target.call{value: weiValue}(data) (#152)
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() (#261) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#262) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#278) is not in mixedCase
Function IUniswapV2Router01.WETH() (#299) is not in mixedCase
Parameter ElonGoat.calculateTaxFee(uint256)._amount (#837) is not in mixedCase
Parameter ElonGoat.calculateLiquidityFee(uint256)._amount (#843) is not in mixedCase
Parameter ElonGoat.setNumTokensSellToAddToLiquidity(uint256)._minimumTokensBeforeSwap (#912) is not in mixedCase
Parameter ElonGoat.setMarketingAddress(address)._marketingAddress (#916) is not in mixedCase
Parameter ElonGoat.setDevAddress(address)._devAddress (#920) is not in mixedCase
Parameter ElonGoat.setBurnAddress(address)._burnAddress (#924) is not in mixedCase
Parameter ElonGoat.setSwapAndLiquifyEnabled(bool)._enabled (#928) is not in mixedCase
Parameter ElonGoat.setContractLockEnabled(bool)._enabled (#933) is not in mixedCase
Variable ElonGoat._taxFee (#464) is not in mixedCase
Variable ElonGoat._liquidityFee (#467) is not in mixedCase
Variable ElonGoat._maxTxAmount (#474) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#36)" inContext (#30-39)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Reentrancy in ElonGoat._transfer(address,address,uint256) (#641-678):
External calls:
- swapTokens(contractTokenBalance) (#666)
- recipient.transfer(amount) (#939)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _liquidityFee = _previousLiquidityFee (#861)
- _liquidityFee = 0 (#856)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _previousLiquidityFee = _liquidityFee (#853)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _previousTaxFee = _taxFee (#852)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#832)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#757)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#748)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#778)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#749)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#768)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#769)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#759)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#780)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _rTotal = _rTotal.sub(rFee) (#787)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _tFeeTotal = _tFeeTotal.add(tFee) (#788)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#834)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#767)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#777)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#758)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#779)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- _taxFee = _previousTaxFee (#860)
- _taxFee = 0 (#855)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (#752)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- Transfer(sender,recipient,tTransferAmount) (#762)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- Transfer(sender,recipient,tTransferAmount) (#772)
- _tokenTransfer(from,to,amount,takeFee) (#677)
- Transfer(sender,recipient,tTransferAmount) (#783)
- _tokenTransfer(from,to,amount,takeFee) (#677)
Reentrancy in ElonGoat.transferFrom(address,address,uint256) (#555-559):
External calls:
- _transfer(sender,recipient,amount) (#556)
- recipient.transfer(amount) (#939)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#557)
- _allowances[owner][spender] = amount (#637)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#638)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#557)
Apply the check-effects-interactions pattern.

Additional information: link

ElonGoat._decimals (#461) should be constant
ElonGoat._name (#459) should be constant
ElonGoat._symbol (#460) should be constant
ElonGoat._tTotal (#455) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#191-194)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#196-200)
getUnlockTime() should be declared external:
- Ownable.getUnlockTime() (#202-204)
getTime() should be declared external:
- Ownable.getTime() (#206-208)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#210-215)
unlock() should be declared external:
- Ownable.unlock() (#217-222)
name() should be declared external:
- ElonGoat.name() (#520-522)
symbol() should be declared external:
- ElonGoat.symbol() (#524-526)
decimals() should be declared external:
- ElonGoat.decimals() (#528-530)
totalSupply() should be declared external:
- ElonGoat.totalSupply() (#532-534)
transfer(address,uint256) should be declared external:
- ElonGoat.transfer(address,uint256) (#541-544)
allowance(address,address) should be declared external:
- ElonGoat.allowance(address,address) (#546-548)
approve(address,uint256) should be declared external:
- ElonGoat.approve(address,uint256) (#550-553)
transferFrom(address,address,uint256) should be declared external:
- ElonGoat.transferFrom(address,address,uint256) (#555-559)
increaseAllowance(address,uint256) should be declared external:
- ElonGoat.increaseAllowance(address,uint256) (#561-564)
decreaseAllowance(address,uint256) should be declared external:
- ElonGoat.decreaseAllowance(address,uint256) (#566-569)
isExcludedFromReward(address) should be declared external:
- ElonGoat.isExcludedFromReward(address) (#571-573)
totalFees() should be declared external:
- ElonGoat.totalFees() (#575-577)
minimumTokensBeforeSwapAmount() should be declared external:
- ElonGoat.minimumTokensBeforeSwapAmount() (#579-581)
deliver(uint256) should be declared external:
- ElonGoat.deliver(uint256) (#583-590)
reflectionFromToken(uint256,bool) should be declared external:
- ElonGoat.reflectionFromToken(uint256,bool) (#593-602)
excludeFromReward(address) should be declared external:
- ElonGoat.excludeFromReward(address) (#610-618)
isExcludedFromFee(address) should be declared external:
- ElonGoat.isExcludedFromFee(address) (#864-866)
excludeFromFee(address) should be declared external:
- ElonGoat.excludeFromFee(address) (#868-870)
includeInFee(address) should be declared external:
- ElonGoat.includeInFee(address) (#872-874)
isLocked(address) should be declared external:
- ElonGoat.isLocked(address) (#876-878)
lockAccount(address) should be declared external:
- ElonGoat.lockAccount(address) (#880-882)
unlockAccount(address) should be declared external:
- ElonGoat.unlockAccount(address) (#884-886)
setSwapAndLiquifyEnabled(bool) should be declared external:
- ElonGoat.setSwapAndLiquifyEnabled(bool) (#928-931)
setContractLockEnabled(bool) should be declared external:
- ElonGoat.setContractLockEnabled(bool) (#933-936)
Use the external attribute for functions never called from the contract.

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.

BscScan page for the token does not contain additional info: website, socials, description, etc.

Additional information: link

Unable to find KYC or doxxing proof

Unable to find token contract audit

Token is not listed at Mobula.Finance

Additional information: link

Unable to find audit link on the website

Unable to find token on CoinHunt

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