KonnektVPN - Audit

In the contract `ClaimTokens` the role `owner`/`ADMIN_ROLE` has authority over the functions shown in the diagram below. Any compromise to the `owner`/`ADMIN_ROLE` account may allow the hacker to take advantage of this authority to transfer tokens to the specified addresses.  In the contract `KPNToken` the role `owner`/`ADMIN_ROLE` has authority over the functions shown in the diagram below. Any compromise to the `owner`/`ADMIN_ROLE` account may allow the hacker to take advantage of this authority to mint tokens.  In the contract `LockTokens` the role `owner`/`ADMIN_ROLE` has authority over the functions shown in the diagram below. Any compromise to the `owner`/`ADMIN_ROLE` account may allow the hacker to take advantage of this authority to change the reward rate.  Introduce a centralization function in remediation via the commit [198ef469111f4b1dfcccf274f499667e7de1d200](https://github.com/VeniGroup/KPNcontracts/tree/198ef469111f4b1dfcccf274f499667e7de1d200): - withdrawUSDTAdmin(): to withdraw USDT from the contract, excluding the amounts reserved in usdtVault.

The contract is serving as the underlying entity to interact with one or more third party protocols. The scope of the audit treats third party entities as black boxes and assumes their functional correctness. However, in the real world, third parties can be compromised and this may lead to lost or stolen assets. In addition, upgrades of third parties can possibly create severe impacts, such as increasing fees of third parties, migrating to new LP pools, etc. ```solidity=15 IMultiSigWallet public multiSigWallet; ``` - The contract `ClaimTokens` interacts with third party contract with `IMultiSigWallet` interface via `multiSigWallet`. --- ```solidity=16 IMultiSigWallet public multiSigWallet; ``` - The contract `KPNToken` interacts with third party contract with `IMultiSigWallet` interface via `multiSigWallet`. --- ```solidity=13 IMultiSigWallet public multiSigWallet; ``` - The contract `LockTokens` interacts with third party contract with `IMultiSigWallet` interface via `multiSigWallet`.