twin/paper-mc
1
0
Fork 0
mirror of https://github.com/PaperMC/Paper.git synced 2025-07-26 09:42:06 -07:00
Code Issues Packages Projects Releases Wiki Activity

Do not send update packets for saddle/body slots

Browse Source 
The player inventory now allows setting the saddle and body equipment
slot. The client menu for this however does not contain slots. As such,
updating the player for such slots leads to an IOOB exception.
This commit is contained in:
Bjarne Koll
2025-06-05 01:23:45 +02:00
parent f8bde6e1d0
commit 921c5f936e
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
paper-server/src/main/java/org/bukkit/craftbukkit/inventory/CraftInventoryPlayer.java
View File

@@ -77,6 +77,8 @@ public class CraftInventoryPlayer extends CraftInventory implements org.bukkit.i
// Paper end - Validate setItem index
super.setItem(index, item);
if (this.getHolder() == null) return;
if (index > 40) return; // Do not send update packets for saddle or body slots as the player menu does not contain these.
ServerPlayer player = ((CraftPlayer) this.getHolder()).getHandle();
if (player.connection == null) return;
// PacketPlayOutSetSlot places the items differently than setItem()