pfm can return nil; transfer_middleware should fall through if this i…

…s the case; fixes #1695
quicksilver-zone · Jul 12, 2024 · 2cd6952 · 2cd6952
1 parent c04a5dc
commit 2cd6952
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/x/interchainstaking/transfer_middleware.go b/x/interchainstaking/transfer_middleware.go
@@ -94,6 +94,10 @@ func (im TransferMiddleware) OnRecvPacket(
 	}
 
 	ack := im.app.OnRecvPacket(ctx, packet, relayer)
+	if ack == nil {
+		// transfer middleware can return nil
+		return nil
+	}
 	if ack.Success() {
 		_, found := im.keeper.GetZoneForWithdrawalAccount(ctx, data.Sender)
 		if found {