Sublime contest - 0v3rf10w's results

M-01:

consistently use safeTransfer instead of transfer

LenderPool.start(uint256,address) (PooledCreditLine/LenderPool.sol#312-330) ignores return value by IERC20(_borrowAsset).transfer(_to,_fee) (PooledCreditLine/LenderPool.sol#327)

Consider to use safeTransfer

L-01:

lack of zero check

constructor(address _lenderPool, address _usdc) {
        lenderPool = _lenderPool;
        USDC = _usdc;
    }

pooledcreditline.sol L#592

G-01:

can break && in require statements to seperate statements(two seperate require statements) to save gas

require(_request.borrowAsset != address(0) && _request.collateralAsset != address(0), 'R4')

to

require(_request.borrowAsset != address(0) , 'R4')
require(_request.collateralAsset != address(0), 'R4')

pooledcreditline.sol L#642