POST AUDIT CONCLUSION
Fixing the Findings
| Sl. No. | Name | Severity |
|---|---|---|
| C-01 | Protocol fees are not correctly implemented | Fixed |
| H-01 | Usage of an incorrect version of Ownable library can potentially malfunction all onlyOwner functions | Fixed |
| H-02 | Signature malleability of EVM's ecrecover | Fixed |
| H-03 | Decimals value can be manipulated | Fixed |
| M-01 | No Storage Gap for Upgradeable Contracts | Fixed |
| M-02 | Possible DOS (out-of-gas) on for loops | Fixed |
| L-01 | call() should be used instead of transfer() on an address payable | Fixed |
| L-02 | Use SafeTransfer instead of transfer | Fixed |
| L-03 | Usage of an incorrect version of SafeERC20 library can potentially malfunction all ERC20 functions | Fixed |
| L-04 | Front-runnable Initializers | Acknowledged |
| L-05 | Floating Pragma Solidity Version | Acknowledged |
| L-06 | Missing event for important parameter change | Fixed |
| I-01 | Missing Revert Message in the withdraw function | Fixed |
| I-02 | Unused internal function | Fixed |
| I-03 | The require check in executeTx needs to be updated | Fixed |
| G-01 | bytes4 conversion can be directly done | Fixed |