store corruption in fetching rollapp packets doesn't return error

[omritoptix]

Currently there are 2 places in rollapp packets where when we fail to unmarshal a specific packet we skip it and continue.
This may cause a corrupted state store which will continue to pile vs having the caller decide if the process should revert due to it.

the relevant places:
GetAllRollappPackets
ListRollappPacketsByStatus

The relevant code parts:

err := k.cdc.Unmarshal(iterator.Value(), &val)
		if err != nil {
                         ...
			continue
		}

[danwt]

Similar cases

dymension/x/eibc/keeper/keeper.go

Lines 154 to 158 in a3c7739

	}
	err := osmoutils.ApplyFuncIfNoError(ctx, wrapFn)
	if err != nil {
	k.Logger(ctx).Error("error unmarshalling demand order", "error", err.Error())
	continue