From a7314b81243b33ee087125951ab8455a6ca1b704 Mon Sep 17 00:00:00 2001 From: Gio Date: Mon, 30 Apr 2018 12:52:50 -0500 Subject: [PATCH] Check player data for corruption Fixed a null pointer exception that occurred with corrupted player data --- .../java/com/gmail/nossr50/util/commands/CommandUtils.java | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/main/java/com/gmail/nossr50/util/commands/CommandUtils.java b/src/main/java/com/gmail/nossr50/util/commands/CommandUtils.java index 76dd50bdf..29aa05b91 100644 --- a/src/main/java/com/gmail/nossr50/util/commands/CommandUtils.java +++ b/src/main/java/com/gmail/nossr50/util/commands/CommandUtils.java @@ -296,6 +296,11 @@ public final class CommandUtils { for (OfflinePlayer offlinePlayer : mcMMO.p.getServer().getOfflinePlayers()) { String playerName = offlinePlayer.getName(); + + if (playerName == null) { //Do null checking here to detect corrupted data before sending it throuogh .equals + System.err.println("[McMMO] Bad player data file with UIID " + offlinePlayer.getUniqueId() ); + continue; //Don't let an error here interrupt the loop + } if (partialName.equalsIgnoreCase(playerName)) { // Exact match