used defer to release the mutex

Signed-off-by: Shriram Sharma <shriram_sharma@intuit.com>

admiral/pkg/controller/common/types.go

@@ -177,10 +177,9 @@ func (s *SidecarEgressMap) Put(identity string, namespace string, fqdn string, c
 }
 
 func (s *SidecarEgressMap) Get(key string) map[string]SidecarEgress {
+	defer s.mutex.Unlock()
 	s.mutex.Lock()
-	val := s.cache[key]
-	s.mutex.Unlock()
-	return val
+	return s.cache[key]
 }
 
 func (s *SidecarEgressMap) Delete(key string) {
@@ -198,9 +197,9 @@ func (s *SidecarEgressMap) Map() map[string]map[string]SidecarEgress {
 
 // Range is a thread safe iterator to iterate through the SidecarEgress map
 func (s *SidecarEgressMap) Range(fn func(k string, v map[string]SidecarEgress)) {
+	defer s.mutex.Unlock()
 	s.mutex.Lock()
 	for k, v := range s.cache {
 		fn(k, v)
 	}
-	s.mutex.Unlock()
 }