ASIA [Asian Doge] Token

Reentrancy in ASIANDOGE._transfer(address,address,uint256) (#518-554):
External calls:
- swapAndSendToFee(sellTokens) (#540)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#588-594)
External calls sending eth:
- swapAndSendToFee(sellTokens) (#540)
- address(_marketingWallet).transfer(ethToSend) (#561)
State variables written after the call(s):
- _tOwned[from] -= amount (#544)
- transferAmount = _getValues(amount,from) (#549)
- _tOwned[address(this)] += marketingFee + liquidityFee (#500)
- _tOwned[to] += transferAmount (#552)
- swapping = false (#541)
Apply the check-effects-interactions pattern.

Additional information: link

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

Contract ownership is not renounced (belongs to a wallet)

ASIANDOGE.allowance(address,address).owner (#460) shadows:
- Ownable.owner() (#156-158) (function)
ASIANDOGE._approve(address,address,uint256).owner (#510) shadows:
- Ownable.owner() (#156-158) (function)
Rename the local variables that shadow another component.

Additional information: link

Reentrancy in ASIANDOGE.constructor() (#417-433):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#422)
State variables written after the call(s):
- _isExcludedFromFee[owner()] = true (#428)
- _isExcludedFromFee[address(this)] = true (#429)
- _isExcludedFromFee[_marketingWallet] = true (#430)
- uniswapV2Pair = _uniswapV2Pair (#425)
- uniswapV2Router = _uniswapV2Router (#424)
Reentrancy in ASIANDOGE.swapAndLiquify() (#564-576):
External calls:
- newBalance = swapTokensForEth(half) (#569)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#588-594)
- addLiquidity(otherHalf,newBalance) (#573)
- (ethFromLiquidity) = uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#603-610)
- (success) = recipient.call{value: amount}() (#100)
- address(_marketingWallet).sendValue(ethAmount - ethFromLiquidity) (#613)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#573)
- (ethFromLiquidity) = uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#603-610)
- (success) = recipient.call{value: amount}() (#100)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (#573)
- _allowances[owner][spender] = amount (#514)
Reentrancy in ASIANDOGE.transferFrom(address,address,uint256) (#469-473):
External calls:
- _transfer(sender,recipient,amount) (#470)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#588-594)
External calls sending eth:
- _transfer(sender,recipient,amount) (#470)
- address(_marketingWallet).transfer(ethToSend) (#561)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()] - amount) (#471)
- _allowances[owner][spender] = amount (#514)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in ASIANDOGE._transfer(address,address,uint256) (#518-554):
External calls:
- swapAndSendToFee(sellTokens) (#540)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#588-594)
External calls sending eth:
- swapAndSendToFee(sellTokens) (#540)
- address(_marketingWallet).transfer(ethToSend) (#561)
Event emitted after the call(s):
- Transfer(from,address(this),marketingFee + liquidityFee) (#501)
- transferAmount = _getValues(amount,from) (#549)
- Transfer(from,to,transferAmount) (#553)
Reentrancy in ASIANDOGE.constructor() (#417-433):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#422)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_tTotal) (#432)
Reentrancy in ASIANDOGE.swapAndLiquify() (#564-576):
External calls:
- newBalance = swapTokensForEth(half) (#569)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#588-594)
- addLiquidity(otherHalf,newBalance) (#573)
- (ethFromLiquidity) = uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#603-610)
- (success) = recipient.call{value: amount}() (#100)
- address(_marketingWallet).sendValue(ethAmount - ethFromLiquidity) (#613)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#573)
- (ethFromLiquidity) = uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#603-610)
- (success) = recipient.call{value: amount}() (#100)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#515)
- addLiquidity(otherHalf,newBalance) (#573)
- SwapAndLiquify(half,newBalance,otherHalf) (#574)
Reentrancy in ASIANDOGE.transferFrom(address,address,uint256) (#469-473):
External calls:
- _transfer(sender,recipient,amount) (#470)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#588-594)
External calls sending eth:
- _transfer(sender,recipient,amount) (#470)
- address(_marketingWallet).transfer(ethToSend) (#561)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#515)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()] - amount) (#471)
Apply the check-effects-interactions pattern.

Additional information: link

ASIANDOGE._transfer(address,address,uint256) (#518-554) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(_lastBuy[to] + _buyCooldown < block.timestamp,Must wait til after coooldown to buy) (#532)
ASIANDOGE.addLiquidity(uint256,uint256) (#598-614) uses timestamp for comparisons
Dangerous comparisons:
- ethAmount - ethFromLiquidity > 0 (#612)
Avoid relying on block.timestamp.

Additional information: link

Address.isContract(address) (#85-94) uses assembly
- INLINE ASM (#92)
Address._functionCallWithValue(address,bytes,uint256,string) (#121-142) uses assembly
- INLINE ASM (#134-137)
Do not use evm assembly.

Additional information: link

ASIANDOGE.addLiquidity(uint256,uint256) (#598-614) is never used and should be removed
ASIANDOGE.swapAndLiquify() (#564-576) is never used and should be removed
Address._functionCallWithValue(address,bytes,uint256,string) (#121-142) is never used and should be removed
Address.functionCall(address,bytes) (#104-106) is never used and should be removed
Address.functionCall(address,bytes,string) (#108-110) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#112-114) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#116-119) is never used and should be removed
Address.isContract(address) (#85-94) is never used and should be removed
Address.sendValue(address,uint256) (#96-102) is never used and should be removed
Context._msgData() (#77-80) is never used and should be removed
SafeMath.add(uint256,uint256) (#22-27) is never used and should be removed
SafeMath.div(uint256,uint256) (#51-53) is never used and should be removed
SafeMath.div(uint256,uint256,string) (#54-60) is never used and should be removed
SafeMath.mod(uint256,uint256) (#62-64) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#66-69) is never used and should be removed
SafeMath.mul(uint256,uint256) (#37-49) is never used and should be removed
SafeMath.sub(uint256,uint256) (#28-30) is never used and should be removed
SafeMath.sub(uint256,uint256,string) (#31-36) is never used and should be removed
Remove unused functions.

Additional information: link

Low level call in Address.sendValue(address,uint256) (#96-102):
- (success) = recipient.call{value: amount}() (#100)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#121-142):
- (success,returndata) = target.call{value: weiValue}(data) (#125)
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() (#213) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#214) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#231) is not in mixedCase
Function IUniswapV2Router01.WETH() (#251) is not in mixedCase
Variable ASIANDOGE._maxTxAmount (#400) is not in mixedCase
Constant ASIANDOGE._name (#402) is not in UPPER_CASE_WITH_UNDERSCORES
Constant ASIANDOGE._symbol (#403) is not in UPPER_CASE_WITH_UNDERSCORES
Constant ASIANDOGE._decimals (#404) is not in UPPER_CASE_WITH_UNDERSCORES
Variable ASIANDOGE._marketingFee (#406) is not in mixedCase
Variable ASIANDOGE._liquidityFee (#407) is not in mixedCase
Variable ASIANDOGE._buyCooldown (#410) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#78)" inContext (#72-81)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Reentrancy in ASIANDOGE._transfer(address,address,uint256) (#518-554):
External calls:
- swapAndSendToFee(sellTokens) (#540)
- address(_marketingWallet).transfer(ethToSend) (#561)
State variables written after the call(s):
- _tOwned[from] -= amount (#544)
- transferAmount = _getValues(amount,from) (#549)
- _tOwned[address(this)] += marketingFee + liquidityFee (#500)
- _tOwned[to] += transferAmount (#552)
- swapping = false (#541)
Event emitted after the call(s):
- Transfer(from,address(this),marketingFee + liquidityFee) (#501)
- transferAmount = _getValues(amount,from) (#549)
- Transfer(from,to,transferAmount) (#553)
Reentrancy in ASIANDOGE.transferFrom(address,address,uint256) (#469-473):
External calls:
- _transfer(sender,recipient,amount) (#470)
- address(_marketingWallet).transfer(ethToSend) (#561)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()] - amount) (#471)
- _allowances[owner][spender] = amount (#514)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#515)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()] - amount) (#471)
Apply the check-effects-interactions pattern.

Additional information: link

Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#256) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#257)
Prevent variables from having similar names.

Additional information: link

Address.sendValue(address,uint256) (#96-102) uses literals with too many digits:
- recipient = address(0x000000000000000000000000000000000000dEaD) (#98)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

ASIANDOGE._buyCooldown (#410) should be constant
ASIANDOGE._liquidityFee (#407) should be constant
ASIANDOGE._marketingFee (#406) should be constant
ASIANDOGE._marketingWallet (#408) should be constant
ASIANDOGE._maxTxAmount (#400) should be constant
ASIANDOGE._tTotal (#399) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#165-168)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#170-174)
name() should be declared external:
- ASIANDOGE.name() (#435-437)
symbol() should be declared external:
- ASIANDOGE.symbol() (#439-441)
decimals() should be declared external:
- ASIANDOGE.decimals() (#443-445)
totalSupply() should be declared external:
- ASIANDOGE.totalSupply() (#447-449)
transfer(address,uint256) should be declared external:
- ASIANDOGE.transfer(address,uint256) (#455-458)
allowance(address,address) should be declared external:
- ASIANDOGE.allowance(address,address) (#460-462)
approve(address,uint256) should be declared external:
- ASIANDOGE.approve(address,uint256) (#464-467)
transferFrom(address,address,uint256) should be declared external:
- ASIANDOGE.transferFrom(address,address,uint256) (#469-473)
increaseAllowance(address,uint256) should be declared external:
- ASIANDOGE.increaseAllowance(address,uint256) (#475-478)
decreaseAllowance(address,uint256) should be declared external:
- ASIANDOGE.decreaseAllowance(address,uint256) (#480-483)
excludeFromFee(address) should be declared external:
- ASIANDOGE.excludeFromFee(address) (#485-487)
includeInFee(address) should be declared external:
- ASIANDOGE.includeInFee(address) (#489-491)
isExcludedFromFee(address) should be declared external:
- ASIANDOGE.isExcludedFromFee(address) (#506-508)
Use the external attribute for functions never called from the contract.

Additional information: link

Average 30d PancakeSwap liquidity is less than $100. Token is either dead or inactive.

Average 30d PancakeSwap volume is less than $100. Token is either dead or inactive.

Number of Binance Smart Chain (BSC) token holders is less than 100. Token is either dead or inactive. Ignore for presale.

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

Average 30d number of PancakeSwap swaps is low.

Token is deployed only at one blockchain

Token has only one trading pair

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