Liquidity: Behandeln Sie Probleme beim Senden von Coins auf das Reservekonto

Zusammenfassung des Fehlers

Wenn jemand Coins auf das Reservekonto eines Pools sendet, kann dies einige Probleme verursachen, darunter:

• Ein Pool mit einem Nullbetrag auf einer Seite der Reservemünze, was zu einer falschen Berechnung des Poolpreises führt
  
  
  
  • Indem Sie alle Coins aus dem Pool abheben und dann nur einen Reservecoin direkt auf das Reservekonto senden
  
  
• Eine Panik in der Einzahlungslogik bei der Berechnung des Reserve-Coin-Verhältnisses (dividiere durch Null-Fehler bei einem oben beschriebenen Pool)
• ...

Ausführung

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

Schritte zum Reproduzieren

Das Ausführen dieses Tests verursacht eine Panik:

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)
}

Zur Verwendung durch Administratoren

• [x] Kein doppeltes Problem
• [x] Entsprechende Labels angebracht
• [ ] Geeignete Mitwirkende markiert
• [ ] Mitwirkender zugewiesen/selbst zugewiesen