Mimo August 2022 contest - ak1's results

Using the exact solidity version is recommended instead of using like this pragma solidity >=0.8.4; Use this way pragma solidity 0.8.4

https://github.com/code-423n4/2022-08-mimo/blob/eb1a5016b69f72bc1e4fd3600a65e908bd228f13/contracts/proxy/MIMOProxy.sol#L30

minGasReserve = 5_000; - Hardcoding minmum gas fee will not be safe always. in future gas fee range can vary.

It is recommended to set gas using the variable while initializing.

https://github.com/code-423n4/2022-08-mimo/blob/eb1a5016b69f72bc1e4fd3600a65e908bd228f13/contracts/proxy/MIMOProxy.sol#L54

function execute(address target, bytes calldata data) public payable override returns (bytes memory response)

It could be safe always to include Non-ReEntrancy modifier in above function.

https://github.com/code-423n4/2022-08-mimo/blob/eb1a5016b69f72bc1e4fd3600a65e908bd228f13/contracts/proxy/MIMOProxy.sol#L67

if (target.code.length == 0) { revert CustomErrors.TARGET_INVALID(target); }

Checking contract code length may not be safe.

https://github.com/code-423n4/2022-08-mimo/blob/eb1a5016b69f72bc1e4fd3600a65e908bd228f13/contracts/proxy/MIMOProxy.sol#L127-L132

Its best practice to check both array length matches before start looping.

if (msg.sender != owner) {
 revert CustomErrors.NOT_OWNER(owner, msg.sender);
}
bytes[] memory results = new bytes[](data.length);
for (uint256 i = 0; i < targets.length; i++) {```

If  `targets` and `data` array length are equal then continue further operation.

Instead of using Errors CustomErrors, it is recommended to use all erros in single place which might be useful for future reference.