105 lines
4.7 KiB
Diff
105 lines
4.7 KiB
Diff
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
From: Aikar <aikar@aikar.co>
|
|
Date: Tue, 1 May 2018 21:33:35 -0400
|
|
Subject: [PATCH] Close Plugin Class Loaders on Disable
|
|
|
|
This should close more memory leaks from /reload and disabling plugins,
|
|
by closing the class loader and the jar file.
|
|
|
|
Note: This patch is no longer necessary as upstream now also closes
|
|
PluginClassLoaders on disable. This patch is now only to keep around
|
|
API methods it previously added, and to add back the log message when a
|
|
PluginClassLoader fails to disable, as upstream decided to ignore the
|
|
exception.
|
|
|
|
diff --git a/src/main/java/org/bukkit/plugin/PluginLoader.java b/src/main/java/org/bukkit/plugin/PluginLoader.java
|
|
index cb530369e667c426c842da356c31304bb5c3ecfa..a11515b81575fc42c771a218a81fea8f05d2289d 100644
|
|
--- a/src/main/java/org/bukkit/plugin/PluginLoader.java
|
|
+++ b/src/main/java/org/bukkit/plugin/PluginLoader.java
|
|
@@ -78,4 +78,21 @@ public interface PluginLoader {
|
|
* @param plugin Plugin to disable
|
|
*/
|
|
public void disablePlugin(@NotNull Plugin plugin);
|
|
+
|
|
+ // Paper start - close Classloader on disable
|
|
+ /**
|
|
+ * This method is no longer useful as upstream has
|
|
+ * made it so plugin classloaders are always closed on disable.
|
|
+ * Use {@link #disablePlugin(Plugin)} instead.
|
|
+ *
|
|
+ * @param plugin Plugin to disable
|
|
+ * @param closeClassloader unused
|
|
+ * @deprecated Classloader is always closed by upstream now.
|
|
+ */
|
|
+ @Deprecated(forRemoval = true)
|
|
+ // provide default to allow other PluginLoader implementations to work
|
|
+ default public void disablePlugin(@NotNull Plugin plugin, boolean closeClassloader) {
|
|
+ disablePlugin(plugin);
|
|
+ }
|
|
+ // Paper end - close Classloader on disable
|
|
}
|
|
diff --git a/src/main/java/org/bukkit/plugin/PluginManager.java b/src/main/java/org/bukkit/plugin/PluginManager.java
|
|
index 03213fde8315384ec56c16031cfc606ade2e8091..94fef99525a3613dcc313a0d0b03e47a91d4117b 100644
|
|
--- a/src/main/java/org/bukkit/plugin/PluginManager.java
|
|
+++ b/src/main/java/org/bukkit/plugin/PluginManager.java
|
|
@@ -162,6 +162,22 @@ public interface PluginManager extends io.papermc.paper.plugin.PermissionManager
|
|
*/
|
|
public void disablePlugin(@NotNull Plugin plugin);
|
|
|
|
+ // Paper start - close Classloader on disable
|
|
+ /**
|
|
+ * This method is no longer useful as upstream has
|
|
+ * made it so plugin classloaders are always closed on disable.
|
|
+ * Use {@link #disablePlugin(Plugin)} instead.
|
|
+ *
|
|
+ * @param plugin Plugin to disable
|
|
+ * @param closeClassloader unused
|
|
+ * @deprecated Classloader is always closed by upstream now.
|
|
+ */
|
|
+ @Deprecated(forRemoval = true)
|
|
+ public default void disablePlugin(@NotNull Plugin plugin, boolean closeClassloader) {
|
|
+ this.disablePlugin(plugin);
|
|
+ }
|
|
+ // Paper end - close Classloader on disable
|
|
+
|
|
/**
|
|
* Gets a {@link Permission} from its fully qualified name
|
|
*
|
|
diff --git a/src/main/java/org/bukkit/plugin/SimplePluginManager.java b/src/main/java/org/bukkit/plugin/SimplePluginManager.java
|
|
index 2b8308989fce7f8a16907f8711b362e671fdbfb6..f96164a2bc4e042bdd7c6045a9b392ad4e4dbea7 100644
|
|
--- a/src/main/java/org/bukkit/plugin/SimplePluginManager.java
|
|
+++ b/src/main/java/org/bukkit/plugin/SimplePluginManager.java
|
|
@@ -532,6 +532,21 @@ public final class SimplePluginManager implements PluginManager {
|
|
}
|
|
}
|
|
|
|
+ // Paper start
|
|
+ /**
|
|
+ * This method is no longer useful as upstream has
|
|
+ * made it so plugin classloaders are always closed on disable.
|
|
+ * Use {@link #disablePlugins()} instead.
|
|
+ *
|
|
+ * @param closeClassloaders unused
|
|
+ * @deprecated Classloader is always closed by upstream now.
|
|
+ */
|
|
+ @Deprecated(forRemoval = true)
|
|
+ public void disablePlugins(boolean closeClassloaders) {
|
|
+ this.disablePlugins();
|
|
+ }
|
|
+ // Paper end
|
|
+
|
|
@Override
|
|
public void disablePlugin(@NotNull final Plugin plugin) {
|
|
if (true) {this.paperPluginManager.disablePlugin(plugin); return;} // Paper
|
|
diff --git a/src/main/java/org/bukkit/plugin/java/JavaPluginLoader.java b/src/main/java/org/bukkit/plugin/java/JavaPluginLoader.java
|
|
index ac1c884e950ce13e9e7b3a7be1378808012fafc4..e0423adf45bf5855c23259257863f67cec1c3d54 100644
|
|
--- a/src/main/java/org/bukkit/plugin/java/JavaPluginLoader.java
|
|
+++ b/src/main/java/org/bukkit/plugin/java/JavaPluginLoader.java
|
|
@@ -368,6 +368,7 @@ public final class JavaPluginLoader implements PluginLoader {
|
|
loader.close();
|
|
} catch (IOException ex) {
|
|
//
|
|
+ this.server.getLogger().log(Level.WARNING, "Error closing the PluginClassLoader for '" + plugin.getDescription().getFullName() + "'", ex); // Paper - log exception
|
|
}
|
|
}
|
|
}
|