[Freeciv-Dev] Re: bug x 2 (PR#333)

[Thue Janus Kristensen <thue@xxxxxxx>]

On second thought I am not quite sure that the first crash is fixed by the
patch I send. Could you give exstra info, especially about the mapsize
you used and if you created tha ai's and connected the client before or
after you specified the map x and y size.

If it isn't fixed I can't just see what it could be; lets wait a little
and see if the problem reoccurs.

(the problem seems to be trash valued for the pdcity pointer in a
player_tile, which could concievably be caused by allocation problems,
and therefore possibly fixed by the patch, but I can't quite see how)

-Thue

> Civserver (cvs 8.4.) crash while completing the first savegame in that
game.  > 
> ...
> 1: Map generator: not enough start positions, fixing.
> 1: last message repeated 2 times
> 1: last message repeated 2 times (total 4 repeats)
> 1: last message repeated 4 times (total 8 repeats)
> 1: last message repeated 8 times (total 16 repeats)
> 1: last message repeated 4 times (total 20 repeats)
> 1: The map has 24 starting positions on 6 isles.
> > save /tuma/civ/tuma8.4.2k.sav
> Segmentation fault (core dumped)
> [tuma@paaskyniemi civ]$ gdb ../download/freeciv_build/server/civserver core
> GNU gdb 19991004
> Copyright 1998 Free Software Foundation, Inc.
> GDB is free software, covered by the GNU General Public License, and you are
> welcome to change it and/or distribute copies of it under certain conditions.
> Type "show copying" to see the conditions.
> There is absolutely no warranty for GDB.  Type "show warranty" for details.
> This GDB was configured as "i386-redhat-linux"...
> Core was generated by `../download/freeciv_build/server/civserver'.
> Program terminated with signal 11, Segmentation fault.
> Reading symbols from /lib/libc.so.6...done.
> Reading symbols from /lib/ld-linux.so.2...done.
> Reading symbols from /lib/libnss_files.so.2...done.
> #0  0x80677db in player_save (plr=0x821faf4, plrno=0, file=0xbfffd678) at 
> plrhand.c:2746
> 2746                secfile_insert_int(file, pdcity->id, "player%d.dc%d.id", 
> plrno, i);
> (gdb) bt
> #0  0x80677db in player_save (plr=0x821faf4, plrno=0, file=0xbfffd678) at 
> plrhand.c:2746
> #1  0x8057e8b in game_save (file=0xbfffd678) at gamehand.c:665
> #2  0x8053016 in save_game (filename=0xbfffd6cc "/tuma/civ/tuma8.4.2k.sav") 
> at civserver.c:848
> #3  0x807211c in save_command (caller=0x0, arg=0xbfffd6cc 
> "/tuma/civ/tuma8.4.2k.sav") at stdinhand.c:1350
> #4  0x8074dbf in handle_stdin_input (caller=0x0, str=0xbffff71c "save 
> /tuma/civ/tuma8.4.2k.sav\n")
>     at stdinhand.c:2249
> #5  0x806c6a5 in sniff_packets () at sernet.c:196
> #6  0x8052655 in main (argc=1, argv=0xbffffa44) at civserver.c:492            
>