Fix roomserver deadlock

Move the mutex lock outside the loop so that we don't lock up if there is more
than one event
This commit is contained in:
Richard van der Hoff 2017-12-19 11:27:18 +00:00
parent ec30d143cd
commit fc8b33e8ad

View file

@ -61,11 +61,10 @@ func (r *RoomserverInputAPI) InputRoomEvents(
request *api.InputRoomEventsRequest,
response *api.InputRoomEventsResponse,
) error {
// We lock as processRoomEvent can only be called once at a time
r.mutex.Lock()
defer r.mutex.Unlock()
for i := range request.InputRoomEvents {
// We lock as processRoomEvent can ony be called once at a time
r.mutex.Lock()
defer r.mutex.Unlock()
if err := processRoomEvent(ctx, r.DB, r, request.InputRoomEvents[i]); err != nil {
return err
}