Fix concurrent writes for UpdateContainerStats

danlenar · thaJeztah · commit 9f650143fafb · 2023-07-14T11:28:00.000+02:00
Signed-off-by: Daniel Lenar <dlenar@vailsys.com> (cherry picked from commit a48dbef) Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
diff --git a/pkg/cri/store/sandbox/sandbox.go b/pkg/cri/store/sandbox/sandbox.go
@@ -125,8 +125,8 @@ func (s *Store) List() []Sandbox {
 }
 
 func (s *Store) UpdateContainerStats(id string, newContainerStats *stats.ContainerStats) error {
-	s.lock.RLock()
-	defer s.lock.RUnlock()
+	s.lock.Lock()
+	defer s.lock.Unlock()
 	id, err := s.idIndex.Get(id)
 	if err != nil {
 		if err == truncindex.ErrNotExist {
