Mines: Add assertions to range-check conversions to short

I think these should be adequately guarded by the new restrictions on
grid size, but I'd prefer to be sure.
This commit is contained in:
Ben Harris
2023-01-31 21:08:05 +00:00
parent c0e08f3087
commit 49841bd0fc

View File

@ -435,7 +435,9 @@ static void ss_add(struct setstore *ss, int x, int y, int mask, int mines)
* Create a set structure and add it to the tree.
*/
s = snew(struct set);
assert(SHRT_MIN <= x && x <= SHRT_MAX);
s->x = x;
assert(SHRT_MIN <= y && y <= SHRT_MAX);
s->y = y;
s->mask = mask;
s->mines = mines;
@ -506,7 +508,9 @@ static struct set **ss_overlap(struct setstore *ss, int x, int y, int mask)
/*
* Find the first set with these top left coordinates.
*/
assert(SHRT_MIN <= xx && xx <= SHRT_MAX);
stmp.x = xx;
assert(SHRT_MIN <= yy && yy <= SHRT_MAX);
stmp.y = yy;
stmp.mask = 0;