From a79ed8c188b0b086ad389afb6ede0eeb875f452d Mon Sep 17 00:00:00 2001 From: "mergify[bot]" <37929162+mergify[bot]@users.noreply.github.com> Date: Mon, 20 Jan 2025 13:55:15 +0100 Subject: [PATCH] chore: Review feedback (backport #23465) (#23466) Co-authored-by: Alexander Peters Co-authored-by: Julien Robert --- x/staking/keeper/genesis.go | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/x/staking/keeper/genesis.go b/x/staking/keeper/genesis.go index 6ea1cd81dc..b6a775f1a6 100644 --- a/x/staking/keeper/genesis.go +++ b/x/staking/keeper/genesis.go @@ -361,7 +361,7 @@ func (k Keeper) ExportGenesis(ctx context.Context) (*types.GenesisState, error) } var initConsAddrs, rotatedConsAddrs []types.RotatedConsensusAddresses - err = k.ConsAddrToValidatorIdentifierMap.Walk(ctx, nil, func(newBz []byte, initBz []byte) (stop bool, err error) { + err = k.ConsAddrToValidatorIdentifierMap.Walk(ctx, nil, func(newBz, initBz []byte) (stop bool, err error) { oldAddr, err2 := k.consensusAddressCodec.BytesToString(initBz) if err2 != nil { return true, fmt.Errorf("decode initial address %X: %w", initBz, err2) @@ -373,7 +373,11 @@ func (k Keeper) ExportGenesis(ctx context.Context) (*types.GenesisState, error) initConsAddrs = append(initConsAddrs, types.RotatedConsensusAddresses{OldAddress: oldAddr, NewAddress: newAddr}) return false, nil }) - err = k.OldToNewConsAddrMap.Walk(ctx, nil, func(oldBz []byte, newBz []byte) (stop bool, err error) { + if err != nil { + return nil, fmt.Errorf("cons addrs to validator ident: %w", err) + } + + err = k.OldToNewConsAddrMap.Walk(ctx, nil, func(oldBz, newBz []byte) (stop bool, err error) { oldAddr, err2 := k.consensusAddressCodec.BytesToString(oldBz) if err2 != nil { return true, fmt.Errorf("decode old address %X: %w", oldBz, err2) @@ -385,6 +389,9 @@ func (k Keeper) ExportGenesis(ctx context.Context) (*types.GenesisState, error) rotatedConsAddrs = append(rotatedConsAddrs, types.RotatedConsensusAddresses{OldAddress: oldAddr, NewAddress: newAddr}) return false, nil }) + if err != nil { + return nil, fmt.Errorf("old to new cons addrs: %w", err) + } return &types.GenesisState{ Params: params,