Biconomy Hyphen 2.0 contest - csanuragjain's results

QA report of the warden: csanuragjain

Issues: https://github.com/code-423n4/2022-03-biconomy-findings/issues/22

ExecutorManager.sol

1. getAllExecutors/addExecutor/removeExecutor function can be declared external instead of public

TokenManager.sol

1. changeFee function: add modifier tokenChecks which will validate the passed token address
2. setDepositConfig function: check if tokenAddresses is non zero and supported
3. getEquilibriumFee/getMaxFee/getTransferConfig function can be declared external instead of public

LiquidityPool.sol

1. tokenManager variable can be immutable
2. setBaseGas function: add a check to see new gas is greater than base gas which was initially set in constructor ie gas>21000