sherlock-audit / 2023-09-perennial-judging

0 stars 0 forks source link

[Post-Contest] Incorrect boolean logic in Order.liquidityCheckApplicable() #62

Closed kbrizzle closed 1 year ago

kbrizzle commented 1 year ago

This check is intended to skip the two liquidity-cased invariant checks: MarketEfficiencyUnderLimitError and MarketInsufficientLiquidityError, under three conditions:

Under the current logic several cases which are not listed above will fail this check, skipping the invariant check mentioned including:

kbrizzle commented 1 year ago

Fixed by: https://github.com/equilibria-xyz/perennial-v2/pull/117.

panprog commented 1 year ago

Fixed