Liquidity: 예비 계좌로 코인을 보내는 문제 처리

버그 요약

누군가가 풀의 예비 계좌로 코인을 보내면 다음과 같은 몇 가지 문제가 발생할 수 있습니다.

• 예비 코인의 한쪽 면에 잔액이 0인 풀로 인해 풀 가격 계산이 잘못됨
  
  
  
  • 풀에서 모든 코인을 인출한 후 하나의 예비 코인만 예비 계좌로 직접 보내는 방식
  
  
• 예비 코인 비율 계산 시 예금 로직의 패닉(위에서 설명한 풀로 0으로 나누기 오류)
• ...

버전

https://github.com/tendermint/liquidity/releases/tag/v1.2.5

번식 단계

이 테스트를 실행하면 패닉이 발생합니다.

func TestSwapHalfZeroReserve(t *testing.T) {
    simapp, ctx := createTestInput()
    params := simapp.LiquidityKeeper.GetParams(ctx)
    pool, addr, err := createPool(simapp, ctx, sdk.NewInt(1000000), sdk.NewInt(1000000), DenomX, DenomY)
    require.NoError(t, err)
    pc := simapp.BankKeeper.GetBalance(ctx, addr, pool.PoolCoinDenom)
    _, err = simapp.LiquidityKeeper.WithdrawLiquidityPoolToBatch(ctx, types.NewMsgWithdrawWithinBatch(addr, pool.Id, pc))
    require.NoError(t, err)
    liquidity.BeginBlocker(ctx, simapp.LiquidityKeeper)
    liquidity.EndBlocker(ctx, simapp.LiquidityKeeper)
    require.True(t, simapp.BankKeeper.GetBalance(ctx, pool.GetReserveAccount(), DenomX).IsZero())
    require.True(t, simapp.BankKeeper.GetBalance(ctx, pool.GetReserveAccount(), DenomY).IsZero())
    err = simapp.BankKeeper.SendCoins(ctx, addr, pool.GetReserveAccount(), sdk.NewCoins(sdk.NewInt64Coin(DenomX, 10000)))
    require.NoError(t, err)
    _, err = simapp.LiquidityKeeper.SwapLiquidityPoolToBatch(ctx,
        types.NewMsgSwapWithinBatch(
            addr, pool.Id, types.DefaultSwapTypeId, sdk.NewInt64Coin(DenomX, 1000), DenomY, sdk.MustNewDecFromStr("1.0"), params.SwapFeeRate), 0)
    require.NoError(t, err)
    liquidity.BeginBlocker(ctx, simapp.LiquidityKeeper)
    liquidity.EndBlocker(ctx, simapp.LiquidityKeeper)
}

관리자용

• [x] 중복되지 않은 문제
• [x] 적절한 라벨 적용
• [ ] 적절한 기여자 태그
• [ ] 기여자 지정/자체 지정