<!-- canonical: https://0xsimao.com/findings/more-markets-params-market-unused-loop -->

# paramsOfTheMarket is unused in the LoopStrategy

Low/Info · Three Sigma · Lending · 2nd December 2024

Finding 3S-MORE-N06 of the MORE Optimizer security review.

- Protocol: https://www.more.markets/
- Report: /reports/more-optimizer
- Source: https://github.com/0xsimao/audits/blob/main/Three%20Sigma/2024-12-02-more-markets.pdf

---

### Description

paramsOfTheMarket is [unused](https://github.com/0x73696d616f/more-markets-loop-strategy/blob/main/src/LoopStrategy.sol#L58) in the LoopStrategy.

### Recommendation

Remove or deprecate the variable (comment or rename it).

### Status

Addressed in [#50a9248](https://github.com/MoreLabsXYZ/MORE-Optimizers/commit/50a9248448b00f5f98b12ec107bea31bbdbc3b53).

Disclosed by 0xSimao (https://0xsimao.com/).

---

Related findings:

- [TOFT in (m)TapiocaOft contracts can be stolen by calling removeCollateral() with a malicious removeParams.market](https://0xsimao.com/findings/tapioca-dao-toft-stolen-collateral-params): Tapioca DAO
