From 7cd54b1419c43b869d38e8fa157738a6126e84a3 Mon Sep 17 00:00:00 2001 From: Niki4tap Date: Fri, 11 Sep 2026 04:51:07 +0300 Subject: [PATCH] fix nullptr during perm check on bad profiles This commit checks if the game profile is complete first, before calling `canSendCommands`, which does not do the check and explodes on null. In practice this should fix weird fakeplayer interactions like thaumcraft golems breaking/placing blocks in claimed chunks. And keep the logs clean of endless stacktraces :) --- .../lib/util/permission/DefaultPermissionHandler.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/serverutils/lib/util/permission/DefaultPermissionHandler.java b/src/main/java/serverutils/lib/util/permission/DefaultPermissionHandler.java index 4a9d8d986..ebe79ca43 100644 --- a/src/main/java/serverutils/lib/util/permission/DefaultPermissionHandler.java +++ b/src/main/java/serverutils/lib/util/permission/DefaultPermissionHandler.java @@ -51,7 +51,7 @@ public boolean hasPermission(GameProfile profile, String node, @Nullable IContex } MinecraftServer server = ServerUtils.getServer(); - return server != null && server.getConfigurationManager().func_152596_g(profile); // canSendCommands + return server != null && profile.isComplete() && server.getConfigurationManager().func_152596_g(profile); // canSendCommands } @Override