Commit Diff

Commit:
a5b487dfe4b4b065a6cde35f9385926719478c91
From:
Markus Uhlin <markus@nifty-networks.net>
Date:
Message:
WriteMoves: fixed possible array underrun
Actions:
Patch | Tree
commit - 160c816f2b1b6c62207f74670cd3ce822d08a4dc
commit + a5b487dfe4b4b065a6cde35f9385926719478c91
blob - 530eb47f3a062da621a616ba0b1b0cc28a0f2b28
blob + 572abf52a3931a1c64e90ec9b775204451621f7f
--- FICS/gamedb.c
+++ FICS/gamedb.c
@@ -886,7 +886,8 @@ WriteMoves(FILE *fp, move_t *m)
 
 	/* Are we using from-file or from-rank in 'algString'? */
 
-	i = strlen(m->algString) - 1;
+	if ((i = strlen(m->algString)) > 0)
+		i -= 1;
 
 	if (m->algString[i] == '+') {
 		check = 1;