shithub: puzzles

Download patch

ref: 6e40605f1ed4ecce400faae5b41c03995e7f862c
parent: 8a3fb82e230c5cf18b82f54687c2a56f53875a38
author: Ben Harris <bjh21@bjh21.me.uk>
date: Mon Jan 9 19:30:48 EST 2023

Last-ditch maximum size limit for Pegs

This makes sure that width * height <= INT_MAX, which it rather needs
to be.

--- a/pegs.c
+++ b/pegs.c
@@ -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 @@
 {
     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 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";