<!-- canonical: https://0xsimao.com/findings/more-markets-typo-loop-max-deposit -->

# Typo in LoopStrategy::_maxDeposit()

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

Finding 3S-MORE-N03 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

LoopStrategy::_maxDeposit() has as a comment totolSuppliable, which has a typo.

### Recommendation

Replace totolSuppliable with totalSuppliable.

### Status

Addressed in [#40132e8](https://github.com/MoreLabsXYZ/MORE-Optimizers/commit/40132e861b9f222451e3fbee757db50c9ac313b1).

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

---

Related findings:

- [`PreDepositVault::maxDeposit/Mint()` are missing `maxDepositLimit` as per the ERC4626 spec](https://0xsimao.com/findings/gaib-deposit-mint-erc4626-spec): GAIB Pre-Vaults
- [`maxDeposit` doesn't comply with ERC-4626](https://0xsimao.com/findings/pooltogether-the-prize-layer-for-defi-deposit-comply-erc-4626): PoolTogether Prize Layer
- [MapleSkyStrategy::assetsUnderManagement() uses maxWithdraw(), which may return 0](https://0xsimao.com/findings/maple-finance-iv-sky-management-withdraw-return): Maple Cash Strategies
