LI.FI contest - Tomio's results

1. > 0 is less efficient than != 0

Proof of Concept: https://github.com/code-423n4/2022-03-lifinance/blob/main/src/Facets/AnyswapFacet.sol#L92 https://github.com/code-423n4/2022-03-lifinance/blob/main/src/Facets/NXTPFacet.sol#L98

Recommended Mitigation Steps: Change to:

require(_postSwapBalance != 0, "ERR_INVALID_AMOUNT");

========================================================================

2. Use of uint8 in for loop increases gas costs

Proof of Concept: https://github.com/code-423n4/2022-03-lifinance/blob/main/src/Facets/HopFacet.sol#L48

Recommended Mitigation Steps: Change uint8 to uint256

========================================================================

3. changing i++ to ++i can save gas

Proof of Concept: https://github.com/code-423n4/2022-03-lifinance/blob/main/src/Facets/HopFacet.sol#L48 https://github.com/code-423n4/2022-03-lifinance/blob/main/src/Facets/DiamondLoupeFacet.sol#L24 https://github.com/code-423n4/2022-03-lifinance/blob/main/src/Facets/DexManagerFacet.sol#L33

Recommended Mitigation Steps: Change i++ to ++i

========================================================================

4. != 0 is a cheaper operation compared to > 0, when dealing with uint

Proof of Concept: https://github.com/code-423n4/2022-03-lifinance/blob/main/src/Libraries/LibDiamond.sol#L121 https://github.com/code-423n4/2022-03-lifinance/blob/main/src/Libraries/LibDiamond.sol#L189 https://github.com/code-423n4/2022-03-lifinance/blob/main/src/Libraries/LibDiamond.sol#L196 https://github.com/code-423n4/2022-03-lifinance/blob/main/src/Libraries/LibDiamond.sol#L212

Recommended Mitigation Steps: Replace > with ! =

========================================================================

5. the best way to use SafeERC20.function for gas opt

Proof of Concept: https://github.com/code-423n4/2022-03-lifinance/blob/main/src/Facets/WithdrawFacet.sol#L9

Recommended Mitigation Steps: by not declaring:

using SafeERC20 for IERC20;

and use:

SafeERC20.safeTransfer(IERC20(_assetAddress), sendTo, _amount);

========================================================================