Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Crash] ticking hopper below Cooking for blockheads' Oven tries to take item from slot 4 (only 3 slots) #123

Closed
MindStudioOfficial opened this issue Feb 13, 2025 · 1 comment
Labels
Bug Something isn't working as expected Fabric Issues occurring on the Fabric platform Minecraft 1.21.1 Issues occurring in Minecraft 1.21.1 Regression This worked in a previous version but is now broken

Comments

@MindStudioOfficial
Copy link

Minecraft Version

1.21.1 (LTS)

Mod Loader

Fabric

Describe the Issue

placing a hopper below a cooking for blockheads oven which has 3 input slots and 3 output slots crashes the server whenever the block is loaded and ticked because it tries to take items from slot 4 of 3 (IndexOutOfBounds)

java.lang.IndexOutOfBoundsException: Index 4 out of bounds for length 3
	at java.base/jdk.internal.util.Preconditions.outOfBounds(Unknown Source)
	at java.base/jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Unknown Source)
	at java.base/jdk.internal.util.Preconditions.checkIndex(Unknown Source)
	at java.base/java.util.Objects.checkIndex(Unknown Source)
	at java.base/java.util.ArrayList.get(Unknown Source)
	at knot//net.fabricmc.fabric.impl.transfer.item.SidedInventoryStorageImpl.createWrapperList(SidedInventoryStorageImpl.java:53)
	at knot//net.fabricmc.fabric.impl.transfer.item.SidedInventoryStorageImpl.<init>(SidedInventoryStorageImpl.java:38)
	at knot//net.fabricmc.fabric.impl.transfer.item.InventoryStorageImpl.getSidedWrapper(InventoryStorageImpl.java:110)
	at knot//net.fabricmc.fabric.impl.transfer.item.InventoryStorageImpl.of(InventoryStorageImpl.java:65)
	at knot//net.fabricmc.fabric.api.transfer.v1.item.InventoryStorage.of(InventoryStorage.java:62)
	at knot//net.blay09.mods.balm.fabric.FabricBalm.lambda$onInitialize$1(FabricBalm.java:58)
	at knot//net.fabricmc.fabric.impl.lookup.block.BlockApiLookupImpl.find(BlockApiLookupImpl.java:99)
	at knot//net.fabricmc.fabric.api.lookup.v1.block.BlockApiLookup.find(BlockApiLookup.java:169)
	at knot//net.minecraft.class_2614.handler$bne000$fabric-transfer-api-v1$hookExtract(class_2614.java:1988)
	at knot//net.minecraft.class_2614.method_11241(class_2614.java:219)
	at knot//net.minecraft.class_2614.method_31691(class_2614.java:107)
	at knot//net.minecraft.class_2614.method_11243(class_2614.java:123)
	at knot//net.minecraft.class_2614.method_31692(class_2614.java:107)
	at knot//net.minecraft.class_2818$class_5563.method_31703(class_2818.java:691)
	at knot//net.minecraft.class_2818$class_5564.method_31703(class_2818.java:745)
	at knot//net.minecraft.class_1937.method_18471(class_1937.java:488)
	at knot//net.minecraft.class_3218.method_18765(class_3218.java:413)
	at knot//net.minecraft.server.MinecraftServer.method_3813(MinecraftServer.java:1021)
	at knot//net.minecraft.class_3176.method_3813(class_3176.java:299)
	at knot//net.minecraft.server.MinecraftServer.method_3748(MinecraftServer.java:912)
	at knot//net.minecraft.server.MinecraftServer.method_29741(MinecraftServer.java:697)
	at knot//net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:281)
	at java.base/java.lang.Thread.run(Unknown Source)

Downgrading to Balm 21.0.23+fabric-1.21.1 fixes the issue so I suspect this commit to be the culprit

Logs

crash-2025-02-10_16.27.20-server.txt

@MindStudioOfficial MindStudioOfficial added the Bug Something isn't working as expected label Feb 13, 2025
@github-actions github-actions bot added Fabric Issues occurring on the Fabric platform Minecraft 1.21.1 Issues occurring in Minecraft 1.21.1 labels Feb 13, 2025
@BlayTheNinth BlayTheNinth moved this to 🌟 Next Items in Issues Feb 13, 2025
@BlayTheNinth BlayTheNinth moved this from 🌟 Next Items to 🚀 Pending Release in Issues Feb 13, 2025
@BlayTheNinth BlayTheNinth added the Regression This worked in a previous version but is now broken label Feb 13, 2025
@BlayTheNinth
Copy link
Member

Should be fixed in Balm 21.0.27+

@github-project-automation github-project-automation bot moved this from 🚀 Pending Release to 🎉 Done in Issues Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working as expected Fabric Issues occurring on the Fabric platform Minecraft 1.21.1 Issues occurring in Minecraft 1.21.1 Regression This worked in a previous version but is now broken
Projects
Archived in project
Development

No branches or pull requests

2 participants