[Freeciv-Dev] Re: (PR#7216) take again client crash

["Raimar Falke" <i-freeciv-lists@xxxxxxxxxxxxx>]

<URL: http://rt.freeciv.org/Ticket/Display.html?id=7216 >

On Thu, Jan 08, 2004 at 04:23:54AM -0800, Per I. Mathisen wrote:
> 
> <URL: http://rt.freeciv.org/Ticket/Display.html?id=7216 >
> 
> You crash the client (gtk2, CVS HEAD) by doing 'take' twice.
> 
> To reproduce:
> Start savegame. Login as 'me', then 'take me player1'.
> Then 'take me player2'.
> *booom*

map.tiles is NULL at the client. It looks like packet_map_info is
missing somehow. However coded the take command should over it again
and see that map related packets (packet_map_info and
packet_game_state) match.

I also propose a patch like this or similar:

 ***************************************************************/
 struct tile *map_get_tile(int x, int y)
 {
+  assert(map.tiles != NULL);
   return MAP_TILE(x, y);
 }

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "Reality? That's where the pizza delivery guy comes from!"