From cf6e4e272c2aa3477f149eb75ffe0436e1a8a2b0 Mon Sep 17 00:00:00 2001 From: Simon Tatham Date: Sun, 11 Dec 2016 09:19:30 +0000 Subject: [PATCH] Correct a logic error in Unequal game desc validation. A user points out that the error check that should have detected a non-digit where a digit should have been was never firing, due to an && that should have been ||. I don't think it was a harmful error - the subsequent check that the number was in range, plus the skipping past only digits to find the next part of the string, combine to arrange that not many kinds of invalid game id could actually get through. But it did have the small effect that a 0 could be elided without triggering an error, e.g. the game ids 4:0,0,0,0,0,0L,0,0,0R,0U,0,0L,0,0,,3, 4:0,0,0,0,0,0L,0,0,0R,0U,0,0L,0,0,0,3, would both be accepted, and would be decoded into the same game, even though the former should have failed syntax validation. Now it does. --- unequal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/unequal.c b/unequal.c index 1c8984a..3664788 100644 --- a/unequal.c +++ b/unequal.c @@ -1214,7 +1214,7 @@ static game_state *load_game(const game_params *params, const char *desc, why = "Too much data to fill grid"; goto fail; } - if (*p < '0' && *p > '9') { + if (*p < '0' || *p > '9') { why = "Expecting number in game description"; goto fail; } n = atoi(p); -- 2.30.2