🚀 go-pugleaf

Hi, I found a problem with GNUCHESS 3.0.
I was playing a game on a PS/2 when the program
crashed during the following endgame.

  |----|----|----|----|----|----|----|----|
8 |    |    |    |    |    |    | *K |    |		BLACK'S MOVE
  +----+----+----+----+----+----+----+----+
7 |    |    |    |    |  R |    |    |  P |
  +----+----+----+----+----+----+----+----+
6 |    |    |    |    |    |    |    |    |
  +----+----+----+----+----+----+----+----+
5 |    |    |    |    |    |    |  K |    |
  +----+----+----+----+----+----+----+----+
4 |    |    |    |    |    |    |    |    |
  +----+----+----+----+----+----+----+----+
3 |    |    |    |    |    |    |    |    |
  +----+----+----+----+----+----+----+----+
2 |    |    |    |    |    |    |    |    |
  +----+----+----+----+----+----+----+----+
1 |    |    |    |    |    |    |    |    |
  |----|----|----|----|----|----|----|----|
    a    b    c    d    e    f    g    h

The program crashed at line C because at line A, whenever row(sq) == 7
(i.e. 63 <= sq <= 56) adding 8 to sq will make sq go over the array bound.
Similar kind of scenerio at line B will also make sq go below the array bound.

int ScoreKPK(side, winner, loser, king1, king2, sq)
{
    .
    .
    .
    if (winner == white) {
        .
        .
        .
        if (row(sq) < 6)	/* line A */
            sq += 16;
        else
            sq += 8;
    }
    else {
        .
        .
        .
        if (row(sq) > 1)	/* line B */
            sq -= 16;
        else
            sq -= 8;
    }
    s += 8 * (taxicab(king2, sq) - taxicab(king1, sq)); /* line C */
	/* taxicab(i,j) == taxidata[i][j] where
		taxidata is declared as taxidata[64][64] */
    return (s);
}

My question is: are the following changes correct??

	line A:				line B:

        if (row(sq) < 6)		if (row(sq) > 1)
            sq += 16;			    sq -= 16;
        else if (row(sq) == 6)		else if (row(sq) == 1)
            sq += 8;			    sq -= 8;

Thanks,
Brian