Adds a fix for Scores that can potentially crash clients.

This commit is contained in:
hauno 2014-05-06 18:04:25 -07:00 committed by md_5
parent 124d44473c
commit 531071af97
1 changed files with 22 additions and 0 deletions

View File

@ -0,0 +1,22 @@
From 101f1c846399b878f4b072bba2ed37c959986aa2 Mon Sep 17 00:00:00 2001
From: hauno <imhauno@gmail.com>
Date: Tue, 6 May 2014 18:01:37 -0700
Subject: [PATCH] Cap Objective Score Length
Adds a check for Score arguments that would crash the client
diff --git a/src/main/java/org/bukkit/craftbukkit/scoreboard/CraftObjective.java b/src/main/java/org/bukkit/craftbukkit/scoreboard/CraftObjective.java
index 9eaec71..0ee1147 100644
--- a/src/main/java/org/bukkit/craftbukkit/scoreboard/CraftObjective.java
+++ b/src/main/java/org/bukkit/craftbukkit/scoreboard/CraftObjective.java
@@ -95,6 +95,7 @@ final class CraftObjective extends CraftScoreboardComponent implements Objective
public Score getScore(String entry) throws IllegalArgumentException, IllegalStateException {
Validate.notNull(entry, "Entry cannot be null");
+ if (entry.length() > 16) throw new IllegalArgumentException("Entry cannot be longer than 16 characters!"); // Spigot
CraftScoreboard scoreboard = checkState();
return new CraftScore(this, entry);
--
1.9.1