fixing deadlock

broadcast/broadcast.go

@@ -232,11 +232,11 @@ func GetContainerNetworks(containers []string) map[string]string {
 // update broadcastState with a map of items
 func UpdateBroadcastState(values map[string]interface{}) error {
 	mu.Lock()
-	defer mu.Unlock()
 	v := reflect.ValueOf(&broadcastState).Elem()
 	for key, value := range values {
 		field := v.FieldByName(key)
 		if !field.IsValid() || !field.CanSet() {
+			mu.Unlock()
 			return fmt.Errorf("field %s does not exist or is not settable", key)
 		}
 		val := reflect.ValueOf(value)
@@ -244,6 +244,7 @@ func UpdateBroadcastState(values map[string]interface{}) error {
 			val = val.Elem() // Extract the underlying value from the interface
 		}
 		if err := recursiveUpdate(field, val); err != nil {
+			mu.Unlock()
 			return err
 		}
 	}