diff options
author | beck <beck@openbsd.org> | 2009-12-10 23:53:03 +0000 |
---|---|---|
committer | beck <beck@openbsd.org> | 2009-12-10 23:53:03 +0000 |
commit | b38b2a39cc55772769088c999ec380535b351725 (patch) | |
tree | 5d15dbaced43d5c3fa00004738ce0a4a3a235bfe /games/mille | |
parent | getcwd() is posix, not ansi; from Thomas Pfaff (diff) | |
download | wireguard-openbsd-b38b2a39cc55772769088c999ec380535b351725.tar.xz wireguard-openbsd-b38b2a39cc55772769088c999ec380535b351725.zip |
fix potential bugs spotted by parfait
ok millert@ pjanzen@
Diffstat (limited to 'games/mille')
-rw-r--r-- | games/mille/move.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/games/mille/move.c b/games/mille/move.c index 9e1f97af09a..9a406bb06be 100644 --- a/games/mille/move.c +++ b/games/mille/move.c @@ -1,4 +1,4 @@ -/* $OpenBSD: move.c,v 1.11 2009/10/27 23:59:25 deraadt Exp $ */ +/* $OpenBSD: move.c,v 1.12 2009/12/10 23:53:05 beck Exp $ */ /* $NetBSD: move.c,v 1.4 1995/03/24 05:01:57 cgd Exp $ */ /* @@ -130,11 +130,11 @@ acc: else for (i = 1; i < HAND_SZ; i++) if (pp->hand[i] == C_INIT) { - for (j = 0; pp->hand[j] == C_INIT; j++) - if (j >= HAND_SZ) { - j = 0; - break; - } + for (j = 0; (j < HAND_SZ) && + (pp->hand[j] == C_INIT); j++) + ; + if (j == HAND_SZ) + j = 0; pp->hand[i] = pp->hand[j]; pp->hand[j] = C_INIT; } |