On Wed, Jun 14, 2006 at 12:52:51AM -0400, Dave Ahn wrote: > > The robot code does use a 2 byte local mapchars: > inl.c: char mapchars[2]; > > But uses strncpy to copy the local mapchars to the player struct: > inl.c: STRNCPY(winners[k].mapchars, j->p_mapchars, 2); Go figure I'd make a silly mistake. What I meant was that it uses strncpy to copy the local struct's p_mapchars to mapchars. Anyway, my point is that at cursory glance, some care seems to have been taken to ensure that p_mapchars and mapchars are used correctly without overruning the bounds.