News:

SimuTranslator
Make Simutrans speak your language.

All servers are down

Started by Phystam, August 12, 2021, 07:15:01 AM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Phystam

Today's nightly update causes segmentation faults in all servers... However, the graphical client can successfully load the save data.

jamespetts

#1
Thank you for the report. Looking into this now. The first indication that I have is the following backtrace:


Program received signal SIGSEGV, Segmentation fault.
0x000055555568f7d5 in ground_desc_t::register_desc(ground_desc_t const*) ()
(gdb) backtrace
#0  0x000055555568f7d5 in ground_desc_t::register_desc(ground_desc_t const*) ()
#1  0x00005555556998a0 in obj_reader_t::read_nodes(_IO_FILE*, obj_desc_t*&, int, unsigned int) ()
#2  0x0000555555699884 in obj_reader_t::read_nodes(_IO_FILE*, obj_desc_t*&, int, unsigned int) ()
#3  0x00005555556999a8 in obj_reader_t::read_file(char const*) ()
#4  0x000055555569a15f in obj_reader_t::load(char const*, char const*) ()
#5  0x0000555555a1cd62 in simu_main(int, char**) ()
#6  0x0000555555aa12b9 in sysmain(int, char**) ()
#7  0x00007ffff79900b3 in __libc_start_main (main=0x55555565a600 <main>, argc=25, argv=0x7fffffffe208, init=<optimized out>,
    fini=<optimized out>, rtld_fini=<optimized out>, stack_end=0x7fffffffe1f8) at ../csu/libc-start.c:308
#8  0x000055555565a66e in _start ()

Edit: Preliminary tests suggest that reverting commit bb2e73bb935f5eaeb89b97b654503c023de38715 may fix the issue. Running a recompile on the server now to test this.
Download Simutrans-Extended.

Want to help with development? See here for things to do for coding, and here for information on how to make graphics/objects.

Follow Simutrans-Extended on Facebook.

jamespetts

Now back up and running again - the above reversion indeed succeeded.
Download Simutrans-Extended.

Want to help with development? See here for things to do for coding, and here for information on how to make graphics/objects.

Follow Simutrans-Extended on Facebook.

Ranran

Thank you for fixing it. The peripheral code for that commit looked similar to standard, but it could have been missing elsewhere. I didn't notice the problem as I was able to load the offline game without problems even after merging it.

TurfIt

The fix for the reverted code is Standard r9707.

Phystam

This bug seems to be fixed. Thank you all!