Fork/CitizensCMD
2
0
Fork 0
forked from Upstream/CitizensCMD
Code Pull Requests Actions Releases Activity

Fix memory leak caused by PermissionsManager

Browse Source
This commit is contained in:
skat 2024-02-17 12:55:16 +03:00
parent 54c252902d
commit 974234a28b
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/main/java/me/mattstudios/citizenscmd/permissions/PermissionsManager.java
View File

@ -46,8 +46,7 @@ public class PermissionsManager {
public void setPermission(Player player, String permission) {
final PermissionAttachment permissionAttachment = player.addAttachment(plugin);
permissionsData.put(player.getUniqueId(), permissionAttachment);
final PermissionAttachment permissionAttachment1 = permissionsData.get(player.getUniqueId());
permissionAttachment1.setPermission(permission, true);
permissionAttachment.setPermission(permission, true);
}
/**
@ -57,7 +56,10 @@ public class PermissionsManager {
* @param permission The permission node to be removed
*/
public void unsetPermission(Player player, String permission) {
permissionsData.get(player.getUniqueId()).unsetPermission(permission);
PermissionAttachment attachment = permissionsData.remove(player.getUniqueId());
if (attachment != null) {
attachment.unsetPermission(permission);
}
}
}