mirrors/dendrite
1
0
Fork 0
mirror of https://github.com/matrix-org/dendrite.git synced 2025-12-26 08:13:09 -06:00
Code Issues Packages Projects Releases Wiki Activity

clearQueue only called by defer so should be safe to delete queue in any case

Browse source
This commit is contained in:
Neil Alexander 2021-02-17 13:43:41 +00:00
parent 6a5826cba4
commit d59b312df0
No known key found for this signature in database
GPG key ID: A02A2019A2BB0944
2 changed files with 7 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
federationsender/queue/destinationqueue.go
View file

@ -247,7 +247,7 @@ func (oq *destinationQueue) backgroundSend() {
}
destinationQueueRunning.Inc()
defer destinationQueueRunning.Dec()
defer oq.queues.clearQueue(oq.destination)
defer oq.queues.clearQueue(oq)
defer oq.running.Store(false)
// Mark the queue as overflowed, so we will consult the database

7
federationsender/queue/queue.go
View file

@ -171,16 +171,15 @@ func (oqs *OutgoingQueues) getQueue(destination gomatrixserverlib.ServerName) *d
return oq
}
func (oqs *OutgoingQueues) clearQueue(destination gomatrixserverlib.ServerName) {
func (oqs *OutgoingQueues) clearQueue(oq *destinationQueue) {
oqs.queuesMutex.Lock()
defer oqs.queuesMutex.Unlock()
if oq, ok := oqs.queues[destination]; ok {
close(oq.notify)
close(oq.interruptBackoff)
delete(oqs.queues, destination)
delete(oqs.queues, oq.destination)
destinationQueueTotal.Dec()
}
}
type ErrorFederationDisabled struct {
Message string