From ce36dbb2b4bee85de9ba54d4cbc97ec72c8d758c Mon Sep 17 00:00:00 2001 From: Andrei Mihu Date: Wed, 9 Nov 2022 12:51:07 +0000 Subject: [PATCH] Fix matchmaker tracking of eligible matches when downsizing for count multiples. --- CHANGELOG.md | 1 + server/matchmaker.go | 3 +++ 2 files changed, 4 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index a4c2976e9..ee04c11b3 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -13,6 +13,7 @@ The format is based on [keep a changelog](http://keepachangelog.com) and this pr - Fix response selection in purchase lookups by identifier. - Ensure corresponding leaderboard rank cache entries are removed when a user is deleted. - Consistently update scheduler when leaderboards and tournaments are deleted. +- Fix matchmaker tracking of eligible matches when downsizing for count multiples. ## [3.14.0] - 2022-10-14 ### Added diff --git a/server/matchmaker.go b/server/matchmaker.go index 2dbce2ab8..f86d6cd04 100644 --- a/server/matchmaker.go +++ b/server/matchmaker.go @@ -547,6 +547,9 @@ func (m *LocalMatchmaker) Process() { } } + // We've removed something, update the known size of the currently considered combo. + l = len(foundCombo) + index.Count + if (len(foundCombo)+index.Count)%index.CountMultiple != 0 { // Removal was insufficient, the combo is still not valid for the required multiple. continue -- GitLab