[crash devel] elevated tracks with stations, crash on weight check

Topic: [crash devel] elevated tracks with stations, crash on weight check (Read 8518 times) previous topic - next topic

[crash devel] elevated tracks with stations, crash on weight check

July 15, 2010, 06:15:19 pm

the crash can be reproduced by building elevated rails (pak britain viaducts), putting a station on them, as soon as a train calls to the station the crash happens. Pure elevated rails without stop/station does not cause the crash in several tests. Saving before a train arrives also prevents the crash.

Code: [Select]

Program received signal SIGSEGV, Segmentation fault.
0x000000000044bdc4 in weg_t::get_max_weight (this=0x0)
    at boden/../boden/wege/weg.h:193
193		uint32 get_max_weight() const { return max_weight; }
(gdb) bt
#0  0x000000000044bdc4 in weg_t::get_max_weight (this=0x0)
    at boden/../boden/wege/weg.h:193
#1  0x000000000064cd4f in waggon_t::ist_befahrbar (this=0x9582f30, 
    bd=0x57dd128) at vehicle/simvehikel.cc:2927
#2  0x00000000006452e9 in vehikel_t::hop_check (this=0x9582f30)
    at vehicle/simvehikel.cc:1157
#3  0x00000000006423c7 in vehikel_basis_t::fahre_basis (this=0x9582f30, 
    distance=92082) at vehicle/simvehikel.cc:303
#4  0x00000000005a746e in convoi_t::sync_step (this=0x95828b0, delta_t=100)
    at simconvoi.cc:660
#5  0x0000000000622a69 in karte_t::sync_step (this=0x11fd240, delta_t=100, 
    sync=true, display=false) at simworld.cc:2766
#6  0x000000000062f51c in karte_t::interactive (this=0x11fd240, 
    quit_month=2147483647) at simworld.cc:5764
#7  0x00000000005e509b in simu_main (argc=1, argv=0x7fffffffe2d8)
    at simmain.cc:1076
#8  0x0000000000669392 in main (argc=1, argv=0x7fffffffe2d8) at simsys_s.cc:749

i thought i reported it earlier, but couldn't find the thread anymore.

Re: [crash devel] elevated tracks with stations, crash on weight check

Reply #1 – July 15, 2010, 10:29:01 pm

I can't reproduce this, I'm afraid. Could you upload a saved game?

Re: [crash devel] elevated tracks with stations, crash on weight check

Reply #2 – July 16, 2010, 02:19:48 am

Quote

FIX: Speculative fix for reported crash that I cannot reproduce.

i'll test it momentarily

crashed, already when running over a normal viaduct

Code: [Select]

Program received signal SIGSEGV, Segmentation fault.
0x000000000044c008 in weg_t::get_max_weight (this=0x0)
    at boden/../boden/wege/weg.h:193
193		uint32 get_max_weight() const { return max_weight; }
(gdb) bt
#0  0x000000000044c008 in weg_t::get_max_weight (this=0x0)
    at boden/../boden/wege/weg.h:193
#1  0x000000000064e564 in waggon_t::ist_befahrbar (this=0x9754a70, 
    bd=0x569a088) at vehicle/simvehikel.cc:2933
#2  0x0000000000646a11 in vehikel_t::hop_check (this=0x9754a70)
    at vehicle/simvehikel.cc:1157
#3  0x0000000000643aef in vehikel_basis_t::fahre_basis (this=0x9754a70, 
    distance=106951) at vehicle/simvehikel.cc:303
#4  0x00000000005a83d6 in convoi_t::sync_step (this=0x9754430, delta_t=100)
    at simconvoi.cc:660
#5  0x0000000000623eff in karte_t::sync_step (this=0x137a060, delta_t=100, 
    sync=true, display=false) at simworld.cc:2765
#6  0x0000000000630c42 in karte_t::interactive (this=0x137a060, 
    quit_month=2147483647) at simworld.cc:5776
#7  0x00000000005e5fd7 in simu_main (argc=1, argv=0x7fffffffe2d8)
    at simmain.cc:1076
#8  0x000000000066ae76 in main (argc=1, argv=0x7fffffffe2d8) at simsys_s.cc:749

I've overlooked something important before, which also explains why the bug does not happen after saving the game.

To cause the crash, the elevated track must be built after the train has calculated it's route already.
Recipe to cause the crash:
- build a lenght of track with two stations,
- let a train run between them.
- while the train leaves one of the stations pause the game
the timing is critical here, i think the change has to be built during the trains route finding.
a good time seems to pause is just after the train is reversed.
- remove a few pieces of track
- build the elevated way and connect it with bridge parts
- wait for the train to come.

