For the complete documentation index, see llms.txt. This page is also available as Markdown.
01/10/2024 - BE - Custody internal state for `Assets` reports too much locked collateral
What
When a user increase a long or short position, the whole size of the position is re-added to the Asset struct, we keep another value LockedAmount in the Custody that is increased properly.
Impact
Every-time someone increase an existing positon, the Assets.locked of the Custody drifts up compared to positionAccounting.lockedAmount.
In term of impact, this affected the borrow rate calculation: users may have paid a higher borrow rate on positions.
Overall this very marginally increased the AUM due to taking more fees (
Resolution
Add integration tests that triggers the issue
Fix the calculation in the code: add only the increased size and not the whole amount in the code.
Add an one time patch IX to the program that doctors the existing values of Assets.Locked, aligning them with the correct LockedAmount kept in the stats
Add related integration tests that check the patch IX
