fix: deadlock caused by rwMutex in limiter mapper (#2053)

apache · Sep 16, 2022 · d924219 · d924219
1 parent 48a0733
commit d924219
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/filter/adaptivesvc/limiter_mapper.go b/filter/adaptivesvc/limiter_mapper.go
@@ -58,6 +58,8 @@ func (m *limiterMapper) newAndSetMethodLimiter(url *common.URL, methodName strin
 		ok bool
 	)
 	m.rwMutex.Lock()
+	defer m.rwMutex.Unlock()
+
 	if l, ok = limiterMapperSingleton.mapper[key]; ok {
 		return l, nil
 	}
@@ -68,7 +70,6 @@ func (m *limiterMapper) newAndSetMethodLimiter(url *common.URL, methodName strin
 		return nil, ErrLimiterTypeNotFound
 	}
 	limiterMapperSingleton.mapper[key] = l
-	m.rwMutex.Unlock()
 	return l, nil
 }