There are some other conditions i couldn't quite understand, since this recipe does not produce the crash for some cases. It worked for 75 km/h 61 t wooden sleeper track, 3 tiles elevated way of 80t type, and 136 t brick railway viaduct. Runing a M7 steam loc with one carriage. It did never crash when i tested with a 100t diesel engine on heavy weight modern tracks/viaducts. Perhaps certain weight ratios must be given.

Some more trying, i couldn't get it to crash whatever i did, with the above setup. But on other occasions it crashed whener i did something.
It is sometimes visible that the train changes it's position (tank engines get turned around!) when it notices that the track has changed. In that case nothing happens. But sometimes it gets overlooked, the train just goes on the viaduct and the program crashes.

when pausing and changing the track right after the train reversed in a station, causes the crash also with heavy tracks and any engine.

Re: [crash devel] elevated tracks with stations, crash on weight check

Reply #3 – July 16, 2010, 09:05:41 pm

Hmm - after much tinkering, I still can't reproduce this one. Are you using a 32-bit or 64-bit platform?

Re: [crash devel] elevated tracks with stations, crash on weight check

Reply #4 – July 16, 2010, 10:18:06 pm

as always, 64 bit linux binaries, compiled with 4.4.3 gcc.

Re: [crash devel] elevated tracks with stations, crash on weight check

Reply #5 – July 21, 2010, 03:33:36 am

here's a savegame where the crash is 'primed'.
Train 841 an EMU is stuck before a newly built tunnel, if you delete it (or fix the third rail) a following steam train drives into the tunnel and crashes the game.

http://dl.dropbox.com/u/1876190/71.sve

Code: [Select]

Program received signal SIGSEGV, Segmentation fault.
0x000000000044c008 in weg_t::get_max_weight (this=0x0)
    at boden/../boden/wege/weg.h:193
193		uint32 get_max_weight() const { return max_weight; }
(gdb) bt
#0  0x000000000044c008 in weg_t::get_max_weight (this=0x0)
    at boden/../boden/wege/weg.h:193
#1  0x000000000064e564 in waggon_t::ist_befahrbar (this=0xa034e10, 
    bd=0x69adec8) at vehicle/simvehikel.cc:2933
#2  0x0000000000646a11 in vehikel_t::hop_check (this=0xa034e10)
    at vehicle/simvehikel.cc:1157
#3  0x0000000000643aef in vehikel_basis_t::fahre_basis (this=0xa034e10, 
    distance=128424) at vehicle/simvehikel.cc:303
#4  0x00000000005a83d6 in convoi_t::sync_step (this=0xa033f10, delta_t=100)
    at simconvoi.cc:660
#5  0x0000000000623eff in karte_t::sync_step (this=0x1213320, delta_t=100, 
    sync=true, display=false) at simworld.cc:2765
#6  0x0000000000630c42 in karte_t::interactive (this=0x1213320, 
    quit_month=2147483647) at simworld.cc:5776
#7  0x00000000005e5fd7 in simu_main (argc=1, argv=0x7fffffffe2d8)
    at simmain.cc:1076
#8  0x000000000066ae76 in main (argc=1, argv=0x7fffffffe2d8) at simsys_s.cc:749

the savegame requires latest development version of pak128 britain experimental and devel branch of experimental. simversion.h version numbers were stepped up to 9. But this should probably not interfere with your testing.

Re: [crash devel] elevated tracks with stations, crash on weight check

Reply #6 – July 21, 2010, 09:29:34 am

SDog,

it does mean that I cannot load the saved game until I step up the numbers myself, which may not be for some time. There may also be (and, indeed, probably will be) further changes to the saved game format between now and the time that I do so, making your saved game incompatible with all released versions.

Re: [crash devel] elevated tracks with stations, crash on weight check

Reply #7 – July 21, 2010, 08:25:34 pm

ciao james,

last time i checked the savegame could be loaded without stepping up the number. Some of the new features won't work however, which shouldn't change a lot for the crash. Besides stepping up the version in simversin.h and compiling it takes roughly a minute. let git restore the old version of the file afterwards.

sdog

ps.: just thought about the old version number again: the train causing the crash is reversing on his schedule. perhaps starting without the reverse schedule patches active could mean it doesn't find it's way to the 'crash-site'. (you should perhaps think about stepping up the version numbers in dev versions anyway. else you will always hide possible bugs from testing. only to be found when you step up the number for release.)