diff options
| author | Ben Harris <bjh21@bjh21.me.uk> | 2023-01-10 00:30:48 +0000 |
|---|---|---|
| committer | Ben Harris <bjh21@bjh21.me.uk> | 2023-01-15 16:24:27 +0000 |
| commit | 6e40605f1ed4ecce400faae5b41c03995e7f862c (patch) | |
| tree | a598952e49b92e720365af0f80ebc7ac51c17b95 | |
| parent | 8a3fb82e230c5cf18b82f54687c2a56f53875a38 (diff) | |
| download | puzzles-6e40605f1ed4ecce400faae5b41c03995e7f862c.zip puzzles-6e40605f1ed4ecce400faae5b41c03995e7f862c.tar.gz puzzles-6e40605f1ed4ecce400faae5b41c03995e7f862c.tar.bz2 puzzles-6e40605f1ed4ecce400faae5b41c03995e7f862c.tar.xz | |
Last-ditch maximum size limit for Pegs
This makes sure that width * height <= INT_MAX, which it rather needs
to be.
| -rw-r--r-- | pegs.c | 7 |
1 files changed, 6 insertions, 1 deletions
@@ -7,6 +7,7 @@ #include <string.h> #include <assert.h> #include <ctype.h> +#include <limits.h> #include <math.h> #include "puzzles.h" @@ -182,6 +183,8 @@ static const char *validate_params(const game_params *params, bool full) { if (full && (params->w <= 3 || params->h <= 3)) return "Width and height must both be greater than three"; + if (params->w > INT_MAX / params->h) + return "Width times height must not be unreasonably large"; /* * It might be possible to implement generalisations of Cross @@ -658,7 +661,9 @@ static char *new_game_desc(const game_params *params, random_state *rs, static const char *validate_desc(const game_params *params, const char *desc) { - int len = params->w * params->h; + int len; + + len = params->w * params->h; if (len != strlen(desc)) return "Game description is wrong length"; |