search

Draylar / get-off-my-lawn

Keep people off your lawn with claims! Built for Fabric.
https://www.curseforge.com/minecraft/mc-mods/get-off-my-lawn
MIT License
4 stars 18 forks source link

bug in mixin that causes server crash #30

Closed Monster-Zer0 closed 3 years ago

Monster-Zer0 commented 3 years ago

this mixin https://github.com/Draylar/get-off-my-lawn/blob/2dcd3c789fc7d1ca2f9adf3434ebf5be31232c21/src/main/java/draylar/goml/mixin/augment/EndermanEntityMixin.java

here is the log

https://hatebin.com/uoyespitiq

not sure what the issue is, but notsteven helped me figure out the issue

Draylar commented 3 years ago

I don't see a crash in your hatebin. Is that the full log?

Monster-Zer0 commented 3 years ago

it's the full log. it just freezes at the end there and everyone is kicked off

Monster-Zer0 commented 3 years ago

nevemind apparently there is a limit

Monster-Zer0 commented 3 years ago

i am making my own hastebin real quick then post the full log

Monster-Zer0 commented 3 years ago

here is the juicy part https://paste.alloffabric.com/ogotosures.csharp

Monster-Zer0 commented 3 years ago

https://paste.alloffabric.com/ulalejadad.yaml

crash log

Draylar commented 3 years ago

Fixed in 1.3.3-beta, which will be out in a second. I assume somebody somehow managed to remove a claim block without removing the claim (like a broken item that bypasses claims and removed the anchor, or an operator using /setblock to remove the anchor). In the next update, any invalid claims like this will self-delete themselves and not NPE. Let me know if you encounter any other issues. Thanks for the report!