I encountere a crash upon jumping into Leesti with AsteroidStorm4.03, and after failing the mission so that the station was destroyed. With thargoid_carrier2.0, the game crashes on entry to the system.
Here's a save file one jump away from Leesti that shows the crash:
https://www.box.com/s/s0p4arjyennkj2uihf6z
I removed all OXPs except these two and the crash still occurs. It makes sense, thargoid_carrier is probably not checking for the possibility that the station does not exist.
CmdrLuke (Bogatyr)
Crash in 1.77: AsteroidStorm4.03 + thargoid_carrier2.0
Moderators: winston, another_commander
- Eric Walch
- Slightly Grand Rear Admiral
- Posts: 5536
- Joined: Sat Jun 16, 2007 3:48 pm
- Location: Netherlands
Re: Crash in 1.77: AsteroidStorm4.03 + thargoid_carrier2.0
Thanks for the clear analysis of the problem.
It is probably this line in Thargoidcarrier were it goes wrong:
This line should return null itself and than the following line:
should throw an error as
Crashing itself is an Oolite bug. I'll check it tonight if
It is probably this line in Thargoidcarrier were it goes wrong:
Code: Select all
var otherSystemStations = system.filteredEntities(this, $isOtherSystemStation, system.mainStation);
Code: Select all
if(otherSystemStations.length > 0)
null
has no length
. Crashing itself is an Oolite bug. I'll check it tonight if
system.filteredEntities
could crash oolite with a null parameter.UPS-Courier & DeepSpacePirates & others at the box and some older versions
- Commander McLane
- ---- E L I T E ----
- Posts: 9520
- Joined: Thu Dec 14, 2006 9:08 am
- Location: a Hacker Outpost in a moderately remote area
- Contact:
Re: Crash in 1.77: AsteroidStorm4.03 + thargoid_carrier2.0
Thanks for the report and analyses. Fixed for the next version. In the meantime you can fix it yourself by inserting the linejust in front of the first line quoted by Eric.
Code: Select all
if(!system.mainStation) return;
Re: Crash in 1.77: AsteroidStorm4.03 + thargoid_carrier2.0
Glad to help out. Thanks for the workaround, I could then jump into Leesti without crashingCommander McLane wrote:Thanks for the report and analyses. Fixed for the next version. In the meantime you can fix it yourself by inserting the linejust in front of the first line quoted by Eric.Code: Select all
if(!system.mainStation) return;