aboutsummaryrefslogtreecommitdiffhomepage
path: root/patches/server/0458-Fix-AdvancementDataPlayer-leak-due-from-quitting-ear.patch
diff options
context:
space:
mode:
Diffstat (limited to 'patches/server/0458-Fix-AdvancementDataPlayer-leak-due-from-quitting-ear.patch')
-rw-r--r--patches/server/0458-Fix-AdvancementDataPlayer-leak-due-from-quitting-ear.patch74
1 files changed, 74 insertions, 0 deletions
diff --git a/patches/server/0458-Fix-AdvancementDataPlayer-leak-due-from-quitting-ear.patch b/patches/server/0458-Fix-AdvancementDataPlayer-leak-due-from-quitting-ear.patch
new file mode 100644
index 0000000000..da7194a954
--- /dev/null
+++ b/patches/server/0458-Fix-AdvancementDataPlayer-leak-due-from-quitting-ear.patch
@@ -0,0 +1,74 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Spottedleaf <[email protected]>
+Date: Mon, 13 Jul 2020 06:22:54 -0700
+Subject: [PATCH] Fix AdvancementDataPlayer leak due from quitting early in
+ login
+
+Move the criterion storage to the AdvancementDataPlayer object
+itself, so the criterion object stores no references - and thus
+needs no cleanup.
+
+diff --git a/src/main/java/net/minecraft/advancements/critereon/SimpleCriterionTrigger.java b/src/main/java/net/minecraft/advancements/critereon/SimpleCriterionTrigger.java
+index 06fc39b19385d36fd0c5bb9a7042a238eb6e8a57..bb1f0e9dbcb792d015d1cb65664a96fdd3e0489e 100644
+--- a/src/main/java/net/minecraft/advancements/critereon/SimpleCriterionTrigger.java
++++ b/src/main/java/net/minecraft/advancements/critereon/SimpleCriterionTrigger.java
+@@ -14,22 +14,24 @@ import net.minecraft.server.level.ServerPlayer;
+ import net.minecraft.world.level.storage.loot.LootContext;
+
+ public abstract class SimpleCriterionTrigger<T extends AbstractCriterionTriggerInstance> implements CriterionTrigger<T> {
+- private final Map<PlayerAdvancements, Set<CriterionTrigger.Listener<T>>> players = Maps.newIdentityHashMap();
++ //private final Map<PlayerAdvancements, Set<CriterionTrigger.Listener<T>>> players = Maps.newIdentityHashMap(); // Paper - moved into AdvancementDataPlayer to fix memory leak
++
++ public SimpleCriterionTrigger() {}
+
+ @Override
+ public final void addPlayerListener(PlayerAdvancements manager, CriterionTrigger.Listener<T> conditions) {
+- this.players.computeIfAbsent(manager, (managerx) -> {
++ manager.criterionData.computeIfAbsent(this, (managerx) -> { // Paper - fix AdvancementDataPlayer leak
+ return Sets.newHashSet();
+ }).add(conditions);
+ }
+
+ @Override
+ public final void removePlayerListener(PlayerAdvancements manager, CriterionTrigger.Listener<T> conditions) {
+- Set<CriterionTrigger.Listener<T>> set = this.players.get(manager);
++ Set<CriterionTrigger.Listener<T>> set = (Set) manager.criterionData.get(this); // Paper - fix AdvancementDataPlayer leak
+ if (set != null) {
+ set.remove(conditions);
+ if (set.isEmpty()) {
+- this.players.remove(manager);
++ manager.criterionData.remove(this); // Paper - fix AdvancementDataPlayer leak
+ }
+ }
+
+@@ -37,7 +39,7 @@ public abstract class SimpleCriterionTrigger<T extends AbstractCriterionTriggerI
+
+ @Override
+ public final void removePlayerListeners(PlayerAdvancements tracker) {
+- this.players.remove(tracker);
++ tracker.criterionData.remove(this); // Paper - fix AdvancementDataPlayer leak
+ }
+
+ protected abstract T createInstance(JsonObject obj, EntityPredicate.Composite playerPredicate, DeserializationContext predicateDeserializer);
+@@ -50,7 +52,7 @@ public abstract class SimpleCriterionTrigger<T extends AbstractCriterionTriggerI
+
+ protected void trigger(ServerPlayer player, Predicate<T> predicate) {
+ PlayerAdvancements playerAdvancements = player.getAdvancements();
+- Set<CriterionTrigger.Listener<T>> set = this.players.get(playerAdvancements);
++ Set<CriterionTrigger.Listener<T>> set = (Set) playerAdvancements.criterionData.get(this); // Paper - fix AdvancementDataPlayer leak
+ if (set != null && !set.isEmpty()) {
+ LootContext lootContext = EntityPredicate.createContext(player, player);
+ List<CriterionTrigger.Listener<T>> list = null;
+diff --git a/src/main/java/net/minecraft/server/PlayerAdvancements.java b/src/main/java/net/minecraft/server/PlayerAdvancements.java
+index 0c2f12e7930646a3da53a50f38be62e0cb1ed2b7..e7ec5e1144c1596b035f97fb1fb86d18e61be3c9 100644
+--- a/src/main/java/net/minecraft/server/PlayerAdvancements.java
++++ b/src/main/java/net/minecraft/server/PlayerAdvancements.java
+@@ -68,6 +68,8 @@ public class PlayerAdvancements {
+ private Advancement lastSelectedTab;
+ private boolean isFirstPacket = true;
+
++ public final Map<net.minecraft.advancements.critereon.SimpleCriterionTrigger, Set<CriterionTrigger.Listener>> criterionData = Maps.newIdentityHashMap(); // Paper - fix advancement data player leakage
++
+ public PlayerAdvancements(DataFixer dataFixer, PlayerList playerManager, ServerAdvancementManager advancementLoader, File advancementFile, ServerPlayer owner) {
+ this.dataFixer = dataFixer;
+ this.playerList = playerManager;