fixing negative custody amount issue (#813)

elys-network · Sep 24, 2024 · 5c8fe66 · 5c8fe66
1 parent c219ea2
commit 5c8fe66
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/x/perpetual/keeper/repay.go b/x/perpetual/keeper/repay.go
@@ -168,7 +168,8 @@ func (k Keeper) Repay(ctx sdk.Context, mtp *types.MTP, pool *types.Pool, ammPool
 	}
 
 	mtp.Custody = mtp.Custody.Sub(amount)
-	if mtp.Custody.IsZero() {
+	// This is for accounting purposes, mtp.Custody gets reduced by borrowInterestPaymentCustody and funding fee. so msg.Amount is greater than mtp.Custody here. So if it's negative it should be closed
+	if mtp.Custody.IsZero() || mtp.Custody.IsNegative() {
 		err = k.DestroyMTP(ctx, mtp.GetAccountAddress(), mtp.Id)
 		if err != nil {
 			return err