2017-01-20 12:21:44 +01:00
|
|
|
From 40e99b788e5e633d514a07da8d71d7f8436e2c0d Mon Sep 17 00:00:00 2001
|
2016-11-13 06:29:32 +01:00
|
|
|
From: Zach Brown <zach.brown@destroystokyo.com>
|
|
|
|
Date: Sat, 12 Nov 2016 23:25:22 -0600
|
|
|
|
Subject: [PATCH] Filter bad data from ArmorStand and SpawnEgg items
|
|
|
|
|
|
|
|
|
2016-11-14 02:36:23 +01:00
|
|
|
diff --git a/src/main/java/com/destroystokyo/paper/PaperWorldConfig.java b/src/main/java/com/destroystokyo/paper/PaperWorldConfig.java
|
2016-11-17 03:23:38 +01:00
|
|
|
index 4ad072a..73bf744 100644
|
2016-11-14 02:36:23 +01:00
|
|
|
--- a/src/main/java/com/destroystokyo/paper/PaperWorldConfig.java
|
|
|
|
+++ b/src/main/java/com/destroystokyo/paper/PaperWorldConfig.java
|
2016-11-17 03:23:38 +01:00
|
|
|
@@ -391,4 +391,12 @@ public class PaperWorldConfig {
|
2016-11-14 02:36:23 +01:00
|
|
|
private void removeCorruptTEs() {
|
|
|
|
removeCorruptTEs = getBoolean("remove-corrupt-tile-entities", false);
|
|
|
|
}
|
|
|
|
+
|
|
|
|
+ public boolean filterNBTFromSpawnEgg = true;
|
|
|
|
+ private void fitlerNBTFromSpawnEgg() {
|
|
|
|
+ filterNBTFromSpawnEgg = getBoolean("filter-nbt-data-from-spawn-eggs-and-related", true);
|
|
|
|
+ if (!filterNBTFromSpawnEgg) {
|
|
|
|
+ Bukkit.getLogger().warning("Spawn Egg and Armor Stand NBT filtering disabled, this is a potential security risk");
|
|
|
|
+ }
|
|
|
|
+ }
|
|
|
|
}
|
|
|
|
diff --git a/src/main/java/net/minecraft/server/EntityFallingBlock.java b/src/main/java/net/minecraft/server/EntityFallingBlock.java
|
2016-12-20 23:34:27 +01:00
|
|
|
index 8059e6d..a4f465e 100644
|
2016-11-14 02:36:23 +01:00
|
|
|
--- a/src/main/java/net/minecraft/server/EntityFallingBlock.java
|
|
|
|
+++ b/src/main/java/net/minecraft/server/EntityFallingBlock.java
|
2016-11-17 03:23:38 +01:00
|
|
|
@@ -251,6 +251,14 @@ public class EntityFallingBlock extends Entity {
|
2016-11-14 02:36:23 +01:00
|
|
|
this.block = Block.getById(nbttagcompound.getByte("Tile") & 255).fromLegacyData(i);
|
|
|
|
}
|
|
|
|
|
|
|
|
+ // Paper start - Block FallingBlocks with Command Blocks
|
2016-11-14 03:20:14 +01:00
|
|
|
+ // Check mappings on update - dc = "repeating_command_block" - dd = "chain_command_block"
|
|
|
|
+ final Block b = this.block.getBlock();
|
|
|
|
+ if (this.world.paperConfig.filterNBTFromSpawnEgg && (b == Blocks.COMMAND_BLOCK || b == Blocks.dc || b == Blocks.dd)) {
|
2016-11-14 02:36:23 +01:00
|
|
|
+ this.block = Blocks.STONE.getBlockData();
|
|
|
|
+ }
|
|
|
|
+ // Paper end
|
|
|
|
+
|
|
|
|
this.ticksLived = nbttagcompound.getInt("Time");
|
|
|
|
Block block = this.block.getBlock();
|
|
|
|
|
2016-11-13 06:29:32 +01:00
|
|
|
diff --git a/src/main/java/net/minecraft/server/ItemMonsterEgg.java b/src/main/java/net/minecraft/server/ItemMonsterEgg.java
|
2016-12-20 23:34:27 +01:00
|
|
|
index 1a01a57..f914ad4 100644
|
2016-11-13 06:29:32 +01:00
|
|
|
--- a/src/main/java/net/minecraft/server/ItemMonsterEgg.java
|
|
|
|
+++ b/src/main/java/net/minecraft/server/ItemMonsterEgg.java
|
2016-12-20 23:34:27 +01:00
|
|
|
@@ -102,7 +102,14 @@ public class ItemMonsterEgg extends Item {
|
2016-11-13 06:29:32 +01:00
|
|
|
NBTTagCompound nbttagcompound1 = entity.e(new NBTTagCompound());
|
|
|
|
UUID uuid = entity.getUniqueID();
|
2016-11-14 02:36:23 +01:00
|
|
|
|
2016-11-13 06:29:32 +01:00
|
|
|
- nbttagcompound1.a(nbttagcompound.getCompound("EntityTag"));
|
|
|
|
+ // Paper start - Filter out position and motion information
|
|
|
|
+ final NBTTagCompound entityTag = nbttagcompound.getCompound("EntityTag");
|
2016-11-14 02:36:23 +01:00
|
|
|
+ if (world.paperConfig.filterNBTFromSpawnEgg) {
|
|
|
|
+ entityTag.remove("Pos");
|
|
|
|
+ entityTag.remove("Motion");
|
|
|
|
+ }
|
2016-11-13 06:29:32 +01:00
|
|
|
+ nbttagcompound1.a(entityTag);
|
|
|
|
+ // Paper end
|
|
|
|
entity.a(uuid);
|
|
|
|
entity.f(nbttagcompound1);
|
|
|
|
}
|
|
|
|
--
|
2016-12-11 00:50:31 +01:00
|
|
|
2.9.3
|
2016-11-13 06:29:32 +01:00
|
|
|
|