fix: uses of copy (#62)

Co-authored-by: Tomasz Zdybał <tomek@zdybal.lap.pl>

Author: facundomedica

proxy/grpc/client.go

@@ -61,7 +61,7 @@ func (c *Client) InitChain(ctx context.Context, genesisTime time.Time, initialHe
 		return types.Hash{}, 0, err
 	}
 
-	var stateRoot types.Hash
+	stateRoot := make([]byte, len(resp.StateRoot))
 	copy(stateRoot[:], resp.StateRoot)
 
 	return stateRoot, resp.MaxBytes, nil
@@ -99,7 +99,7 @@ func (c *Client) ExecuteTxs(ctx context.Context, txs []types.Tx, blockHeight uin
 		return types.Hash{}, 0, err
 	}
 
-	var updatedStateRoot types.Hash
+	updatedStateRoot := make([]byte, len(resp.UpdatedStateRoot))
 	copy(updatedStateRoot[:], resp.UpdatedStateRoot)
 
 	return updatedStateRoot, resp.MaxBytes, nil

proxy/grpc/client_server_test.go

@@ -56,10 +56,11 @@ func TestClientServer(t *testing.T) {
 		chainID := "test-chain"
 
 		// initialize a new Hash with a fixed size
-		expectedStateRoot := make([]byte, 32)
+		expectedStateRoot := types.Hash(make([]byte, 32))
 		copy(expectedStateRoot, []byte{1, 2, 3})
-		var stateRootHash types.Hash
+		stateRootHash := types.Hash(make([]byte, 32))
 		copy(stateRootHash[:], expectedStateRoot)
+		assert.Equal(t, expectedStateRoot, stateRootHash[:])
 
 		expectedMaxBytes := uint64(1000000)
 

proxy/grpc/server.go

@@ -83,7 +83,7 @@ func (s *Server) ExecuteTxs(ctx context.Context, req *pb.ExecuteTxsRequest) (*pb
 		txs[i] = tx
 	}
 
-	var prevStateRoot types.Hash
+	prevStateRoot := make([]byte, len(req.PrevStateRoot))
 	copy(prevStateRoot[:], req.PrevStateRoot)
 
 	updatedStateRoot, maxBytes, err := s.exec.ExecuteTxs(