From ab29122cf189c1553147b5ccf9dc662b956c5c89 Mon Sep 17 00:00:00 2001 From: Miles Holder Date: Sat, 22 Jun 2024 18:48:51 +1000 Subject: [PATCH] #1433: HumanEntity#openInventory(InventoryView) should only support views belonging to the player --- .../java/org/bukkit/craftbukkit/entity/CraftHumanEntity.java | 1 + 1 file changed, 1 insertion(+) diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftHumanEntity.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftHumanEntity.java index ffd0e83f5..025e84420 100644 --- a/src/main/java/org/bukkit/craftbukkit/entity/CraftHumanEntity.java +++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftHumanEntity.java @@ -371,6 +371,7 @@ public class CraftHumanEntity extends CraftLivingEntity implements HumanEntity { @Override public void openInventory(InventoryView inventory) { + Preconditions.checkArgument(this.equals(inventory.getPlayer()), "InventoryView must belong to the opening player"); if (!(getHandle() instanceof EntityPlayer)) return; // TODO: NPC support? if (((EntityPlayer) getHandle()).connection == null) return; if (getHandle().containerMenu != getHandle().inventoryMenu) {