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.