Merge pull request #2223 from tedyu/sandbox-anno

vc: Sandbox#Annotations should use annotationsLock read lock
kata-containers · Nov 20, 2019 · 1f71114 · 1f71114
2 parents 48c8d66 + 1afad1c
commit 1f71114
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/virtcontainers/sandbox.go b/virtcontainers/sandbox.go
@@ -225,6 +225,9 @@ func (s *Sandbox) Logger() *logrus.Entry {
 
 // Annotations returns any annotation that a user could have stored through the sandbox.
 func (s *Sandbox) Annotations(key string) (string, error) {
+	s.annotationsLock.RLock()
+	defer s.annotationsLock.RUnlock()
+
 	value, exist := s.config.Annotations[key]
 	if !exist {
 		return "", fmt.Errorf("Annotations key %s does not exist", key)