From f2b82c1e1d67ce3edec1b4da005c1ab46f27633e Mon Sep 17 00:00:00 2001 From: Miroslav Lichvar Date: Tue, 24 Nov 2015 12:30:54 +0100 Subject: [PATCH] conf: don't allow disabling clientloglimit Don't treat zero as a special value disabling clientloglimit. It's not useful, the amount of available memory is never unlimited. --- chrony.texi.in | 8 +++++--- conf.c | 5 ----- 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/chrony.texi.in b/chrony.texi.in index 895b9d51..639b58d8 100644 --- a/chrony.texi.in +++ b/chrony.texi.in @@ -1355,9 +1355,11 @@ directive}). @subsection clientloglimit This directive specifies the maximum size of the memory allocated to log client accesses. When the limit is reached, only information for -clients that have already been logged will be updated. If 0 is -specified, the memory size will be unlimited. The default is 524288 -bytes. +clients that have already been logged will be updated. The default is +524288 bytes. + +In older @code{chrony} versions if the limit was set to 0, the memory +allocation was unlimited. An example of the use of this directive is diff --git a/conf.c b/conf.c index 56266964..3fbdbc04 100644 --- a/conf.c +++ b/conf.c @@ -862,11 +862,6 @@ parse_clientloglimit(char *line) if (sscanf(line, "%lu", &client_log_limit) != 1) { command_parse_error(); } - - if (client_log_limit == 0) { - /* unlimited */ - client_log_limit = (unsigned long)-1; - } } /* ================================================== */ -- 2.47.2