ACK [AcknoLedger] Token

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

Contract ownership is not renounced (belongs to a wallet)

Reentrancy in AcknoLedgerToken.recoverToken(address,address,uint256) (AcknoLedgerToken.sol#86-95):
External calls:
- require(bool,string)(IERC20(token).transfer(destination,amount),Retrieve failed) (AcknoLedgerToken.sol#93)
Event emitted after the call(s):
- RecoverToken(token,destination,amount) (AcknoLedgerToken.sol#94)
Apply the check-effects-interactions pattern.

Additional information: link

ERC20Permit.permit(address,address,uint256,uint256,uint8,bytes32,bytes32) (ERC20Permit.sol#40-61) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp <= deadline,Permit: expired deadline) (ERC20Permit.sol#49)
UsingLiquidityProtectionService.passed(uint256) (UsingLiquidityProtectionService.sol#115-117) uses timestamp for comparisons
Dangerous comparisons:
- _timestamp < block.timestamp (UsingLiquidityProtectionService.sol#116)
Avoid relying on block.timestamp.

Additional information: link

ERC20Permit.constructor() (ERC20Permit.sol#19-34) uses assembly
- INLINE ASM (ERC20Permit.sol#21-23)
Do not use evm assembly.

Additional information: link

Different versions of Solidity is used:
- Version used: ['0.8.9', '^0.8.0']
- 0.8.9 (AcknoLedgerToken.sol#3)
- ^0.8.0 (Context.sol#3)
- ^0.8.0 (Counters.sol#3)
- ^0.8.0 (ERC20.sol#3)
- 0.8.9 (ERC20Permit.sol#3)
- ^0.8.0 (IERC20.sol#3)
- ^0.8.0 (IERC20Metadata.sol#3)
- 0.8.9 (IERC2612Permit.sol#3)
- 0.8.9 (IPLPS.sol#2)
- ^0.8.0 (Ownable.sol#3)
- 0.8.9 (UniswapV2Library.sol#2)
- 0.8.9 (UniswapV3Library.sol#2)
- 0.8.9 (UsingLiquidityProtectionService.sol#2)
Use one Solidity version.

Additional information: link

Context._msgData() (Context.sol#20-22) is never used and should be removed
Counters.decrement(Counters.Counter) (Counters.sol#31-37) is never used and should be removed
Counters.reset(Counters.Counter) (Counters.sol#39-41) is never used and should be removed
ERC20._burn(address,uint256) (ERC20.sol#273-288) is never used and should be removed
UsingLiquidityProtectionService.ProtectionSwitch_block(uint256) (UsingLiquidityProtectionService.sol#107-109) is never used and should be removed
UsingLiquidityProtectionService.ProtectionSwitch_manual() (UsingLiquidityProtectionService.sol#99-101) is never used and should be removed
UsingLiquidityProtectionService.blockPassed(uint256) (UsingLiquidityProtectionService.sol#111-113) is never used and should be removed
UsingLiquidityProtectionService.counterToken() (UsingLiquidityProtectionService.sol#47-49) is never used and should be removed
UsingLiquidityProtectionService.protectionAdminCheck() (UsingLiquidityProtectionService.sol#43) is never used and should be removed
UsingLiquidityProtectionService.protectionChecker() (UsingLiquidityProtectionService.sol#53-55) is never used and should be removed
UsingLiquidityProtectionService.token_balanceOf(address) (UsingLiquidityProtectionService.sol#42) is never used and should be removed
UsingLiquidityProtectionService.token_transfer(address,address,uint256) (UsingLiquidityProtectionService.sol#41) is never used and should be removed
UsingLiquidityProtectionService.uniswapFactory() (UsingLiquidityProtectionService.sol#46) is never used and should be removed
UsingLiquidityProtectionService.uniswapVariety() (UsingLiquidityProtectionService.sol#44) is never used and should be removed
UsingLiquidityProtectionService.uniswapVersion() (UsingLiquidityProtectionService.sol#45) is never used and should be removed
Remove unused functions.

Additional information: link

Pragma version0.8.9 (AcknoLedgerToken.sol#3) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
Pragma version^0.8.0 (Context.sol#3) allows old versions
Pragma version^0.8.0 (Counters.sol#3) allows old versions
Pragma version^0.8.0 (ERC20.sol#3) allows old versions
Pragma version0.8.9 (ERC20Permit.sol#3) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
Pragma version^0.8.0 (IERC20.sol#3) allows old versions
Pragma version^0.8.0 (IERC20Metadata.sol#3) allows old versions
Pragma version0.8.9 (IERC2612Permit.sol#3) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
Pragma version0.8.9 (IPLPS.sol#2) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
Pragma version^0.8.0 (Ownable.sol#3) allows old versions
Pragma version0.8.9 (UniswapV2Library.sol#2) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
Pragma version0.8.9 (UniswapV3Library.sol#2) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
Pragma version0.8.9 (UsingLiquidityProtectionService.sol#2) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
solc-0.8.9 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

Function AcknoLedgerToken.token_transfer(address,address,uint256) (AcknoLedgerToken.sol#34-36) is not in mixedCase
Parameter AcknoLedgerToken.token_transfer(address,address,uint256)._from (AcknoLedgerToken.sol#34) is not in mixedCase
Parameter AcknoLedgerToken.token_transfer(address,address,uint256)._to (AcknoLedgerToken.sol#34) is not in mixedCase
Parameter AcknoLedgerToken.token_transfer(address,address,uint256)._amount (AcknoLedgerToken.sol#34) is not in mixedCase
Function AcknoLedgerToken.token_balanceOf(address) (AcknoLedgerToken.sol#37-39) is not in mixedCase
Parameter AcknoLedgerToken.token_balanceOf(address)._holder (AcknoLedgerToken.sol#37) is not in mixedCase
Parameter AcknoLedgerToken.setGovernance(address)._governance (AcknoLedgerToken.sol#73) is not in mixedCase
Variable ERC20Permit.DOMAIN_SEPARATOR (ERC20Permit.sol#17) is not in mixedCase
Function IPLPS.LiquidityProtection_beforeTokenTransfer(address,address,address,uint256) (IPLPS.sol#5-6) is not in mixedCase
Function UsingLiquidityProtectionService.LiquidityProtection_setLiquidityProtectionService(IPLPS) (UsingLiquidityProtectionService.sol#37-39) is not in mixedCase
Parameter UsingLiquidityProtectionService.LiquidityProtection_setLiquidityProtectionService(IPLPS)._plps (UsingLiquidityProtectionService.sol#37) is not in mixedCase
Function UsingLiquidityProtectionService.token_transfer(address,address,uint256) (UsingLiquidityProtectionService.sol#41) is not in mixedCase
Function UsingLiquidityProtectionService.token_balanceOf(address) (UsingLiquidityProtectionService.sol#42) is not in mixedCase
Function UsingLiquidityProtectionService.LiquidityProtection_beforeTokenTransfer(address,address,uint256) (UsingLiquidityProtectionService.sol#61-68) is not in mixedCase
Parameter UsingLiquidityProtectionService.LiquidityProtection_beforeTokenTransfer(address,address,uint256)._from (UsingLiquidityProtectionService.sol#61) is not in mixedCase
Parameter UsingLiquidityProtectionService.LiquidityProtection_beforeTokenTransfer(address,address,uint256)._to (UsingLiquidityProtectionService.sol#61) is not in mixedCase
Parameter UsingLiquidityProtectionService.LiquidityProtection_beforeTokenTransfer(address,address,uint256)._amount (UsingLiquidityProtectionService.sol#61) is not in mixedCase
Parameter UsingLiquidityProtectionService.revokeBlocked(address[],address)._holders (UsingLiquidityProtectionService.sol#70) is not in mixedCase
Parameter UsingLiquidityProtectionService.revokeBlocked(address[],address)._revokeTo (UsingLiquidityProtectionService.sol#70) is not in mixedCase
Function UsingLiquidityProtectionService.LiquidityProtection_unblock(address[]) (UsingLiquidityProtectionService.sol#85-89) is not in mixedCase
Parameter UsingLiquidityProtectionService.LiquidityProtection_unblock(address[])._holders (UsingLiquidityProtectionService.sol#85) is not in mixedCase
Function UsingLiquidityProtectionService.ProtectionSwitch_manual() (UsingLiquidityProtectionService.sol#99-101) is not in mixedCase
Function UsingLiquidityProtectionService.ProtectionSwitch_timestamp(uint256) (UsingLiquidityProtectionService.sol#103-105) is not in mixedCase
Parameter UsingLiquidityProtectionService.ProtectionSwitch_timestamp(uint256)._timestamp (UsingLiquidityProtectionService.sol#103) is not in mixedCase
Function UsingLiquidityProtectionService.ProtectionSwitch_block(uint256) (UsingLiquidityProtectionService.sol#107-109) is not in mixedCase
Parameter UsingLiquidityProtectionService.ProtectionSwitch_block(uint256)._block (UsingLiquidityProtectionService.sol#107) is not in mixedCase
Parameter UsingLiquidityProtectionService.blockPassed(uint256)._block (UsingLiquidityProtectionService.sol#111) is not in mixedCase
Parameter UsingLiquidityProtectionService.passed(uint256)._timestamp (UsingLiquidityProtectionService.sol#115) is not in mixedCase
Parameter UsingLiquidityProtectionService.not(bool)._condition (UsingLiquidityProtectionService.sol#119) is not in mixedCase
Parameter UsingLiquidityProtectionService.feeToUint24(UsingLiquidityProtectionService.UniswapV3Fees)._fee (UsingLiquidityProtectionService.sol#123) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

UsingLiquidityProtectionService.HUNDRED_PERCENT (UsingLiquidityProtectionService.sol#11) is never used in AcknoLedgerToken (AcknoLedgerToken.sol#15-96)
UsingLiquidityProtectionService.QUICKSWAP (UsingLiquidityProtectionService.sol#14) is never used in AcknoLedgerToken (AcknoLedgerToken.sol#15-96)
UsingLiquidityProtectionService.SUSHISWAP (UsingLiquidityProtectionService.sol#15) is never used in AcknoLedgerToken (AcknoLedgerToken.sol#15-96)
Remove unused state variables.

Additional information: link

symbol() should be declared external:
- ERC20.symbol() (ERC20.sol#68-70)
decimals() should be declared external:
- ERC20.decimals() (ERC20.sol#85-87)
totalSupply() should be declared external:
- ERC20.totalSupply() (ERC20.sol#92-94)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (ERC20.sol#111-114)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (ERC20.sol#119-121)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (ERC20.sol#130-133)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (ERC20.sol#148-162)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (ERC20.sol#176-179)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (ERC20.sol#195-203)
permit(address,address,uint256,uint256,uint8,bytes32,bytes32) should be declared external:
- ERC20Permit.permit(address,address,uint256,uint256,uint8,bytes32,bytes32) (ERC20Permit.sol#40-61)
nonces(address) should be declared external:
- ERC20Permit.nonces(address) (ERC20Permit.sol#66-68)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (Ownable.sol#53-55)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (Ownable.sol#61-64)
Use the external attribute for functions never called from the contract.

Additional information: link

UsingLiquidityProtectionService.revokeBlocked(address[],address) (UsingLiquidityProtectionService.sol#70-83) has external calls inside a loop: lps().isBlocked(pool,holder) (UsingLiquidityProtectionService.sol#78)
UsingLiquidityProtectionService.LiquidityProtection_beforeTokenTransfer(address,address,uint256) (UsingLiquidityProtectionService.sol#61-68) has external calls inside a loop: lps().LiquidityProtection_beforeTokenTransfer(getLiquidityPool(),_from,_to,_amount) (UsingLiquidityProtectionService.sol#66)
Favor pull over push strategy for external calls.

Additional information: link

Token is deployed only at one blockchain

Unable to find token contract audit

Unable to find audit link on the website

Unable to find token on CoinHunt

Additional information: link

Token has relatively low CoinGecko rank

Young tokens have high risks of price dump / death