🔒 Seize strategy can create underflow

Open transmissions11 opened this issue 3 years ago • 0 comments

seize strategy is rly unsafe imo, consider removing. regardless, this line can cause underflow in totalHoldings.

should change to if (lockedProfit() > totalStrategyHoldings - strategyBalance)

https://github.com/Rari-Capital/vaults/blob/2edd5eb011553af0a9de78d18132fe07c8d841e8/src/Vault.sol#L856

Mar 04 '22 05:03 transmissions11