client: make ListSwaps thread safe

bhandras · bhandras · commit 62f654eef1cd · 2020-05-04T23:18:15.000+02:00
diff --git a/loopd/swapclient_server.go b/loopd/swapclient_server.go
@@ -255,6 +255,9 @@ func (s *swapClientServer) ListSwaps(_ context.Context,
 		err      error
 	)
 
+	s.swapsLock.Lock()
+	defer s.swapsLock.Unlock()
+
 	// We can just use the server's in-memory cache as that contains the
 	// most up-to-date state including temporary failures which aren't
 	// persisted to disk. The swaps field is a map, that's why we need an

Original file line number	Diff line number	Diff line change
`@@ -255,6 +255,9 @@ func (s *swapClientServer) ListSwaps(_ context.Context,`
`255`	`255`	`err error`
`256`	`256`	`)`
`257`	`257`
	`258`	`+ s.swapsLock.Lock()`
	`259`	`+ defer s.swapsLock.Unlock()`
	`260`	`+`
`258`	`261`	`// We can just use the server's in-memory cache as that contains the`
`259`	`262`	`// most up-to-date state including temporary failures which aren't`
`260`	`263`	`// persisted to disk. The swaps field is a map, that's why we need an`