From b84d6fed2c280a4c97dcfbcbd7e472333c12c1e8 Mon Sep 17 00:00:00 2001 From: Ori Newman Date: Wed, 25 May 2022 01:16:54 +0300 Subject: [PATCH] broadcast is using `refreshUTXOs`, so it should be protected by lock (#2061) --- cmd/kaspawallet/daemon/server/broadcast.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/cmd/kaspawallet/daemon/server/broadcast.go b/cmd/kaspawallet/daemon/server/broadcast.go index 2b93a6d7f..13b674d87 100644 --- a/cmd/kaspawallet/daemon/server/broadcast.go +++ b/cmd/kaspawallet/daemon/server/broadcast.go @@ -13,9 +13,6 @@ import ( ) func (s *server) Broadcast(_ context.Context, request *pb.BroadcastRequest) (*pb.BroadcastResponse, error) { - s.lock.Lock() - defer s.lock.Unlock() - txIDs, err := s.broadcast(request.Transactions, request.IsDomain) if err != nil { return nil, err @@ -25,6 +22,8 @@ func (s *server) Broadcast(_ context.Context, request *pb.BroadcastRequest) (*pb } func (s *server) broadcast(transactions [][]byte, isDomain bool) ([]string, error) { + s.lock.Lock() + defer s.lock.Unlock() txIDs := make([]string, len(transactions)) var tx *externalapi.DomainTransaction