Incompletely Fixed Error Handling that can Lead to n-No-cost Minting

[Hellobloc]

Intro

In the following code content, the error handling of x.bankKeeper.SendCoins is missing, which will result in a malicious user being able to set the status without cost.

ibcswap/modules/apps/100-atomic-swap/keeper/msg_server.go

Line 122 in 3aa2b1b

k.bankKeeper.SendCoins(ctx, escrowAddr, receiver, sdk.NewCoins(order.Maker.SellToken))

REF:
#8