From 530f1ac53cf4057de39225a21e910f10fb8ab62f Mon Sep 17 00:00:00 2001 From: Rasmus Dahlberg Date: Tue, 23 Mar 2021 15:38:14 +0100 Subject: fixed copy bug The get-entries response was copied incorrectly. Therefore, the returned range contained the same (last) StItem repeatedly. Fixed by ensuring that `&item` refers to a newly allocated `item` in the loop. --- client/client.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'client/client.go') diff --git a/client/client.go b/client/client.go index 1a3470f..ba81f4d 100644 --- a/client/client.go +++ b/client/client.go @@ -198,7 +198,8 @@ func (c *Client) GetEntries(ctx context.Context, start, end uint64) ([]*types.St return nil, fmt.Errorf("Unmarshal: %v", err) } ret := make([]*types.StItem, 0, len(list.Items)) - for _, item := range list.Items { + for i, _ := range list.Items { + item := list.Items[i] if got, want := item.Format, types.StFormatSignedChecksumV1; got != want { return nil, fmt.Errorf("unexpected StItem format: %v", got) } -- cgit v1.2.3