BUG/MINOR: server: Fix test on number of fields allowed in a server-state line

When a server-state line is parsed, a test is performed to be sure there is
enough but not too much fields. However the test is buggy. The bug was
introduced in the commit ea2cdf55e ("MEDIUM: server: Don't introduce a new
server-state file version").

No backport needed.
This commit is contained in:
Christopher Faulet 2021-02-20 12:15:22 +01:00
parent 938e85b228
commit 28d7876a0c

View File

@ -3166,8 +3166,8 @@ static void srv_state_parse_line(char *buf, const int version, char **params, ch
/* if line is incomplete line, then ignore it.
* otherwise, update useful flags */
if (version == 1 &&
arg < SRV_STATE_FILE_MIN_FIELDS_VERSION_1 &&
arg > SRV_STATE_FILE_MAX_FIELDS_VERSION_1)
(arg < SRV_STATE_FILE_MIN_FIELDS_VERSION_1 ||
arg > SRV_STATE_FILE_MAX_FIELDS_VERSION_1))
params[0] = NULL;
}