ACC Token

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

ACCToken.isContractSwitch (#270) is never initialized. It is used in:
- ACCToken._transfer(address,address,uint256) (#448-513)
Initialize all the variables. If a variable is meant to be initialized to zero, explicitly set it to zero to improve code readability.

Additional information: link

Contract ownership is not renounced (belongs to a wallet)

ACCToken.USDT (#286) should be constant
ACCToken._decimals (#260) should be constant
ACCToken._maxTxAmount (#284) should be constant
ACCToken._name (#263) should be constant
ACCToken._symbol (#264) should be constant
ACCToken._tTotal (#261) should be constant
ACCToken.isContractSwitch (#270) should be constant
ACCToken.totalBuyFee (#267) should be constant
ACCToken.totalSellFee (#268) should be constant
ACCToken.transferFee (#269) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

owner() should be declared external:
- Ownable.owner() (#157-159)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#166-170)
name() should be declared external:
- ACCToken.name() (#335-337)
symbol() should be declared external:
- ACCToken.symbol() (#339-341)
decimals() should be declared external:
- ACCToken.decimals() (#343-345)
totalSupply() should be declared external:
- ACCToken.totalSupply() (#347-349)
transfer(address,uint256) should be declared external:
- ACCToken.transfer(address,uint256) (#355-358)
allowance(address,address) should be declared external:
- ACCToken.allowance(address,address) (#360-362)
approve(address,uint256) should be declared external:
- ACCToken.approve(address,uint256) (#364-367)
transferFrom(address,address,uint256) should be declared external:
- ACCToken.transferFrom(address,address,uint256) (#369-373)
increaseAllowance(address,uint256) should be declared external:
- ACCToken.increaseAllowance(address,uint256) (#375-378)
decreaseAllowance(address,uint256) should be declared external:
- ACCToken.decreaseAllowance(address,uint256) (#380-383)
excludeFromFee(address) should be declared external:
- ACCToken.excludeFromFee(address) (#385-387)
setWhiteAddress(address,bool) should be declared external:
- ACCToken.setWhiteAddress(address,bool) (#389-391)
setBlack(address,bool) should be declared external:
- ACCToken.setBlack(address,bool) (#393-395)
excludeFromFees(address[]) should be declared external:
- ACCToken.excludeFromFees(address[]) (#397-402)
includeInFee(address) should be declared external:
- ACCToken.includeInFee(address) (#404-406)
setisFee(bool) should be declared external:
- ACCToken.setisFee(bool) (#408-410)
setExcludedEx(bool) should be declared external:
- ACCToken.setExcludedEx(bool) (#412-414)
setIsSell(bool) should be declared external:
- ACCToken.setIsSell(bool) (#416-418)
setIsBuy(bool) should be declared external:
- ACCToken.setIsBuy(bool) (#420-422)
isExcludedFromFee(address) should be declared external:
- ACCToken.isExcludedFromFee(address) (#434-437)
Use the external attribute for functions never called from the contract.

Additional information: link

Reentrancy in ACCToken._transfer(address,address,uint256) (#448-513):
External calls:
- swapAndLiquify(contractTokenBalance) (#474)
- uniswapV2Router.addLiquidity(address(this),USDT,tokenAmount,usdtAmount,0,0,holder,block.timestamp) (#568-577)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,to,block.timestamp) (#555-561)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,param) (#512)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#591)
- _tOwned[to] = _tOwned[to].add(tValue) (#430)
- _tOwned[recipient] = _tOwned[recipient].add(param.tTransferAmount) (#592)
Apply the check-effects-interactions pattern.

Additional information: link

ACCToken._transfer(address,address,uint256).param (#485) is a local variable never initialized
Initialize all the variables. If a variable is meant to be initialized to zero, explicitly set it to zero to improve code readability.

Additional information: link

ACCToken.addLiquidity(uint256,uint256) (#564-578) ignores return value by uniswapV2Router.addLiquidity(address(this),USDT,tokenAmount,usdtAmount,0,0,holder,block.timestamp) (#568-577)
Ensure that all the return values of the function calls are used.

Additional information: link

ACCToken.allowance(address,address).owner (#360) shadows:
- Ownable.owner() (#157-159) (function)
ACCToken._approve(address,address,uint256).owner (#439) shadows:
- Ownable.owner() (#157-159) (function)
Rename the local variables that shadow another component.

Additional information: link

Reentrancy in ACCToken.constructor() (#308-328):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),USDT) (#316-317)
State variables written after the call(s):
- _isExcludedFromFee[holder] = true (#322)
- _isExcludedFromFee[address(this)] = true (#323)
- _owner = msg.sender (#325)
- exPairs[uniswapV2Pair] = true (#320)
- uniswapV2Pair = _uniswapV2Pair (#319)
Reentrancy in ACCToken.swapAndLiquify(uint256) (#533-545):
External calls:
- swapTokensForUSDT(half,address(this)) (#540)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,to,block.timestamp) (#555-561)
- addLiquidity(otherHalf,newBalance) (#544)
- uniswapV2Router.addLiquidity(address(this),USDT,tokenAmount,usdtAmount,0,0,holder,block.timestamp) (#568-577)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (#544)
- _allowances[owner][spender] = amount (#443)
Reentrancy in ACCToken.transferFrom(address,address,uint256) (#369-373):
External calls:
- _transfer(sender,recipient,amount) (#370)
- uniswapV2Router.addLiquidity(address(this),USDT,tokenAmount,usdtAmount,0,0,holder,block.timestamp) (#568-577)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,to,block.timestamp) (#555-561)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#371)
- _allowances[owner][spender] = amount (#443)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in ACCToken._transfer(address,address,uint256) (#448-513):
External calls:
- swapAndLiquify(contractTokenBalance) (#474)
- uniswapV2Router.addLiquidity(address(this),USDT,tokenAmount,usdtAmount,0,0,holder,block.timestamp) (#568-577)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,to,block.timestamp) (#555-561)
Event emitted after the call(s):
- Transfer(from,to,tValue) (#431)
- _tokenTransfer(from,to,amount,param) (#512)
- Transfer(sender,recipient,param.tTransferAmount) (#593)
- _tokenTransfer(from,to,amount,param) (#512)
Reentrancy in ACCToken.constructor() (#308-328):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),USDT) (#316-317)
Event emitted after the call(s):
- Transfer(address(0),holder,_tTotal) (#327)
Reentrancy in ACCToken.swapAndLiquify(uint256) (#533-545):
External calls:
- swapTokensForUSDT(half,address(this)) (#540)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,to,block.timestamp) (#555-561)
- addLiquidity(otherHalf,newBalance) (#544)
- uniswapV2Router.addLiquidity(address(this),USDT,tokenAmount,usdtAmount,0,0,holder,block.timestamp) (#568-577)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#444)
- addLiquidity(otherHalf,newBalance) (#544)
Reentrancy in ACCToken.transferFrom(address,address,uint256) (#369-373):
External calls:
- _transfer(sender,recipient,amount) (#370)
- uniswapV2Router.addLiquidity(address(this),USDT,tokenAmount,usdtAmount,0,0,holder,block.timestamp) (#568-577)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,to,block.timestamp) (#555-561)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#444)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#371)
Apply the check-effects-interactions pattern.

Additional information: link

Address.isContract(address) (#93-98) uses assembly
- INLINE ASM (#96)
Address._functionCallWithValue(address,bytes,uint256,string) (#128-149) uses assembly
- INLINE ASM (#141-144)
ACCToken._isContract(address) (#607-611) uses assembly
- INLINE ASM (#609)
Do not use evm assembly.

Additional information: link

Address._functionCallWithValue(address,bytes,uint256,string) (#128-149) is never used and should be removed
Address.functionCall(address,bytes) (#108-110) is never used and should be removed
Address.functionCall(address,bytes,string) (#113-115) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#118-120) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#123-126) is never used and should be removed
Address.isContract(address) (#93-98) is never used and should be removed
Address.sendValue(address,uint256) (#100-106) is never used and should be removed
Context._msgData() (#85-88) is never used and should be removed
SafeMath.mod(uint256,uint256) (#69-71) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#74-77) is never used and should be removed
SafeMath.mul(uint256,uint256) (#47-54) is never used and should be removed
TransferHelper.safeApprove(address,address,uint256) (#225-229) is never used and should be removed
TransferHelper.safeTransferFrom(address,address,address,uint256) (#237-241) is never used and should be removed
Remove unused functions.

Additional information: link

Low level call in Address.sendValue(address,uint256) (#100-106):
- (success) = recipient.call{value: amount}() (#104)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#128-149):
- (success,returndata) = target.call{value: weiValue}(data) (#132)
Low level call in TransferHelper.safeApprove(address,address,uint256) (#225-229):
- (success,data) = token.call(abi.encodeWithSelector(0x095ea7b3,to,value)) (#227)
Low level call in TransferHelper.safeTransfer(address,address,uint256) (#231-235):
- (success,data) = token.call(abi.encodeWithSelector(0xa9059cbb,to,value)) (#233)
Low level call in TransferHelper.safeTransferFrom(address,address,address,uint256) (#237-241):
- (success,data) = token.call(abi.encodeWithSelector(0x23b872dd,from,to,value)) (#239)
Low level call in TransferHelper.safeTransferETH(address,uint256) (#243-246):
- (success) = to.call{value: value}(new bytes(0)) (#244)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

Variable Ownable._owner (#153) is not in mixedCase
Function IUniswapV2Router02.WETH() (#182) is not in mixedCase
Parameter ACCToken.setIsSell(bool)._isSell (#416) is not in mixedCase
Parameter ACCToken.setIsBuy(bool)._isBuy (#420) is not in mixedCase
Variable ACCToken._maxTxAmount (#284) is not in mixedCase
Variable ACCToken.USDT (#286) is not in mixedCase
Constant ACCToken.burnAddress (#290) is not in UPPER_CASE_WITH_UNDERSCORES
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#86)" inContext (#80-89)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Variable IUniswapV2Router02.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#209) is too similar to IUniswapV2Router02.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#210)
Prevent variables from having similar names.

Additional information: link

ACCToken.slitherConstructorVariables() (#249-615) uses literals with too many digits:
- _tTotal = 7900000 * 10 ** 18 (#261)
ACCToken.slitherConstructorVariables() (#249-615) uses literals with too many digits:
- _maxTxAmount = 100000 * 10 ** 18 (#284)
ACCToken.slitherConstructorConstantVariables() (#249-615) uses literals with too many digits:
- burnAddress = address(0x000000000000000000000000000000000000dEaD) (#290)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

Contract has 1% buy tax and 0% 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

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