From 72d5f50bb74a07151b10431a91a4984372751ce6 Mon Sep 17 00:00:00 2001 From: md_5 Date: Mon, 14 Apr 2014 17:22:57 +1000 Subject: [PATCH] Check for Blank OfflinePlayer Names - the internal representation does not allow them, so lets catch it early when the issue lies with the user. --- ...-Check-for-blank-OfflinePlayer-Names.patch | 21 +++++++++++++++++++ 1 file changed, 21 insertions(+) create mode 100644 CraftBukkit-Patches/0137-Check-for-blank-OfflinePlayer-Names.patch diff --git a/CraftBukkit-Patches/0137-Check-for-blank-OfflinePlayer-Names.patch b/CraftBukkit-Patches/0137-Check-for-blank-OfflinePlayer-Names.patch new file mode 100644 index 0000000000..bb8e17fb8f --- /dev/null +++ b/CraftBukkit-Patches/0137-Check-for-blank-OfflinePlayer-Names.patch @@ -0,0 +1,21 @@ +From ffc8031afadcab95da7bf7991fa4903265de4e77 Mon Sep 17 00:00:00 2001 +From: md_5 +Date: Mon, 14 Apr 2014 17:21:24 +1000 +Subject: [PATCH] Check for blank OfflinePlayer Names + + +diff --git a/src/main/java/org/bukkit/craftbukkit/CraftServer.java b/src/main/java/org/bukkit/craftbukkit/CraftServer.java +index c30197b..9f0517b 100644 +--- a/src/main/java/org/bukkit/craftbukkit/CraftServer.java ++++ b/src/main/java/org/bukkit/craftbukkit/CraftServer.java +@@ -1304,6 +1304,7 @@ public final class CraftServer implements Server { + + public OfflinePlayer getOfflinePlayer(String name) { + Validate.notNull(name, "Name cannot be null"); ++ com.google.common.base.Preconditions.checkArgument( !org.apache.commons.lang.StringUtils.isBlank( name ), "Name cannot be blank" ); // Spigot + + OfflinePlayer result = getPlayerExact(name); + if (result == null) { +-- +1.8.3.2 +