fix(staking): avoid the double iterator close in staking (backport #2…

…1823) (#21824)

Co-authored-by: cool-developer <51834436+cool-develope@users.noreply.github.com>

x/staking/keeper/cons_pubkey.go

@@ -258,8 +258,7 @@ func (k Keeper) GetBlockConsPubKeyRotationHistory(ctx context.Context) ([]types.
 	if err != nil {
 		return nil, err
 	}
-	defer iterator.Close()
-
+	// iterator would be closed in the CollectValues
 	return indexes.CollectValues(ctx, k.RotationHistory, iterator)
 }