Tents: fix bias in tent placement.

In the very first stage of game generation, we're supposed to pick 1/5
of the grid squares to put tents in, in a uniformly random manner. It
failed to be uniform, because I had the wrong limit in random_upto -
but it was too small rather than too large, so it never overran the
buffer in a way that something like ASan or valgrind would have caught.

1 files changed, 1 insertions, 1 deletions

diff --git a/tents.c b/tents.c
index 5eb34b0..8b93653 100644
--- a/tents.c
+++ b/tents.c
@@ -998,7 +998,7 @@ static char *new_game_desc(const game_params *params_in, random_state *rs,
 	    int dy, dx;
             bool ok = true;
 
-            which = i + random_upto(rs, j);
+            which = i + random_upto(rs, w*h - i);
             tmp = order[which];
             order[which] = order[i];
             order[i] = tmp;