From aa06a5ace11d02ad1066a4569cbd9c5d7fcc6d8b Mon Sep 17 00:00:00 2001 From: CraftBukkit/Spigot Date: Fri, 29 Mar 2013 22:43:05 -0400 Subject: [PATCH] Properly return contents of Inventory. Fixes BUKKIT-3930 When an array of an inventory's contents is requested, we loop through the contents of the NMS inventory's ItemStacks in order to return Bukkit ItemStacks that can be used through the API. However, the NMS ItemStack can, in some cases, be larger than the physical size of the inventory. Using the size of the NMS array as a limit on the loop that follows can result in an ArrayIndexOutOfBoundsException because the Bukkit array's length is the actual size of the inventory, and thus will be smaller. With this commit we use the smaller of the two arrays' length as the limit in the loop, thus eliminating the possibility that the smaller array will be asked for an index higher than its length. By: h31ix --- .../java/org/bukkit/craftbukkit/inventory/CraftInventory.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/paper-server/src/main/java/org/bukkit/craftbukkit/inventory/CraftInventory.java b/paper-server/src/main/java/org/bukkit/craftbukkit/inventory/CraftInventory.java index 31a2907242..577e2101c6 100644 --- a/paper-server/src/main/java/org/bukkit/craftbukkit/inventory/CraftInventory.java +++ b/paper-server/src/main/java/org/bukkit/craftbukkit/inventory/CraftInventory.java @@ -54,7 +54,8 @@ public class CraftInventory implements Inventory { ItemStack[] items = new ItemStack[getSize()]; net.minecraft.server.ItemStack[] mcItems = getInventory().getContents(); - for (int i = 0; i < mcItems.length; i++) { + int size = Math.min(items.length, mcItems.length); + for (int i = 0; i < size; i++) { items[i] = mcItems[i] == null ? null : CraftItemStack.asCraftMirror(mcItems[i]); }