2018-04-22 14:40:36 +00:00
|
|
|
From 6d65fb462e07a362f643770fce990e722470e233 Mon Sep 17 00:00:00 2001
|
2017-05-14 01:26:19 +00:00
|
|
|
From: Zach Brown <zach.brown@destroystokyo.com>
|
|
|
|
Date: Sat, 13 May 2017 20:11:21 -0500
|
|
|
|
Subject: [PATCH] Add system property to disable book size limits
|
|
|
|
|
|
|
|
If anyone comes in with a watchdog crash related to books after this patch
|
|
|
|
you will not only be publicly shamed but also made an example of.
|
|
|
|
|
|
|
|
Disables the security limits on books entirely, allowing plugins AND players
|
|
|
|
to make books with as much data as they want. Do not use this without
|
|
|
|
limiting incoming data from packets in some other way.
|
|
|
|
|
|
|
|
diff --git a/src/main/java/org/bukkit/craftbukkit/inventory/CraftMetaBook.java b/src/main/java/org/bukkit/craftbukkit/inventory/CraftMetaBook.java
|
2018-04-22 14:40:36 +00:00
|
|
|
index a7e51d9f1..ffdb7ec82 100644
|
2017-05-14 01:26:19 +00:00
|
|
|
--- a/src/main/java/org/bukkit/craftbukkit/inventory/CraftMetaBook.java
|
|
|
|
+++ b/src/main/java/org/bukkit/craftbukkit/inventory/CraftMetaBook.java
|
2017-12-05 01:34:10 +00:00
|
|
|
@@ -40,6 +40,7 @@ public class CraftMetaBook extends CraftMetaItem implements BookMeta {
|
2017-05-14 01:26:19 +00:00
|
|
|
static final int MAX_PAGES = 50;
|
2017-05-14 01:37:46 +00:00
|
|
|
static final int MAX_PAGE_LENGTH = 320; // 256 limit + 64 characters to allow for psuedo colour codes
|
2017-05-14 01:26:19 +00:00
|
|
|
static final int MAX_TITLE_LENGTH = 32;
|
|
|
|
+ private static final boolean OVERRIDE_CHECKS = Boolean.getBoolean("disable.book-limits"); // Paper - Add override
|
|
|
|
|
|
|
|
protected String title;
|
|
|
|
protected String author;
|
2018-04-22 14:40:36 +00:00
|
|
|
@@ -196,7 +197,7 @@ public class CraftMetaBook extends CraftMetaItem implements BookMeta {
|
2017-05-14 01:26:19 +00:00
|
|
|
if (title == null) {
|
|
|
|
this.title = null;
|
|
|
|
return true;
|
|
|
|
- } else if (title.length() > MAX_TITLE_LENGTH) {
|
|
|
|
+ } else if (title.length() > MAX_TITLE_LENGTH && !OVERRIDE_CHECKS) { // Paper - Add override
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2018-04-22 14:40:36 +00:00
|
|
|
@@ -232,7 +233,7 @@ public class CraftMetaBook extends CraftMetaItem implements BookMeta {
|
2017-05-14 01:26:19 +00:00
|
|
|
throw new IllegalArgumentException("Invalid page number " + page + "/" + pages.size());
|
|
|
|
}
|
|
|
|
|
|
|
|
- String newText = text == null ? "" : text.length() > MAX_PAGE_LENGTH ? text.substring(0, MAX_PAGE_LENGTH) : text;
|
|
|
|
+ String newText = text == null ? "" : text.length() > MAX_PAGE_LENGTH && !OVERRIDE_CHECKS ? text.substring(0, MAX_PAGE_LENGTH) : text;
|
|
|
|
pages.set(page - 1, CraftChatMessage.fromString(newText, true)[0]);
|
|
|
|
}
|
|
|
|
|
2018-04-22 14:40:36 +00:00
|
|
|
@@ -244,13 +245,13 @@ public class CraftMetaBook extends CraftMetaItem implements BookMeta {
|
2017-05-14 01:26:19 +00:00
|
|
|
|
|
|
|
public void addPage(final String... pages) {
|
|
|
|
for (String page : pages) {
|
|
|
|
- if (this.pages.size() >= MAX_PAGES) {
|
|
|
|
+ if (this.pages.size() >= MAX_PAGES && !OVERRIDE_CHECKS) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (page == null) {
|
|
|
|
page = "";
|
|
|
|
- } else if (page.length() > MAX_PAGE_LENGTH) {
|
|
|
|
+ } else if (page.length() > MAX_PAGE_LENGTH && !OVERRIDE_CHECKS) { // Paper - Add override
|
|
|
|
page = page.substring(0, MAX_PAGE_LENGTH);
|
|
|
|
}
|
|
|
|
|
|
|
|
--
|
2018-04-22 14:40:36 +00:00
|
|
|
2.17.0
|
2017-05-14 01:26:19 +00:00
|
|
|
|