Limit number of Loki ring offsets and prevent cost calculation overflow

VazkiiMods / Botania

A tech mod for Minecraft themed around the magic of nature and plant life.

http://botaniamod.net/index.html

Other

1.25k stars 497 forks source link

Limit number of Loki ring offsets and prevent cost calculation overflow #4686

Closed TheRealWormbo closed 2 weeks ago

TheRealWormbo commented 2 weeks ago

Fixes #4681 by restricting the number of offsets to prevent the ring's NBT data from exceeding protocol limits. This also contains a fix to a potential integer overflow in the mana calculation that can happen within intended use of the ring already.