Carry On

Carry On

112M Downloads

1.12.2 NPE in ScriptReader

fonnymunkey opened this issue ยท 0 comments

commented

Expected Behavior

Not crash, load the files correctly using File.separator.

Actual Behavior

Crashes on certain systems that do not use "/" to separate files.

Steps to Reproduce

Load carryon on system incompatible with "/" file separator. (FreeBSD is known to crash for this)

Version of Minecraft, Carry On, Forge

MC 1.12.2, CarryOn 1.12.6.20, Forge 2860

net.minecraftforge.fml.common.LoaderExceptionModCrash: Caught exception from Carry On (carryon)
Caused by: java.lang.NullPointerException
at tschipp.carryon.common.scripting.ScriptReader.preInit(ScriptReader.java:36)
at tschipp.carryon.common.CommonProxy.preInit(CommonProxy.java:31)
at tschipp.carryon.CarryOn.preInit(CarryOn.java:57)

https://github.com/Tschipp/CarryOn/blob/1.12/src/main/java/tschipp/carryon/common/scripting/ScriptReader.java#L31