aboutsummaryrefslogtreecommitdiff
path: root/samegame.c
diff options
context:
space:
mode:
authorBen Harris <bjh21@bjh21.me.uk>2023-01-10 10:54:37 +0000
committerBen Harris <bjh21@bjh21.me.uk>2023-01-15 16:24:27 +0000
commitd5ec2758ee3b7a8934475a1122813ad2312dc850 (patch)
tree13563867c63f5f2eabbfaa22b4b1f6f5d4441587 /samegame.c
parentb090c82df1527dcf348c96765f10ab5736c68c29 (diff)
downloadpuzzles-d5ec2758ee3b7a8934475a1122813ad2312dc850.zip
puzzles-d5ec2758ee3b7a8934475a1122813ad2312dc850.tar.gz
puzzles-d5ec2758ee3b7a8934475a1122813ad2312dc850.tar.bz2
puzzles-d5ec2758ee3b7a8934475a1122813ad2312dc850.tar.xz
Last-ditch maximum size limit for Same Game
This makes sure that width * height <= INT_MAX, which it rather needs to be.
Diffstat (limited to 'samegame.c')
-rw-r--r--samegame.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/samegame.c b/samegame.c
index 3597e19..3abd042 100644
--- a/samegame.c
+++ b/samegame.c
@@ -67,6 +67,7 @@
#include <string.h>
#include <assert.h>
#include <ctype.h>
+#include <limits.h>
#include <math.h>
#include "puzzles.h"
@@ -285,6 +286,8 @@ static const char *validate_params(const game_params *params, bool full)
{
if (params->w < 1 || params->h < 1)
return "Width and height must both be positive";
+ if (params->w > INT_MAX / params->h)
+ return "Width times height must not be unreasonably large";
if (params->ncols > 9)
return "Maximum of 9 colours";